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

Exozodi fix #402

Open
wants to merge 27 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
6af49cf
Current status
CoreySpohn Oct 16, 2024
c1def85
switching to using standard_bands for V band calculations and adding …
dsavransky Oct 16, 2024
732167a
fixing typo in concepts doc
dsavransky Oct 16, 2024
a9516de
Create new system for working with exozodi intensity
CoreySpohn Oct 23, 2024
497ab98
Update documentation
CoreySpohn Oct 24, 2024
8fc9ac3
Add the script that generates the curve fit plot
CoreySpohn Oct 24, 2024
e65f638
Add .venv to .gitignore
CoreySpohn Oct 24, 2024
38c3ed2
Code cleanup and moving F0V attribute to be a ZL attribute
CoreySpohn Oct 24, 2024
103ef16
Initial test updates
CoreySpohn Oct 24, 2024
5bb3ddd
Merge branch 'master' into exozodi_fix
CoreySpohn Oct 24, 2024
4529ebd
Add nEZ for all SU modules
CoreySpohn Oct 25, 2024
987d05f
Update SurveySim modules to use JEZ instead of fEZ
CoreySpohn Oct 25, 2024
a6d3b2b
Update tests to have the JEZ units and proper accesses
CoreySpohn Oct 25, 2024
b842d6b
Add F0V attribute to ZL docstring
CoreySpohn Oct 25, 2024
a289256
Update docstrings in schedulers
CoreySpohn Oct 25, 2024
19e9166
Make ruff happy
CoreySpohn Oct 25, 2024
7202c3c
Make black happy
CoreySpohn Oct 25, 2024
7448252
Fix filter
CoreySpohn Oct 28, 2024
18ed7b8
Add pInds keyword to scale_JEZ
CoreySpohn Oct 28, 2024
7417d53
Use pInds keyword in SurveySimulation and fix some units
CoreySpohn Oct 28, 2024
5297145
Once again make Black happy instead of ruff
CoreySpohn Oct 28, 2024
66b267f
Fix usage of JEZ for completeness calls which should be using the def…
CoreySpohn Oct 28, 2024
64d2f3b
Back in black
CoreySpohn Oct 28, 2024
a389f86
fix: Incorrect caching behavior for JEZ0 caused by differing inclinat…
CoreySpohn Nov 19, 2024
57024a3
fix: Remove unnecessary keyword from get_spectral_template
CoreySpohn Nov 19, 2024
e49634e
fix: Add luminosity scaling to EZ calculations
CoreySpohn Nov 22, 2024
3a2a9ae
Update concepts document and the references to it
CoreySpohn Nov 22, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch 'master' into exozodi_fix
  • Loading branch information
CoreySpohn committed Oct 24, 2024
commit 5bb3dddbdfafc326570bb8fdca372b0562a8c787
4 changes: 2 additions & 2 deletions EXOSIMS/OpticalSystem/Nemati.py
Original file line number Diff line number Diff line change
Expand Up @@ -502,8 +502,8 @@ def ddMag_dt(

"""

# cast sInds, WA, fZ, JEZ, and intTimes to arrays
sInds = np.array(sInds, ndmin=1, copy=False)
# cast sInds, WA, fZ, fEZ, and intTimes to arrays
sInds = np.array(sInds, ndmin=1, copy=copy_if_needed)
WA = np.array(WA.value, ndmin=1) * WA.unit
fZ = np.array(fZ.value, ndmin=1) * fZ.unit
JEZ = np.array(JEZ.value, ndmin=1) * JEZ.unit
Expand Down
4 changes: 2 additions & 2 deletions EXOSIMS/PlanetPopulation/AlbedoByRadiusDulzPlavchan.py
Original file line number Diff line number Diff line change
Expand Up @@ -47,8 +47,8 @@ def __init__(
Rb=[1.4],
**specs,
):
self.ps = np.array(ps, ndmin=1)
self.Rb = np.array(Rb, ndmin=1)
self.ps = np.array(ps, ndmin=1, copy=copy_if_needed)
self.Rb = np.array(Rb, ndmin=1, copy=copy_if_needed)
specs["prange"] = [np.min(ps), np.max(ps)]
DulzPlavchan.__init__(
self, starMass=starMass, occDataPath=occDataPath, esigma=esigma, **specs
Expand Down
2 changes: 1 addition & 1 deletion EXOSIMS/Prototypes/OpticalSystem.py
Original file line number Diff line number Diff line change
Expand Up @@ -1953,7 +1953,7 @@ def Cp_Cb_Csp_helper(self, TL, sInds, fZ, JEZ, dMag, WA, mode):
)

# cast sInds to array
sInds = np.array(sInds, ndmin=1)
sInds = np.array(sInds, ndmin=1, copy=copy_if_needed)

# Star fluxes (ph/m^2/s)
flux_star = TL.starFlux(sInds, mode)
Expand Down
13 changes: 7 additions & 6 deletions EXOSIMS/Prototypes/TargetList.py
Original file line number Diff line number Diff line change
Expand Up @@ -1537,7 +1537,7 @@ def revise_lists(self, sInds):
"""

# cast sInds to array
sInds = np.array(sInds, ndmin=1)
sInds = np.array(sInds, ndmin=1, copy=copy_if_needed)

if len(sInds) == 0:
raise IndexError("Requested target revision would leave 0 stars.")
Expand Down Expand Up @@ -1736,7 +1736,7 @@ def starprop(self, sInds, currentTime, eclip=False):
currentTime = currentTime[0]

# cast sInds to array
sInds = np.array(sInds, ndmin=1)
sInds = np.array(sInds, ndmin=1, copy=copy_if_needed)

# get all array sizes
nStars = sInds.size
Expand Down Expand Up @@ -1926,7 +1926,8 @@ def starFlux(self, sInds, mode):
u.ph / u.s / u.m**2
)

# Identify which star indices need new flux calculations
# figure out which target indices (if any) need new calculations to be done
sInds = np.array(sInds, ndmin=1, copy=copy_if_needed)
novals = np.isnan(self.star_fluxes[mode["hex"]][sInds])
inds = np.unique(sInds[novals])

Expand Down Expand Up @@ -2061,7 +2062,7 @@ def calc_IWA_AU(self, sInds, **kwargs):
"""

# cast sInds to array
sInds = np.array(sInds, ndmin=1)
sInds = np.array(sInds, ndmin=1, copy=copy_if_needed)
return (
self.dist[sInds].to(u.parsec).value
* self.OpticalSystem.IWA.to(u.arcsec).value
Expand Down Expand Up @@ -2093,7 +2094,7 @@ def calc_HZ(self, sInds, S, A, B, C, arcsec=False):

"""
# cast sInds to array
sInds = np.array(sInds, ndmin=1)
sInds = np.array(sInds, ndmin=1, copy=copy_if_needed)

T_eff = self.Teff[sInds]

Expand Down Expand Up @@ -2126,7 +2127,7 @@ def calc_EEID(self, sInds, arcsec=False):

"""
# cast sInds to array
sInds = np.array(sInds, ndmin=1)
sInds = np.array(sInds, ndmin=1, copy=copy_if_needed)

d_EEID = (1 / ((1 * u.AU) ** 2 * (self.L[sInds]))) ** (-0.5)
# ((L_sun/(1*AU^2)/(0.25*L_sun)))^(-0.5)
Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.