Closed Michel-Heinz closed 3 months ago
The modification addresses a compatibility issue within Molara's molecule loading functionality, specifically enhancing its ability to process .xyz files. The update ensures that atomic symbols, regardless of their case in the input file, are correctly recognized by converting lowercase atomic symbols to uppercase before their conversion to atomic numbers. This change broadens the tool's usability with .xyz files generated from various sources that may use lowercase atomic symbols.
File Path | Change Summary |
---|---|
.../Structure/io/importer.py |
Capitalize atomic symbols in .xyz files before conversion to atomic numbers. |
🎉 A whimsical ode by CodeRabbit 🎉
In the realm of codes and bits, a tiny fix was sown, To let the molecules parade in cases not well-known. From 'c' to 'C', and 'h' to 'H', with elegance they climb, Across the bridge from text to form, in less than no time. So cheers to those who type in small, your letters now stand tall, Within Molara's halls they rise, uniting one and all. 🐇
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
This fixes #343
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 68.18%. Comparing base (
0fc7b51
) to head (584480a
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Summary by CodeRabbit