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

LANL1D model addition #379

Open
wants to merge 17 commits into
base: main
Choose a base branch
from

Conversation

mreuter01
Copy link

This fork is the repository I built my new model on. It runs successfully with the rest of NMMA and does not interfere with the function of other people's models and/or other functions within the code.

Adding first rendition of LANL1D function
Un-commenting the Ye_wind parameter
Commenting out KNtheta parameter in LANL1D
Adding parameters for LANL1D
Creation of LANL1d prior file
edited vej_dyn to the proper numbers
update get_knprops_from_LANLfilename to account for masses in filenames that aren't followed by another letter
Adjusting the timeshift
changing from log_10_mej_dyn to mej_dyn
Changed from log_10_mej_dyn to mej_dyn in LANL1D
Changing timeshift
updating log10_mej_dyn
updating log10_mej_dyn
Setting timeshift=0 and luminosity_distance=40.7 to properly reflect the files
@sahiljhawar
Copy link
Member

@mreuter01 I am assuming this model uses SVDLightCurveModel, if so, can you point to the model files as well?

@sahiljhawar
Copy link
Member

@mreuter01 also, any reference paper for this model?

Reverting file back to its previous state
@bking-astro
Copy link
Collaborator

@mreuter01 I am assuming this model uses SVDLightCurveModel, if so, can you point to the model files as well?

What do you mean by this Sahil?

@mreuter01 remove the unnecessary commented lines please

Removed unnecessary commenting from LANL1D
@mreuter01
Copy link
Author

@mreuter01 also, any reference paper for this model?

@sahiljhawar fixed.

@mreuter01 remove the unnecessary commented lines please

@bking-astro fixed.

@sahiljhawar
Copy link
Member

sahiljhawar commented Aug 21, 2024

@bking-astro SVD files (h5 or pkl files). If such files exists, those needs to be added to the gitlab repo

@mreuter01
Copy link
Author

@bking-astro SVD files (h5 or pkl files). If such files exists, those needs to be added to the gitlab repo

They should be added now. Let me know if there's something else I need to do.

@sahiljhawar
Copy link
Member

LGTM

@sahiljhawar
Copy link
Member

@bking-astro feel free to merge if you are okay with this

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.

3 participants