Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

'Variable' object has no attribute 'shape' [v0.2] #2315

Closed
miguelvr opened this issue Aug 7, 2017 · 3 comments
Closed

'Variable' object has no attribute 'shape' [v0.2] #2315

miguelvr opened this issue Aug 7, 2017 · 3 comments

Comments

@miguelvr
Copy link

miguelvr commented Aug 7, 2017

Apparently x.shape works if x is a tensor, but not if it is a Variable.
It is possible to do x.data.shape and x.size() is x is a Variable, but not x.shape
This should be the same for consistency, no matter if Variable or Tensor.
@fmassa
Copy link
Member

fmassa commented Aug 7, 2017

This has just been fixed on master, and will be present in the next release #2306

@soumith soumith closed this as completed Aug 7, 2017
@acgtyrant
Copy link
Contributor

What is the next release, 0.3 or 0.2.x?

@fmassa
Copy link
Member

fmassa commented Sep 13, 2017

@acgtyrant probably 0.2.x

@vsoch vsoch mentioned this issue Feb 8, 2018
Open
@houseroad houseroad mentioned this issue Sep 16, 2019
Closed
houseroad added a commit to houseroad/pytorch that referenced this issue Sep 17, 2019
@houseroad @facebook-github-bot
Automatic update of fbcode/onnx to 1316afc9f972f81340faa05763e2898f38…
8ac9087 
…bcc3b0 (pytorch#26309)

Summary:
Pull Request resolved: pytorch#26309

Previous import was 95252c2adec185e305e34486c6756ece9aa8f57f

Included changes:
- **[1316afc9](onnx/onnx@1316afc9)**: Update IR doc to clarify initializers are permitted as node inputs (pytorch#2320) <G. Ramalingam>
- **[5e920d0c](onnx/onnx@5e920d0c)**: Avoid uses of special chars (pytorch#2315) <Wei-Sheng Chin>
- **[2fa08b0f](onnx/onnx@2fa08b0f)**: Regenerate ONNX proto and add release date to ver 6 IR (pytorch#2316) <Wei-Sheng Chin>
- **[adf9c7a3](onnx/onnx@adf9c7a3)**: Add description of default type about y_zero_point (pytorch#2110) <Takeshi Watanabe>
- **[ee7072c7](onnx/onnx@ee7072c7)**: Support make_attribute empty string (pytorch#2129) <shjwudp>
- **[f913b6e7](onnx/onnx@f913b6e7)**: More unsqueeze tests (pytorch#2200) <James Allingham>
- **[57b51937](onnx/onnx@57b51937)**: Fix resize shape inference issue in opset10 (pytorch#2294) <Bowen Bao>
- **[d7595f34](onnx/onnx@d7595f34)**: Sequence related ops (pytorch#2249) <Bowen Bao>
- **[599f3da9](onnx/onnx@599f3da9)**: Add helper function update_inputs_outputs_dims to tools (pytorch#2148) <Bowen Bao>
- **[3e6382bc](onnx/onnx@3e6382bc)**: Update documentation about required input output types (pytorch#2310) <G. Ramalingam>
- **[0c765d9b](onnx/onnx@0c765d9b)**: Shape inference for NMS (pytorch#2269) <Hariharan Seshadri>
- **[89266710](onnx/onnx@89266710)**: Fix extra collect_snippets warning (pytorch#2277) (pytorch#2307) <Lutz Roeder>

Test Plan: ci

Reviewed By: hl475

Differential Revision: D17403954

fbshipit-source-id: 8f7cbf77b7e09b73db2015f3e0f436772482b322
facebook-github-bot pushed a commit that referenced this issue Sep 17, 2019
@houseroad @facebook-github-bot
Automatic update of fbcode/onnx to 1316afc9f972f81340faa05763e2898f38…
bebc3d6 
…bcc3b0 (#26309)

Summary:
Pull Request resolved: #26309

Previous import was 95252c2adec185e305e34486c6756ece9aa8f57f

Included changes:
- **[1316afc9](onnx/onnx@1316afc9)**: Update IR doc to clarify initializers are permitted as node inputs (#2320) <G. Ramalingam>
- **[5e920d0c](onnx/onnx@5e920d0c)**: Avoid uses of special chars (#2315) <Wei-Sheng Chin>
- **[2fa08b0f](onnx/onnx@2fa08b0f)**: Regenerate ONNX proto and add release date to ver 6 IR (#2316) <Wei-Sheng Chin>
- **[adf9c7a3](onnx/onnx@adf9c7a3)**: Add description of default type about y_zero_point (#2110) <Takeshi Watanabe>
- **[ee7072c7](onnx/onnx@ee7072c7)**: Support make_attribute empty string (#2129) <shjwudp>
- **[f913b6e7](onnx/onnx@f913b6e7)**: More unsqueeze tests (#2200) <James Allingham>
- **[57b51937](onnx/onnx@57b51937)**: Fix resize shape inference issue in opset10 (#2294) <Bowen Bao>
- **[d7595f34](onnx/onnx@d7595f34)**: Sequence related ops (#2249) <Bowen Bao>
- **[599f3da9](onnx/onnx@599f3da9)**: Add helper function update_inputs_outputs_dims to tools (#2148) <Bowen Bao>
- **[3e6382bc](onnx/onnx@3e6382bc)**: Update documentation about required input output types (#2310) <G. Ramalingam>
- **[0c765d9b](onnx/onnx@0c765d9b)**: Shape inference for NMS (#2269) <Hariharan Seshadri>
- **[89266710](onnx/onnx@89266710)**: Fix extra collect_snippets warning (#2277) (#2307) <Lutz Roeder>

Test Plan: ci

Reviewed By: hl475

Differential Revision: D17403954

fbshipit-source-id: 78a9c3ecf5aa7f7a0ba8ea30286eab61ee903772
samnordmann pushed a commit to samnordmann/pytorch that referenced this issue Jan 12, 2023
@naoyam
Revert CMakeLists.txt as build fails with clang due to unused variabl…
98029e4 
…es (pytorch#2315)
jjsjann123 added a commit to jjsjann123/pytorch that referenced this issue Jan 17, 2023
@jjsjann123
Code cleaning (pytorch#2320)
1b6393c 
Revert "Revert CMakeLists.txt as build fails with clang due to unused variables (pytorch#2315)", so more compiler warning are enabled.
fixing compiler warnings. Verified on both gcc & clang-14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@soumith @acgtyrant @miguelvr @fmassa