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

Multi-user use of Armature data #100

Open
Ckang3D opened this issue Mar 15, 2023 · 14 comments
Open

Multi-user use of Armature data #100

Ckang3D opened this issue Mar 15, 2023 · 14 comments
Assignees
Labels
bug Something isn't working

Comments

@Ckang3D
Copy link

Ckang3D commented Mar 15, 2023

Thank you very much for your excellent work in making the model available quickly with a wonderful UI panel control.

Here are some ideas I had while assembling the UI to the model:

Multi-user use of Armature data

After the initial loading of the UI to the model, the Armature data of the Rig will be occupied by the UI, which will prevent a addon called Auto IK Rigger from working properly because the Armature is used by multiple entities.

blender_Bjil5kvbUX
blender_VLybqNK6tP

My solution is to make a single-user copy and then go to the link lib and delete the original Armature data, then rebuilt in the UI (without testing if not rebuilding would have an effect on the model).

blender_2KAwXwyLMJ

I'm not sure if this problem can be solved, although it's not a problem to clean up the Armature data.


The second one is whether it is possible to add support for curved hairs?

Currently I have replaced the particle hairs by curved hairs, but the UI is currently unable to control curved hairs through the hairs panel.

My current solution is to add it to the Outfits panel as an Extra props.

image

If needed, my model can be found on Smutbase.

Thanks again for the excellent panel and look forward to your updates in the future.

@Mustard2
Copy link
Owner

Hi!

I'm aware of the problem of the two users on the armature, never actually thought it might create some issues. But since it does, I'll try to investigate it a bit more. Atm I'm not sure where in the code that problem arises to be honest.

For the others, sure! I was waiting for Blender to finalize the support for hair, but with 3.5 I guess it's time to properly add them to MustardUI. :)

Thanks again for the suggestions, and sorry for the late reply!

@Mustard2 Mustard2 self-assigned this Mar 31, 2023
@Mustard2
Copy link
Owner

Mustard2 commented Jun 1, 2023

  • Fix the Armature object duplicate error
  • Introduce support to curved hairs

@Mustard2 Mustard2 added this to the Version 0.27 milestone Jun 1, 2023
@Mustard2 Mustard2 added bug Something isn't working enhancement New feature or request labels Jun 1, 2023
Mustard2 added a commit that referenced this issue Jun 2, 2023
@Mustard2
Copy link
Owner

Mustard2 commented Jun 2, 2023

Hi @Ckang2021 , can you please check if the new version #106 is what you wanted?
I added curves hair as for particle hair, in the sense that they aren't on the list and thus they aren't mutually exclusive.

@Ckang3D
Copy link
Author

Ckang3D commented Jun 2, 2023

Hi @Mustard2, praise for your hard work! After testing, I have identified some issues:

  1. The Hair feature requires a meshes to be present in the Hair collection in order to activate. It cannot be used solely with curves hair.
  1. Enabling meshes hair affects the usage of curves hair. When switching to any other meshes hair, the curves hair gets disabled.

After conducting some additional tests, I found that particle hair is also affected by the first issue mentioned, while the second issue does not affect it.

Mustard2 added a commit that referenced this issue Jun 2, 2023
@Mustard2
Copy link
Owner

Mustard2 commented Jun 2, 2023

Hi @Ckang2021 , thanks for checking it. Now the two issues should be solved.

@Mustard2
Copy link
Owner

Mustard2 commented Jun 2, 2023

Now looking at your images, would you prefer to have the Char_name Hair removed from the name? Might be nicer I guess :)

@Ckang3D
Copy link
Author

Ckang3D commented Jun 5, 2023

Thank you for the update.

The switching of mesh hair no longer affects the display of curve hair, but the issue remains that mesh hair is still required to activate the hair functionality. Currently, my workaround is to include the Proxy Mesh in the hair collection but disable their visibility and rendering. However, this brings up a new issue where the toggle switch in the Hair List affects the entire collection, resulting in the curve hair not being displayed/rendered properly.

img1 img2

@Mustard2
Copy link
Owner

Mustard2 commented Jun 5, 2023

@Ckang2021 Thank you very much for beta testing this and for the valuable feedbacks!

I updated the version again, hoping it works correctly now. Please let me know, and thank you again! :)

@Ckang3D
Copy link
Author

Ckang3D commented Jun 6, 2023

Thanks for the recent update, the curve hair is now working properly, although it says "An error occurred", maybe I can add a model hair to avoid this problem?

Is there a possibility to have mesh/curve/particle hair on equal footing so that they can coexist without conflicts and be freely combined?
(There is no real impact, but the console will keep printing errors)

image


The second thing is that the display/render indicator for particle hairs seems to be the wrong icon, I looked at other designs and it only has a blue background when it is disabled.
(For comparison, I turned off Pubic hair as a reference)

image


Thank you once again for the time and effort you have dedicated to this project. I hope it continues to improve. :D

@Mustard2
Copy link
Owner

Mustard2 commented Jun 7, 2023

Hi!

For the first issue, as it says, try to enter configuration mode and then exit it. That should solve the issue in principle.

As for the second, it's how Blender works unfortunately. In case of hair particles, the property is called show_particle, in the other case hide_object, so the blue "embossing" effect is exactly the opposite. :(

Let me know if you can fix the first issue clicking on the button Enter Configuration Mode!

@Ckang3D
Copy link
Author

Ckang3D commented Jun 8, 2023

Hello, thank you for your explanation and response. I have tested it, but entering and exiting the configuration did not solve the issue.

At the same time, the error message C:\Users\blender\git\blender-v350\blender.git\source\blender\python\intern\bpy_rna.c:1341 pyrna_enum_to_py: current value '0' matches no enum in 'MustardUI_RigSettings', '', 'hair_list' is still being output when hovering over any component of the UI. I have tested renaming the hair collection and curve hair, but the issue persists. If you need more information or further testing, please let me know.

This is my test project, with all packed textures removed: https://files.catbox.moe/f5cyd7.blend (14.5MB)

@Mustard2
Copy link
Owner

Mustard2 commented Jun 8, 2023

Ok, you're right, there was another bug :(
Now it should be fixed, I hope :D

@Mustard2 Mustard2 changed the title Some ideas and suggestions when assembling UI to models Multi-user use of Armature data Jun 10, 2023
@Mustard2 Mustard2 removed this from the Version 0.27 milestone Jun 10, 2023
@Mustard2 Mustard2 removed the enhancement New feature or request label Jun 12, 2023
@Ckang3D
Copy link
Author

Ckang3D commented Sep 15, 2023

@Mustard2
New problems caused by multi-user use of Armature data:

Hello, in my recent modeling process, I accidentally assembled the UI before converting Daz armature to Rigify (Diffeomorphic) .

Similarly, this will cause problems with automatic conversion. I need to separate the Armature data first, but this leads to a new problem: all models' Morph targets will be incorrect.

The Morph targets are based on the original Armature data. I have tried mapping the data (which makes the newly separated Armature become multi-user use again), which means I cannot restore the data to a state without UI after assembling it again, even if I have uninstalled the UI.

Currently, my solution is to revert back to the project before I assembled the UI and restore the added content. Then, after assembling Rigify, I add the UI again.

I'm not sure if I made a mistake at some point, but I have tested it myself and found that Morph can be used normally when it converting to Rigify without UI, If you need me to provide the project or more information, please let me know.

Thank you for your work. 🫡

@Mustard2
Copy link
Owner

Hi @Ckang2021 ! Sorry I missed this comment (I don't receive the notifications :( ).
Sure, if you have any blend file I can try to take a look at it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants