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

Improve support for Blender 2.93 LTS #32

Open
wants to merge 12 commits into
base: blender_29
Choose a base branch
from

Conversation

BTOdell
Copy link

@BTOdell BTOdell commented Jan 25, 2022

Closes #31
Closes #33
Closes #35


alpha_default = 1.0

# mesh.calc_tessface() # Is this needed?
# mesh.calc_loop_triangles() # Is this needed?

Choose a reason for hiding this comment

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

The call likely ensures that the data will be up-to-date. I think it should be uncommented.

Copy link
Author

Choose a reason for hiding this comment

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

calc_loop_triangles() is called earlier in the mesh_triangulate function (line 39). And it doesn't seem like the mesh is modified in any way between mesh_triangulate and this commented out line.

Copy link

Choose a reason for hiding this comment

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

In that case the comment could be removed.

Copy link
Author

Choose a reason for hiding this comment

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

Yup, already done:
https://github.com/CoDEmanX/blender-cod/pull/32/files#diff-6580964a45e40f7fbc9076d04d07f7ebc3bc57206acb8c37703fafdb131e4edeL261

This PR is pretty much ready to merge. Some testing of the vertex colors feature would be nice but I don't have a model to test with. I actually just started using Blender, so I'm a noob when it comes to actually making models. 😅

@BTOdell BTOdell marked this pull request as ready for review January 27, 2022 04:42
@BTOdell
Copy link
Author

BTOdell commented Jan 27, 2022

I have tested this PR with a test model that I made and it successfully exports to .XMODEL_EXPORT format without error and it successfully loads into the CoD WaW Mod Tools (Asset Manager/Viewer).

@BTOdell
Copy link
Author

BTOdell commented Jan 27, 2022

My test model only had a color map (image texture), no spec maps or bump maps or anything like that.
And I haven't tested the vertex colors feature.

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

Successfully merging this pull request may close these issues.

None yet

2 participants