Regression: new OBJ importer fails to load geometry using TAB character as delimiter #97417

New Issue

Jesse Yurkovich · 2022-04-18T04:37:45+02:00

Jesse Yurkovich commented

2022-04-18 04:37:45 +02:00

System Information
Operating system: Windows-10-10.0.19044-SP0 64 Bits
Graphics card: NVIDIA GeForce GTX 1070 with Max-Q Design/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 511.09

Blender Version
Broken: version: 3.2.0 Alpha, branch: master, commit date: 2022-04-17 22:21, hash: bf80dc2bd4
Worked: (newest version of Blender that worked as expected)

Short description of error
I have an older set of Cornell box .OBJ models where 2 of the simpler ones fail to load (oddly enough). The existing python importer seems to handle them better.

Exact steps for others to reproduce the error
cbox.zip

Unpack the provided file above
- Contains the .OBJs and a test.blend file
Open the provided blend and run the script in the editor
Observe that the 2 models referenced in the screenshot below show some degenerate/collapsed geometry
- The 2 models are "CornellBox-Original.obj" and "CornellBox-Mirror.obj"
- There's a few other models that are missing their right-side walls as well

Note: Consider using the filename for the imported object name instead of "New object" if no other name is provided.

This caused by tabs as delimiters in some occasions.
For example CornellBox-Original.obj:
new importer chokes on:

## Object rightwall 
v	1.00  0.00  -1.04   
v	1.00  0.00   0.99
v	1.00  1.99   0.99
v	1.00  1.99  -1.04

this would read in fine

## Object rightwall 
v 1.00  0.00  -1.04   
v 1.00  0.00   0.99
v 1.00  1.99   0.99
v 1.

**System Information** Operating system: Windows-10-10.0.19044-SP0 64 Bits Graphics card: NVIDIA GeForce GTX 1070 with Max-Q Design/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 511.09 **Blender Version** Broken: version: 3.2.0 Alpha, branch: master, commit date: 2022-04-17 22:21, hash: `bf80dc2bd4` Worked: (newest version of Blender that worked as expected) **Short description of error** I have an older set of Cornell box .OBJ models where 2 of the simpler ones fail to load (oddly enough). The existing python importer seems to handle them better. **Exact steps for others to reproduce the error** [cbox.zip](https://archive.blender.org/developer/F13007213/cbox.zip) - Unpack the provided file above - Contains the .OBJs and a `test.blend` file - Open the provided blend and run the script in the editor - Observe that the 2 models referenced in the screenshot below show some degenerate/collapsed geometry - The 2 models are "CornellBox-Original.obj" and "CornellBox-Mirror.obj" - There's a few other models that are missing their right-side walls as well Note: Consider using the filename for the imported object name instead of "New object" if no other name is provided. ![obj-bad-geo.png](https://archive.blender.org/developer/F13007203/obj-bad-geo.png) This caused by `tabs` as delimiters in some occasions. For example `CornellBox-Original.obj`: new importer chokes on: ``` ## Object rightwall v 1.00 0.00 -1.04 v 1.00 0.00 0.99 v 1.00 1.99 0.99 v 1.00 1.99 -1.04 ``` this would read in fine ``` ## Object rightwall v 1.00 0.00 -1.04 v 1.00 0.00 0.99 v 1.00 1.99 0.99 v 1.

Jesse Yurkovich commented

2022-04-18 04:37:45 +02:00

Added subscriber: @deadpin

Aras Pranckevicius commented

2022-04-18 08:43:03 +02:00

Added subscriber: @aras_p

Pratik Borhade commented

2022-04-19 05:08:12 +02:00

Changed status from 'Needs Triage' to: 'Confirmed'

Pratik Borhade commented

2022-04-19 05:08:16 +02:00

Added subscriber: @PratikPB2123

Aras Pranckevicius self-assigned this 2022-04-19 19:50:52 +02:00

Philipp Oeser commented

2022-04-26 14:36:53 +02:00

Added subscriber: @lichtwerk

Philipp Oeser commented

2022-04-26 14:36:53 +02:00

This caused by tabs as delimiters in some occasions.
For example CornellBox-Original.obj:
new importer chokes on:

## Object rightwall 
v	1.00  0.00  -1.04   
v	1.00  0.00   0.99
v	1.00  1.99   0.99
v	1.00  1.99  -1.04

this would read in fine

## Object rightwall 
v 1.00  0.00  -1.04   
v 1.00  0.00   0.99
v 1.00  1.99   0.99
v 1.00  1.99  -1.04

I could not find anything in the specification(s) that clearly say which delimiters to use, but I do think this is a regression (not sure how many applications are out there writing with tabs).
Will mark regression with high prio.

This caused by `tabs` as delimiters in some occasions. For example `CornellBox-Original.obj`: new importer chokes on: ``` ## Object rightwall v 1.00 0.00 -1.04 v 1.00 0.00 0.99 v 1.00 1.99 0.99 v 1.00 1.99 -1.04 ``` this would read in fine ``` ## Object rightwall v 1.00 0.00 -1.04 v 1.00 0.00 0.99 v 1.00 1.99 0.99 v 1.00 1.99 -1.04 ``` I could not find anything in the specification(s) that clearly say which delimiters to use, but I do think this is a regression (not sure how many applications are out there writing with tabs). Will mark regression with high prio.

Philipp Oeser changed title from ~~New OBJ importer fails to load some geometry from older files~~ to Regression: new OBJ importer fails to load geometry using TAB character as delimiter

2022-04-26 14:37:59 +02:00

blender-admin commented

2022-05-01 19:07:03 +02:00

This issue was referenced by 33518f9da1

This issue was referenced by 33518f9da160cf2d9e91dd7a69cd5788303dcca4

Aras Pranckevicius commented

2022-05-01 19:07:27 +02:00

Changed status from 'Confirmed' to: 'Resolved'

Aras Pranckevicius closed this issue

2022-05-01 19:07:27 +02:00

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

Regression: new OBJ importer fails to load geometry using TAB character as delimiter #97417