Skip to content
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

Unit Tests #181

Open
13 tasks
natashabatalha opened this issue Jan 17, 2024 · 1 comment
Open
13 tasks

Unit Tests #181

natashabatalha opened this issue Jan 17, 2024 · 1 comment

Comments

@natashabatalha
Copy link
Owner

natashabatalha commented Jan 17, 2024

Unit tests for all picaso functionality

  • Adding Clouds </notebooks/2_AddingClouds.ipynb> @melrows
  • Transmission Spectroscopy Exoplanets </notebooks/5_AddingTransitSpectrum.ipynb> @sirpetergao
  • Thermal Emission Exoplanets </notebooks/5_AddingThermalFlux.ipynb> - @Ben-Lew
  • Brown Dwarfs </notebooks/6_BrownDwarfs.ipynb> - @Ben-Lew
  • Moving Away From Box Models </notebooks/7_PairingPICASOToVIRGA.ipynb> @semoran
  • Grid Search Analysis </notebooks/fitdata/GridSearch.ipynb> - @agagnebin
  • Brown Dwarfs </notebooks/climate/12a_BrownDwarf.ipynb> @sagnickm
  • Planet </notebooks/climate/12b_Exoplanet.ipynb> @sagnickm
  • Brown Dwarfs w/ Disequilibrium Chemistry </notebooks/climate/12c_BrownDwarf_DEQ.ipynb> @sagnickm
  • Contribution Functions </notebooks/11_ContributionFunctions.ipynb> - @ Theodora
  • t_start & get_fluxes & other individual functions @aditya-sengupta
  • Getting started (Reflected light) @James-Mang
  • Model storage and reuse @James-Mang
@natashabatalha natashabatalha changed the title Climate Unit Tests Unit Tests Jan 28, 2025
@James-Mang
Copy link
Collaborator

James-Mang commented Feb 5, 2025

Added unit test for reflected light and model storage notebooks/functions in unit test branch with commit @676d931

One issue to note: there's a difference in whether the gravity, mp or rp is saved based on the logic present in output_xarray(). There is a difference between newdev and master. So if saving a new basecase and/or running the notebook in newdev, it will throw an error when trying to reuse spectrum to compute reflected+thermal. I will work on getting this fixed in newdev

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

No branches or pull requests

2 participants