EpicGamesExt / BlenderTools

Blender addons that improve the game development workflow between Blender and Unreal.
https://epicgamesext.github.io/BlenderTools/
MIT License
2.81k stars 61 forks source link

Send to Unreal - MaterialSearchLocation not working #442

Closed dubhlaidh closed 2 years ago

dubhlaidh commented 2 years ago

Addon: Send to Unreal

Steps to Reproduce the Problem:

Modified ue5 project DefaultEditorPerProjectUserSettings.ini to AllAssets instead of Local. This causes it to do AllAssets but then as expected Send2UE exporting to Local now doesn't work

[/Script/UnrealEd.FbxTextureImportData]
MaterialSearchLocation=EMaterialSearchLocation::AllAssets

I modified addon to dump property_data on ingest_data and it shows it had the setting... Maybe it's changed on the ue5 side? I looked for it in ue5 but only found the c++ code not the python stuff. snippet from dumped property_data:

'material_search_location': {'value': 'unreal.MaterialSearchLocation.ALL_ASSETS', 'unreal_type': 'Enum'}

The pythonapi docs https://docs.unrealengine.com/5.0/en-US/PythonAPI/class/MaterialSearchLocation.html still look right though.

OS: win10 Addon Version: 2.0.2 Blender Version: 2.93 & 3.01 Unreal Version: 5.0.1 broken. 4.27 working

james-baber commented 2 years ago

That is at least my guess as well. Do you mind closing and reposting this here https://github.com/EpicGames/BlenderTools/issues/405

james-baber commented 2 years ago

At least I believe these are related issues, but feel free to leave open if they are not

dubhlaidh commented 2 years ago

checking

dubhlaidh commented 2 years ago

Is related, but I must be missing something. That one says the option doesn't exist in addon? And comments indicate it doesn't work because importfbx doesn't have it? But the addon has the option and 4.27 works...

dubhlaidh commented 2 years ago

Nevermind! I was missing the obvious. #405 is "Material Import Method". This one is "Search Location"

james-baber commented 2 years ago

@dubhlaidh do you think changing the displayed option name from Search Location to Material Import Method would suffice, then we can close this out?

dubhlaidh commented 2 years ago

Nope, the other ticket was a different field. I just did a quick test with 5.0.2 that I downloaded yesterday though and it might be fixed. I'll retest it properly in a little bit and close this if UE5.0.2 fixed it.

dubhlaidh commented 2 years ago

UE 5.0.1 always uses "unreal.MaterialSearchLocation.LOCAL" UE 5.0.2 always uses "unreal.MaterialSearchLocation.ALL_ASSETS" and changing send2ue settings won't make it do anything different.

Pretty sure now that this is a UE 5 screw up though, so will just close this since send2ue is giving the proper settings to UE.