-
Notifications
You must be signed in to change notification settings - Fork 89
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WIP] Added waveder support #187
base: main
Are you sure you want to change the base?
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #187 +/- ##
==========================================
- Coverage 76.90% 76.57% -0.33%
==========================================
Files 62 62
Lines 5317 5328 +11
Branches 755 757 +2
==========================================
- Hits 4089 4080 -9
- Misses 1020 1040 +20
Partials 208 208
|
Hi Jimmy, sorry I didn't get back to you about this PR. I think spoofing something very small in the optics directory would be great. |
""" | ||
try: | ||
return Waveder.from_binary(dir_name / "WAVEDER") | ||
except Exception: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we not catch all exceptions?
Hi @jmmshn, what's the status of this PR? |
Added support to store Wavder objects in pymatgen
The
Wavder
object is helpful for a host of optical analysis functions so I'm trying to figure out how to store it properly.I'm of the opinion that by default people will not want to use it for general optical workflows but will like to have it for special cases.
@utf Do you have any opinion on how test files should be added?
These files are quite large so I was thinking about spoofing something in the current optics test folders.