← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Implementing X3D model support to 3d-viewer


Thanks for the feedback!

There is a problem with the export function as you said. For example, Blender vrml/x3d importer seems to have a problem with inlined x3d files inside vrml files. I think it would still be nice to use x3d models from other sources than Wings3d just inside the 3d-viewer.

I think that it souldn't be too hard to patch the VRML exporter so that it can take both, vrml and x3d files, as they are very similar in structure and I already have all the model information extracted from x3d file. Writing an x3d export function may also be quite simple if I base it to existing vrml exporter. I will take a look at it.

- Tuomas Vaherkoski

On 20.3.2013 1:23, Miguel Angel Ajo Pelayo wrote:
The patch doesn't look bad to me, well written, and seems to comply (visual check) with the coding standards.

Still untested, I could test and commit, I see nothing bad, If it becomes a maintenance problem in the future,
and we don't use it we could simple drop it, but if it's useful, I'm sure we will take care to maintain it.

What do others think?

Tuomas, thanks a lot for taking care of code standards and submitting.

Miguel Angel Ajo
skype: ajoajoajo

On 19/03/2013, at 23:20, Cirilo Bernardo <cirilo_bernardo@xxxxxxxxx> wrote:

From: Tuomas <tuomasvaherkoski@xxxxxxxxx>
To: kicad-developers@xxxxxxxxxxxxxxxxxxx
Sent: Wednesday, March 20, 2013 4:25 AM
Subject: [Kicad-developers] [PATCH] Implementing X3D model support to 3d-viewer

Hello everyone,

I've been writing some code to implement X3D support to 3d-viewer/pcbnew so that it would be easier to use other software than just Wings3D to export 3d-models. The basic functionality is implemented and it seems to work fine. There is still some things to be done, like supporting Transform rotation property.
I just wanted to post this patch to get some feedback.

The parser is implemented in its own class which fills the required fields of S3D_MASTER class.
I was thinking that I could also move the current wrl parser into its own class.

I also made changes to dialog_edit_module_for_Modedit.cpp so that it now supports loading of *.x3d files.

Please let me know what you think. The source is branched from r4017.
- Tuomas Vaherkoski​

On the practical side I really don't think we need yet another format for 3D views. Wings3D is so awful that I created my own tools for developing 3D VRML models, but although I can understand the frustration I think effort is better directed to supporting the existing 3D viewer functionality and in the future supporting tools to create solid models and STEP output.

On the other hand, if there are good vrml->X3D converters and we can make use of the existing VRML models, I see nothing wrong with replacing the 3D viewer and 3D exporter with new ones. Better still if the 3D viewer can take both X3D and VRML models - the only thing to keep in mind is that people currently use the VRML export as part of their process to create STEP files, so if the exporter is replaced there needs to be a tested method for creating STEP files.

- Cirilo

Mailing list: https://launchpad.net/~kicad-developers
Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp

Follow ups