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

Current script state #2

Closed
le717 opened this issue May 3, 2015 · 4 comments
Closed

Current script state #2

le717 opened this issue May 3, 2015 · 4 comments

Comments

@le717
Copy link
Contributor

le717 commented May 3, 2015

Hey!

I am wanting to resume some work on the importer script during the next few weeks, and you know just as well as I do that eventually we will need to completely replace that script because of all the issues it has. I have two ideas for accomplishing that, and one of them involves this script here.

Basically, we would use this script for parsing it and my script would simply convert the data generated by this into mesh. Your script would remain over here, and mine over there. This would make maintenance and releases much easier, as if there were changes made over here, I would simply need to update mine with the latest version and release.

I am wondering what the current state of this script is, if it is in a good state that I could begin integrating it into my script, and if there is anything I can do to help improve this script further.

I really would like to pursue this idea, as I like it more than the alternative, rewriting everything on my side.

Thanks!

@JoshTheDerf
Copy link
Owner

@le717 I've tested this script, and actually built an almost-functioning WebGL parser/renderer, but I ran into some issues related to backface culling, which would require a sort of rewrite, since previous lines can affect later ones. MPD support would be pretty simple, but at the moment, I'm not really in a hurry to do anything with it. The amount of special cases required to render every model properly is insane.

If you plan to start soon, I can start working on this again. :3

@le717
Copy link
Contributor Author

le717 commented May 4, 2015

I ran into some issues related to backface culling

I'm no 3D artist (can't do a lick of anything in 3D), but that can be a pain to work with and fix. Hopefully it won't be too much work!

MPD support would be pretty simple

That would be a 👍. MPD models are very common.

If you plan to start soon, I can start working on this again. :3

Yes, I have a few weeks break and I've wanting to code something, so I figured I'd resume work on the importer but start up here and help you out some.

@JoshTheDerf
Copy link
Owner

@le717 I pretty much got around BFC in my renderer by disabling it completely but... well... See screenshot. I don't even know how to begin covering all the special cases. Only one part, the 1x1 brick, rendered properly. Not too sure how much needs to be dealt with in the converter or the mesh builder.
image

@le717
Copy link
Contributor Author

le717 commented May 6, 2015

Yea, it'll probably be best to just not support BFC unless there's someone who knows how to implement it.

@le717 le717 closed this as completed May 6, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants