Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

create FXyz-Importers #42

Merged
merged 9 commits into from Oct 26, 2018
Merged

create FXyz-Importers #42

merged 9 commits into from Oct 26, 2018

Conversation

jperedadnr
Copy link
Contributor

Includes the obj importer from the 3DViewer project, adds PolygonMesh and related classes to FXyz-Core, and includes a sample to FXyz-Samples with a 3D model from the Duke repository.

@nlisker
Copy link
Collaborator

nlisker commented Oct 15, 2018

Except for package declarations and build/config files, did you make any changes to the 3DViewer files?

@jperedadnr
Copy link
Contributor Author

No, basically I've just moved classes, I just wanted to have a first PR in place. We can discuss required changes and improvements once we have the 3DViewer importers transferred.

Copy link
Collaborator

@nlisker nlisker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's all good, just wanted to make sure about the starting point.

@nlisker
Copy link
Collaborator

nlisker commented Oct 24, 2018

Are you waiting for another review or can it be merged?

@jperedadnr jperedadnr merged commit bec98e3 into FXyz:master Oct 26, 2018
@jperedadnr
Copy link
Contributor Author

@nlisker I've just merged it. I will start adding the Maya importer now. The DAE importer doesn't really work for the samples I've tested so it will required more time to review it before including it.

@nlisker nlisker removed the WIP Work in Progress label Nov 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants