[Feature Request/Extension] Importing a model should not unhide everything #277

New issue

Closed

opened 2025-01-18 04:41:15 +00:00 by Ghost · 3 comments

Ghost commented

2025-01-18 04:41:15 +00:00

(Migrated from git.disroot.org)

Blender Version:
4.2.2 LTS

Cats Version:
4.2.2.4

Operating System:
Windows 11

Steps to Reproduce:

Hide child objects in existing scene
Import a new model with cats
Observe that everything hidden is now unhidden

Description of Issue: I was quite pleased with the recent update to cats to prevent export model from unhiding everything and I deliberated making an issue to request import model to not do the same for a few days. My reason being that I like to fit outfits to my models and I like to keep my models in their default look in their .blend as well as hiding the armature to de-clutter. I can understand if unhiding objects that were imported are necessary, but I don't like that it also unhides existing objects. Blender Version: 4.2.2 LTS Cats Version: 4.2.2.4 Operating System: Windows 11 Steps to Reproduce: 1. Hide child objects in existing scene 2. Import a new model with cats 3. Observe that everything hidden is now unhidden

Ghost commented

2025-01-21 09:57:06 +00:00

(Migrated from git.disroot.org)

Hi there,

So when it comes to importing we have to be careful on what we do, quite a few functions in Cats (And this is more a blender requirement and really not Cats problem) need all items to be visible or it will fail.

So if we just randomly remove the code to make this happen it may cause confusion on other users as this is an expected thing.

However, what we could do is have a option in Cats settings to enable/ disable this instead so when it is enabled it will keep objects hidden which you hidden but when disabled it will unhide everything just like the current way of doing things. By default is will be disabled but this then gives you the ability to choose what you want, which should also be saved between sessions.

I hope this solution suits your needs, it most likely won't be unto next week unto I get round to it though, but I do like this idea.

Hi there, So when it comes to importing we have to be careful on what we do, quite a few functions in Cats (And this is more a blender requirement and really not Cats problem) need all items to be visible or it will fail. So if we just randomly remove the code to make this happen it may cause confusion on other users as this is an expected thing. However, what we could do is have a option in Cats settings to enable/ disable this instead so when it is enabled it will keep objects hidden which you hidden but when disabled it will unhide everything just like the current way of doing things. By default is will be disabled but this then gives you the ability to choose what you want, which should also be saved between sessions. I hope this solution suits your needs, it most likely won't be unto next week unto I get round to it though, but I do like this idea.

Ghost commented

2025-01-29 03:57:38 +00:00

(Migrated from git.disroot.org)

So this issue bolds down to this few lines of code

So basically almost every function in cats calls this function, even if you just merging two meshes everything get's unhidden.

Though this will be an easy fix for the Importer I did fumble the idea of maybe making this a system wide change however, that would be alot of work. For now I will change the way the importer does thing and make sure it doesn't unhide things, no setting needed tbh seeing as Cats calls that select function for everything more or less.....

So this issue bolds down to this few lines of code ![Image](https://github.com/user-attachments/assets/27e28732-1a36-4311-949e-877f710dfbeb) So basically almost every function in cats calls this function, even if you just merging two meshes everything get's unhidden. Though this will be an easy fix for the Importer I did fumble the idea of maybe making this a system wide change however, that would be alot of work. For now I will change the way the importer does thing and make sure it doesn't unhide things, no setting needed tbh seeing as Cats calls that select function for everything more or less.....

Ghost commented

2025-01-29 04:10:18 +00:00

(Migrated from git.disroot.org)

This has now been sorted and is on the dev branch for Blender 4.3 and 4.2 and will be released in the next stable due to be out in the next week or two.