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

Meshplorer: Replacing meshes with different bone counts #383

Closed
Tajfun403 opened this issue Jul 5, 2023 · 3 comments
Closed

Meshplorer: Replacing meshes with different bone counts #383

Tajfun403 opened this issue Jul 5, 2023 · 3 comments
Assignees

Comments

@Tajfun403
Copy link

This is not fully a feature request, but rather an extended question.

The issue
When you try to replace a mesh in Meshplorer tool with one that has a different amount of bones, an error is thrown which says that it is not possible. This can, however, be easily circumvented by opening both the source and target mesh in Package Editor windows, and then simply performing a Replace Export Data operation.

The question
My question is whether the Meshplorer's error is a safety measure against users who might have tried to replace totally unassociated meshes which would lead to different issues down the road (incorrect animations, for example), or rather that the Meshplorer's replace code is widely different from what would the Replace Export Data perform, and is thus impossible technical-side.

Solution 1
If it's a safety measure, I believe a warning would be a better fit than an error. While a mesh with a different skeleton might indeed be a bad fit, there are sometimes legitimate cases of this usage, and the need of always performing this operation from Package Editor slightly limits the Meshplorer's feature set. For example, a modder might make sure themselves that the animation fits the new skeleton (this happened to me often when updating weapons), or the surplus bones might not even be used in the mesh at all - which quite often happens with BioWare assets where the entire skeleton is shipped even if unncessary.

Solution 2
If it's a technical limitation, I think it would be beneficial if the error message included a tip that replacement with a different bone count can be performed from Package Editor. It took me a long time to figure this out - at first, I just took the Meshplorer's error as definite, which closed my eyes on seeking alternatives which were in fact rather simple.

@Mgamerz
Copy link
Member

Mgamerz commented Jul 5, 2023

Can you post a file and destination export to repro this against?

@Tajfun403
Copy link
Author

Sure. Let's say we want to replace export 8 BIOG_ASA_HGR_LIASAM_R.LGTx.ASA_HGR_LGT_MDL from attached BioG_HMF_LiaraBreather.pcc (which was based on assets from BioH_Mystic_01.pcc) by the mesh stored inside attached BreatherNew.upk.

MeshplorerBones.zip

@SirCxyrtyx SirCxyrtyx self-assigned this Jul 8, 2023
@Tajfun403
Copy link
Author

Thanks!

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

3 participants