search

ansys / pyedb

pyedb is a Python library to use the EDB client library.
https://edb.docs.pyansys.com/
MIT License
3 stars 0 forks source link

edb.materials.load_amat(material_file) loading wrong material #156

Closed svandenb-dev closed 1 week ago

svandenb-dev commented 2 months ago

🔍 Before submitting the issue

🐞 Description of the bug

Loading Amat files provides wrong material values in EDB

✅ Expected behavior

get correct values

📝 Steps to reproduce

NA

📃 Relevant log output

NA

SMoraisAnsys commented 1 month ago

@svandenb-dev Is it related to the discussion we had last friday ? If not, can you show me a reproducer so that I could help you on that ?

svandenb-dev commented 1 month ago

@SMoraisAnsys this was reported by users. Loaded material values are wrong. To reproduce we just need to load the file and check material properties

SMoraisAnsys commented 1 month ago

I'm not able to reproduce the problem but there is something that we should rework with the way materials are currently handled. Sometimes their names are handled as they are provided, e.g. "FR4_epoxy", and other times the names are "lowered", e.g. "fr4_epoxy". This is an issue when one is trying to perform multiple actions mixing those concepts, e.g. materials duplicate, _load_materials.

@svandenb-dev How would you like to work on this issue ?

SMoraisAnsys commented 1 week ago

Recent refactoring and features should have handled this issue, see #290 and #361. Closing issue but feel free to open it back with extra information if recent updates were not sufficient enough.