Regression: New OBJ Exporter Incorrectly saving the materials in the MTL (materials) file, extra space before texture files #96763

New Issue

Alexandre Mano · 2022-03-24T17:51:49+01:00

Alexandre Mano commented

2022-03-24 17:51:49 +01:00

System Information
Operating system: Windows 10
Graphics card: RTX2060

Blender Version
Broken: 3.2.0 Alpha (98d4872d7f)
Worked: 3.00 Alpha (d7b231baa8)

After exporting a model with the new Obj exporter, the MTL materials file of said model is incapable of correctly linking to the needed texture files.
In the produced MTL file, the line with "map_Kd" is written with an extra space in between the map_Kd call and the texture file, which results in some programs like "Microsoft 3D Builder" being unable to import said texture.

Exact steps for others to reproduce the error
In Blender 3.20, export any mesh model with a material linking to a diffuse texture file. Openning it in another 3d file viewer program (for example Microsoft 3D Builder or 3D Viewer) will cause texture issues.

**System Information** Operating system: Windows 10 Graphics card: RTX2060 **Blender Version** Broken: 3.2.0 Alpha (98d4872d7f42) Worked: 3.00 Alpha (d7b231baa87e) After exporting a model with the new Obj exporter, the MTL materials file of said model is incapable of correctly linking to the needed texture files. In the produced MTL file, the line with "map_Kd" is written with an extra space in between the map_Kd call and the texture file, which results in some programs like "Microsoft 3D Builder" being unable to import said texture. **Exact steps for others to reproduce the error** In Blender 3.20, export any mesh model with a material linking to a diffuse texture file. Openning it in another 3d file viewer program (for example Microsoft 3D Builder or 3D Viewer) will cause texture issues.

Alexandre Mano commented

2022-03-24 17:51:49 +01:00

Added subscriber: @Xande

Omar Emara commented

2022-03-25 08:29:46 +01:00

Added subscriber: @OmarEmaraDev

Omar Emara commented

2022-03-25 08:29:46 +01:00

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

Omar Emara commented

2022-03-25 08:29:46 +01:00

I can confirm this behavior. It is unclear if the specification mandate a single space, but this seems like an easy fix, so confirming.

Ankit Meel commented

2022-03-25 08:42:59 +01:00

Added subscriber: @ankitm

Ankit Meel commented

2022-03-25 08:42:59 +01:00

somewhere in the code is written that file names can also have leading spaces, so only one space should be used for disambiguation

Philipp Oeser commented

2022-03-25 14:57:21 +01:00

Added subscriber: @lichtwerk

Philipp Oeser commented

2022-03-25 14:57:21 +01:00

Well, if this is a regression, it should be High prio I think.

Philipp Oeser changed title from ~~New OBJ Exporter Incorrectly saving the materials in the MTL (materials) file, extra space before texture files~~ to Regression: New OBJ Exporter Incorrectly saving the materials in the MTL (materials) file, extra space before texture files

2022-03-25 14:57:40 +01:00

Aras Pranckevicius commented

2022-03-30 08:34:16 +02:00

Added subscriber: @aras_p

Aras Pranckevicius self-assigned this 2022-03-30 11:27:46 +02:00

Aras Pranckevicius commented

2022-03-30 11:27:46 +02:00

Yeah, I see the problem. There's a code comment saying "make sure to emit only one space" and the code right below it proceeds to emit two spaces :) I'll fix this.

Alexandre Mano commented

2022-03-30 12:27:56 +02:00

Nice, thanks! It's very easy to fix the files after exporting, but without previous knowledge of the bug it could be confused with user error during model preparation (I sure did for a while).

Aras Pranckevicius commented

2022-03-31 09:34:08 +02:00

Note to self: while trying to fix this and add test coverage, I noticed some other issues with .mtl export that I'll try to fix in the same revision:

Emissive texture map (map_Ke) is not exported,
When Mapping node is used on the texture UVs, the "Location" and "Scale" values were mixed up (location written as "scale", scale written as "location),
The texture filenames are always written as being in the same folder, effectively as if "Path Mode: Strip Path" option was used in the old exporter. The path mode setting is gone in the new exporter, seemingly accidentally, I'll try to bring it back.

Note to self: while trying to fix this and add test coverage, I noticed some other issues with .mtl export that I'll try to fix in the same revision: - Emissive texture map (map_Ke) is not exported, - When Mapping node is used on the texture UVs, the "Location" and "Scale" values were mixed up (location written as "scale", scale written as "location), - The texture filenames are always written as being in the same folder, effectively as if "Path Mode: Strip Path" option was used in the old exporter. The path mode setting is gone in the new exporter, seemingly accidentally, I'll try to bring it back.

blender-admin commented

2022-03-31 12:38:59 +02:00

This issue was referenced by d120a083da

This issue was referenced by d120a083da1f4bbead4895209dd064d1455bc7d6

Aras Pranckevicius commented

2022-04-01 14:00:11 +02:00

Changed status from 'Confirmed' to: 'Resolved'

Aras Pranckevicius closed this issue

2022-04-01 14:00:11 +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 Exporter Incorrectly saving the materials in the MTL (materials) file, extra space before texture files #96763