Skip to content

Option to see "live" version of all climate profiles #213

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

Closed
natashabatalha opened this issue Aug 12, 2024 · 5 comments · Fixed by #215
Closed

Option to see "live" version of all climate profiles #213

natashabatalha opened this issue Aug 12, 2024 · 5 comments · Fixed by #215

Comments

@natashabatalha
Copy link
Owner

Option to read out all TP profiles during a climate run so that users can debug climate runs that oscillating

@aditya-sengupta
Copy link
Contributor

I'll probably need this feature, so I can implement this - are there any known cases with oscillations that'd make a good test/demonstration?

@natashabatalha
Copy link
Owner Author

@sagnickm or @James-Mang both have examples I think!

My recommendation is to make key word save_all_profiles, which currently is True/False accept a string argument that would be a HDF5 filename. Then the functionality would be:

True: stores all output but not live to a H5 file
False: stores nothing
String: stores all output and updates live to a H5 file

@natashabatalha
Copy link
Owner Author

natashabatalha commented Aug 12, 2024

@aditya-sengupta also you can develop the functionality for any case (even just the standard BD or Exoplanet case) it will just be the most useful for those tricky cases that are not converging well.

@James-Mang
Copy link
Collaborator

I think any cold cloudy case with H2O clouds will have large oscillations, especially in the 225-275 K range of brown dwarfs. As for clear cases, some high metallicity (ie. +2.0), high C/O ratio (2.5) cases for some T/Y dwarfs also change quite a bit while trying to converge. I can't think of a specific case right now but I can take a look for some.

@natashabatalha
Copy link
Owner Author

Merged to dev. Right now this only works outside of profile/profile_deq but if we think we need more granular updates then we can pass @aditya-sengupta 's code into those functions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants