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

Code Comprehensive Review #427

Open
6 of 9 tasks
HajimeKawahara opened this issue Oct 23, 2023 · 3 comments
Open
6 of 9 tasks

Code Comprehensive Review #427

HajimeKawahara opened this issue Oct 23, 2023 · 3 comments
Assignees
Milestone

Comments

@HajimeKawahara
Copy link
Owner

HajimeKawahara commented Oct 23, 2023

Based on the reflections from Bug #414 , we will thoroughly inspect the calculations and implementations of the code up to now.

emission spectrum comparison with PetitRadtrans, which was performed when version 1 released, but now many changes since then

physical settings

opfac * xsmatrix * dParr[:, None] * mixing_ratio[:, None] / (mass * gravity)

opfac is equivalent to 1.e3/m_u, where m_u = scipy.constants.m_u (atomic mass in kg). 1.e3 converts to cgs unit.

mixing_ratio: volume mixing ratio (VMR) or mass mixing ratio (MMR) [N_layer]
mass: mean molecular weight for VMR or molecular mass for MMR

Note: when using MMR as molecular abundance parameters, the spectrum does not depend on the mean molecular weight, and therefore scalable by g, i.e. MMR/g.

constants

  • constants.py

continuum

radiative transfer

  • ArtEmisPure
  • ArtTransPure
  • ArtReflectPure -- compares with pure absorption (ArtAbsPure)
@ykawashima
Copy link
Collaborator

ykawashima commented Jan 11, 2024

I performed the comparison of the spectra calculated with the pure absorption case with petitRADTRANS. The scripts for these plots are included in tests/integration/comparison/twostream. Exojax calculations were performed with release v1.4.2.

wide_R7000
narrow_R70000
CIA_R70000

@HajimeKawahara
Copy link
Owner Author

Thanks! #457

@ykawashima
Copy link
Collaborator

ykawashima commented Jan 13, 2024

I realized that it makes more sense to compare intensity-based calculation with petitRADTRANS, the results of which are attached below. The spectra are now much more similar. The scripts for these plots are included in tests/integration/comparison/twostream. Exojax calculations were performed with the develop branch on Jan. 13th (a5409ae).

wide_R7000
narrow_R70000
CIA_R70000

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