mathutils
import
@Bujus_Krachus The fix for lack of decimals in vector components is pretty simple. I've committed a patch to add the normalization step. The Machine.wrl
file is correctly imported now, with no…
I think it's useless, but I don't know the API enough to decide, so I'll take your conservative advice and leave it in.
The color attribute is fixed, my fault of a half imported patch,…
PS : As for reintroducing line 1960, I've tested that and didn't see any difference. Are you sure it's required since normals_split_custom_set
does all the job ? Also, shouldn't we use…
@Bujus_Krachus I finally had time to work on this. I've added the support of per vertex normals w/o normal index and changed the conditions for setting the polygon smooth.
Depending the data in…
One more question : are normals from this file correctly imported ? Seems weird. It comes from the Siemens NX CAD software, using the same Parasolid modeler than TopSolid, so it should be as clean…
I spent way too much time on that connexion problem, I finally made the change online, as you suggested @Bujus_Krachus.
The last commit remove the useless comments and only keep one validate()
…
Hi @Sergey, I finally find out how to make a proper link to a subproject PR : extensions/io_scene_x3d#1 Sorry for the inconvenience.
I couldn't install the extension on 4.2.0 for now, the installation fail (I already opened an issue). So yes, it…
Ok, then for the record, the normals.wrl
file is issued by TopSolid V6 with normals per vertex. colors.wrl
is a truncated file from another ticket I can't get my hand on anymore.
- A : File…
For the record, PR #1 will fix the issue and load this invalid file.
There was no opened ticket for that, should I've had open one first ?
@Bujus_Krachus That's clean ! 😃 Thanks for the update.
Who's reviewing and accepting the PR ? (yes, I'm new here)
Works fine with my VRML files, thanks 👍