Skip to content

Conversation

eford
Copy link
Member

@eford eford commented Sep 11, 2025

No description provided.

Comment on lines +1433 to +1434
flux_star_interp[om.star.log_λ .< minimum(om.metadata[:fixed_vectors]["log_λ"])] .= 1
flux_star_interp[om.star.log_λ .> maximum(om.metadata[:fixed_vectors]["log_λ"])] .= 1
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kment Could you add a comment about the logic here?
I'm wondering if setting these pixels to 1 might affect the predictions between the minimum and maximum.

flux_star_interp[oms[1,1].star.log_λ .< minimum(fixed_vectors["log_λ"])] .= 1
flux_star_interp[oms[1,1].star.log_λ .> maximum(fixed_vectors["log_λ"])] .= 1
oms[1,1].star.lm.μ[:] = flux_star_interp
flux_star .*= flux_star_interp
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kment Was flux_star initialized to 1? If not, please document why you're multiplying here.

using FITSIO
using DataInterpolations

teldir = "/home/kment/RV/telluric" # Directory containing telluric model spectra
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kment Please replace this path with something more general. We could create a data directory and then we can access it with joinpath(pkgdir(StellarSpectraObservationFitting, "data"), filename)

Copy link
Member Author

@eford eford Sep 11, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If the files are small, we can just stick them in this repository. If they're large then we can add a script in 'deps/build.jl' that downloads the files into the data subdirectory. It could be from a separate GitHub repository (maybe TelluricModels or SSOFTelluricModels?) or a Zenodo url.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Another option is to take the directory from a LocalPreferences.toml file in the same directory as the Project.toml.

Copy link
Member Author

@eford eford left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kment See comments for a few questions and one request about path to input data files.

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

Successfully merging this pull request may close these issues.

2 participants