rtx-remix icon indicating copy to clipboard operation
rtx-remix copied to clipboard

[Toolkit feature]: Define Inwards Displacement at Ingestion Time

Open skurtyyskirts opened this issue 1 year ago • 6 comments

Contact Details (Optional)

No response

What is the context of the feature/improvement?

Can we adjust the default Inwards Displacement from 0.05 to 0.01? In my experience 0.05 has always been too much displacement for textures, and I feel 0.01 is more of a general displacement default.

Please describe the feature/improvement?

To adjust the 0.05 displacement to 0.01 as the default.

What would be your solution? (optional)

...

Version

2024.5.1

Media

Screenshot 2024-11-17 222213

skurtyyskirts avatar Nov 18 '24 05:11 skurtyyskirts

Thanks @skurtyyskirts. These defaults are managed in an MDL file outside of the toolkit. Since these values will be different for every asset depending on how wide and high the heightmap should be interpreted, it would be good to get some consensus from the community. Feel free to invite folks to thumbs up this change to show support for a smaller value.

nxkb avatar Dec 12 '24 23:12 nxkb

Sounds good, thank you! I'll ask the community and see what they think :)

skurtyyskirts avatar Dec 13 '24 00:12 skurtyyskirts

Had a brief conversation about this with Skurtyskirts on discord here: https://discordapp.com/channels/1028444667789967381/1028444668918235197/1316976941118783528

For manually authored materials, the value should really be set per material, and the current default is pretty good.

Skurty's use case is more focused on the batch AI material ingestion. In their workflow, every material gets a heightmap but the AI gives no indication of how deep that heightmap is intended to go. In this case, a value of 0.01 would be small enough that most of the materials wouldn't need to be adjusted.

Unfortunately, POM is not free, and I'd rather avoid having workflows that encourage users to have barely noticeable POM on every surface in the game.

The best case here would be for the texture generation AI to also generate a displace_in value, and somehow pass that through the ingestion pipeline.

Another possible option would be to allow users to set up their own default values before ingesting a batch of materials.

MarkEHenderson avatar Dec 16 '24 17:12 MarkEHenderson

Hey @skurtyyskirts - checking in to see if you have any other questions about this, or if you're alright with us closing it? Thanks!

NV-LL avatar Jan 09 '25 21:01 NV-LL

Sorry for the delayed response. Mark's correct, this would be moreso beneficial to batch AI material ingestion. I am a fan of the option to allow users to set the default value for these values before ingesting. Perhaps when the toolkit identifies we're ingesting more than 1 material if we'd like to set a default to all other ingested materials of this type, like an "apply to all" or a "dismiss never show again' type of thing.

skurtyyskirts avatar Jan 09 '25 23:01 skurtyyskirts

Thanks for clarifying! We've filed REMIX-3885 for internal investigation.

NV-LL avatar Feb 13 '25 22:02 NV-LL