New OBJ importer: material library filenames might be quoted (see T67266) #97794

New Issue

Jesse Yurkovich · 2022-05-03T07:41:03+02:00

Jesse Yurkovich commented

2022-05-03 07:41:03 +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-05-02 09:18, hash: f0f44fd92f

Short description of error
Going through old reports. It seems as though some software or existing .obj files may be using quotes surrounding their material library names.

There's nothing in the spec in this regard really but a decision was made for the python importer to deal with it...

Exact steps for others to reproduce the error

Load the provided .obj with .mtl file from blender/blender-addons#67266
Observe failure to load the library

OBJ import: cannot read from MTL file: 'G:\triage\"name with blanks.mtl"'

Note After D14822 is checked in that particular test will work by accident since another material library, with a proper name, will be added implicitly. However, the scenario won't work generally.

Inside the OBJ file it looks like mtllib "name with blanks.mtl"

**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-05-02 09:18, hash: `f0f44fd92f` **Short description of error** Going through old reports. It seems as though some software or existing .obj files may be using quotes surrounding their material library names. There's nothing in the spec in this regard really but a decision was made for the python importer to deal with it... **Exact steps for others to reproduce the error** - Load the provided .obj with .mtl file from blender/blender-addons#67266 - Observe failure to load the library `OBJ import: cannot read from MTL file: 'G:\triage\"name with blanks.mtl"'` **Note** After [D14822](https://archive.blender.org/developer/D14822) is checked in that particular test will work by accident since another material library, with a proper name, will be added implicitly. However, the scenario won't work generally. Inside the OBJ file it looks like `mtllib "name with blanks.mtl"`

Jesse Yurkovich commented

2022-05-03 07:41:03 +02:00

Added subscriber: @deadpin

Pratik Borhade commented

2022-05-03 11:38:48 +02:00

Added subscriber: @PratikPB2123

Pratik Borhade commented

2022-05-03 11:38:48 +02:00

I'm not able to repro this in current master.

I mean I don't get any error when loading those files. Maybe I'm missing something for reproducing it correctly

I'm not able to repro this in current master. I mean I don't get any error when loading those files. Maybe I'm missing something for reproducing it correctly

Aras Pranckevicius commented

2022-05-03 12:58:47 +02:00

Added subscriber: @aras_p

Jesse Yurkovich commented

2022-05-03 19:53:25 +02:00

Try with the following set of files. You'll notice the error inside the terminal and that the object's Material is incorrect (base color should be Green; can compare with python importer)

test spaces lib.mtl

test spaces.obj

Try with the following set of files. You'll notice the error inside the terminal and that the object's Material is incorrect (base color should be Green; can compare with python importer) [test spaces lib.mtl](https://archive.blender.org/developer/F13049700/test_spaces_lib.mtl) [test spaces.obj](https://archive.blender.org/developer/F13049699/test_spaces.obj)

Pratik Borhade commented

2022-05-04 04:31:53 +02:00

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

Aras Pranckevicius self-assigned this 2022-05-04 05:47:59 +02:00

blender-admin commented

2022-05-04 14:10:54 +02:00

This issue was referenced by cbeb8770cc

This issue was referenced by cbeb8770cc4daec5c81f8d8ce060a1b594c039bb

Aras Pranckevicius commented

2022-05-04 14:11:07 +02:00

Changed status from 'Confirmed' to: 'Resolved'

Aras Pranckevicius closed this issue

2022-05-04 14:11:07 +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

New OBJ importer: material library filenames might be quoted (see T67266) #97794