Age | Commit message (Collapse) | Author |
|
The addon would assume an OBJ range of [0.0-900.0] for the Ns value,
when it actually is supposed to be [0.0-1000.0].
WARNING: This is introducing a slight incompatibility (value shifting of
the roughness parameter) with older OBJ files exported by Blender.
|
|
Contributed by luzpaz.
Differential Revision: https://developer.blender.org/D5801
|
|
|
|
Also remove commented `use_mesh_modifiers_render` option which is
unlikely to be added back since this option no longer exists internally.
|
|
OBJ files require that parameters are specified after every line start element except the "end" command.
This patch skips all lines that are missing that information unless there is a multi line context.
Reviewed By: mont29
Maniphest Tasks: T83671
Differential Revision: https://developer.blender.org/D9828
|
|
In rBA6aa8e130eff59059886e203ff95221609f63b222 all occurrences
of "lamp" where replaced with "light" which also accidentally
renamed "clamp" to "clight". In case of the x3d importer and
object carver add-on this broke some functionality. This
commit fixes the names to match the use the correct properties
of the Python API and use semantically correct names for other
add-on where the renaming didn't cause functional changes.
Reviewed By: mont29
Differential Revision: https://developer.blender.org/D9782
|
|
|
|
Since panels of importer are defined after the operator, do the same for
the exporter...
|
|
Please do not commit random things to code maintained by other people,
especially if you are breaking basic 101 things like codestyle. This is
loss of time for everybody. Even worse since not ansewring to comments
on original commit.
Also no need to add extra `text` parameters (and more useless UI
messages) in add-ons, when the only usage of the property's name itself
is in own add-on' UI...
|
|
|
|
|
|
|
|
|
|
Looks like a plain oversight in rBA53aec1ccff1b.
Maniphest Tasks: T72558
Differential Revision: https://developer.blender.org/D6448
|
|
Thing here was that files could actually be loaded ('obj_image_load' has
smart code for this), but tokenizing the corresponding line afterwards
could still fail [would have part of a filename still in image_data
list].
Now also correct this 'image_data' list in case 'obj_image_load' found
images with filename spaces.
Maniphest Tasks: T72148
Differential Revision: https://developer.blender.org/D6358
|
|
|
|
|
|
|
|
MTL standard does consider g and b values as optional...
|
|
|
|
Patch contributed by luzpaz.
Differential Revision: https://developer.blender.org/D5800
|
|
Changes in UI are significant enough to justify it.
|
|
See e4de25e78b59e9.
|
|
Updates importers/exporters for the new file-browser design. They are
now reorganized into sub-panels.
Updated the Blender version requirement (won't be compatible with older
Blender versions). Left the Add-on versions untouched, will leave that
up to Authors to change.
|
|
Reviewers: mont29
Maniphest Tasks: T68618
Differential Revision: https://developer.blender.org/D5479
|
|
|
|
Looks like loose edges need to be properly tagged as such nowadays...
|
|
Clamp input values to avoid invalid ones.
|
|
|
|
2.79b.
While this is not officialy part of OBJ format, some softwares 'solve'
the spaces-in-filenames issue by delimiting those with quotes.
Since this is not too much a hassle, let's add back support for this.
Based on work from Robert Guetzkow (@rjg), thanks.
Note: not sure whether this should be ported to 2.80 release, while this
change should be reasonably safe, it's not an obvious one-liner either,
and the issue is not really critical (don’t think any major software
uses that trick?)...
|
|
|
|
Caused by rB8252cc7044ea (fix for T65215), we actually need a default
material in edges-only case too, not to exclude None (default) one in
that case...
|
|
fails on re-import if object has adges but no faces.
|
|
Addresses new behavior of object.to_mesh().
This is corresponding part for D4875.
Reviewers: brecht
Reviewed By: brecht
Differential Revision: https://developer.blender.org/D4876
|
|
Mainly search-and-replace approach.
Tested the enabled-by-default export/import addons. Seems to work with
an exception of X3D which is still referencing Blender Internal material
properties.
Reviewers: brecht
Reviewed By: brecht
Differential Revision: https://developer.blender.org/D4866
|
|
Much better than using Principled's Transmission setting as we did
before...
Part of T64609.
|
|
|
|
We need a way for add-ons to generate a temp render depsgraph and
evaluate it, for this to work again, with new Blender 2.8 design.
|
|
Not really important here (we expect to set a value for all items), but
avoids useless computation anyway.
|
|
Thanks to @BzztPloink for the headup.
|
|
We cannot clear a face's vnors/uvs indices in case none are defined in
the OBJ file, we need indices for all loops when defining them in
Blender's mesh...
|
|
instances.
Looks like that was skipped somehow when OBJ IO was ported to 2.8...
|
|
As discussed in D4303.
|
|
Regression/side effect from rBA9448cef00d1b3, while we do want to get
one Blender object per 'o' line (object declaration) in OBJ file, we do
want to 'reuse' same objects when same OBJ groups ('g' lines) are used
inside of a same object, in case we split OBJ groups into objects...
Thanks to Jacques Lucke (@JacquesLucke) for initial investigation.
|
|
|
|
|
|
Bloody stupid mistake in 'speed-up parsing' work, yet annoyingly
difficult to spot on... :(
|
|
scale/offset was specified.
|
|
named the same.
Ensure we use a different key for each OBJ object (usung similar naming
increment as Blender one).
|
|
|