openPMD / openPMD-projects

:clipboard: Overview on Projects around openPMD
http://www.openPMD.org
Creative Commons Zero v1.0 Universal
8 stars 14 forks source link

Add MALA (Material Learning Algorithms) #33

Closed franzpoeschel closed 1 year ago

franzpoeschel commented 1 year ago

Pinging @acangi and @RandomDefaultUser for your input on this

RandomDefaultUser commented 1 year ago

Thanks for including MALA! This looks great to me with one exception: In the first line, (materials learning algorithms) is given at the end of the domain description. "materials learning algorithms" is not a domain-specific term, but simply the written out form of MALA. I would remove it from the first line and either not mention it or in the header line.

Edit: Added a suggestion to this effect, didn't realize I had access rights for that when writing this comment.

franzpoeschel commented 1 year ago

Thank you Lenz, I've added your suggestion. I think it's alright to have the full name somewhere, given that the name is somewhat informative about what the software does ;)