neumond / blender-md3

.md3 models support for Blender
MIT License
62 stars 23 forks source link

QOL: Skip over hidden objects, and provide a more clear error when a packing error occurs (fixed) #11

Closed Xabis closed 6 years ago

Xabis commented 6 years ago

The current triangulation method will fail on any object that is hidden. This may or may not be resolved with PR #1, where the triangulation does not require a context. However, it seems better, to me, to allow the user to selectively decide which pieces will be exported via hide/show instead.

Added a try/except to catch struct errors. Any error here will more than likely be the 32k short limit, so this message is more user friendly than the giant call stack. In my opinion.

This is a fixed version of PR #10

neumond commented 6 years ago

Thanks for PR, I'm noticing you that I just set a license of project to MIT (it had no explicit license in repository, blender wiki currently points it as GPL, but it's wrong and going to be changed). If you agree I'll accept PR.

This actually touches wider question, whether to use user selection to export specific parts of scene. I'm going to create issue with voting to decide this.

Xabis commented 6 years ago

I leave these changes in the public domain; do with as you wish.