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

Updating instructions for newer exotic-ld versions #616

Merged
merged 2 commits into from
Feb 20, 2024
Merged

Conversation

taylorbell57
Copy link
Collaborator

This PR resolves #540

I've set a lower limit on the exotic-ld version to 3.1.2 which is their latest version (contains the updated post commissioning sensitivity files for JWST), and I've updated the documentation to make sure folks know how to download those reference files and know that the reference files change with different ExoTiC-LD reference files.

This resolves #540

I've set a lower limit on the exotic-ld version to 3.1.2 which is their latest version (contains the updated post commissioning sensitivity files for JWST), and I've updated the documentation to make sure folks know how to download those reference files and know that the reference files change with different ExoTiC-LD reference files.
@taylorbell57 taylorbell57 added documentation Improvements or additions to documentation enhancement New feature or request labels Feb 8, 2024
@taylorbell57 taylorbell57 requested a review from kevin218 February 8, 2024 00:01
kevin218
kevin218 previously approved these changes Feb 9, 2024
@taylorbell57
Copy link
Collaborator Author

Just realized this is failing because the ExoTiC-LD code (on pip and GitHub) only has version 3.0.0 while the reference files are on 3.1.2. I've sent David and Hannah an email to make sure we're supposed to use those two together; Hannah is out of the office for a while, so hopefully David responds fairly soon. Either way, I'll have to tweak some of the files and wording to make that clearer in these instructions.

@taylorbell57 taylorbell57 marked this pull request as draft February 9, 2024 21:32
@taylorbell57 taylorbell57 marked this pull request as ready for review February 12, 2024 23:09
@taylorbell57 taylorbell57 merged commit a97ac7b into main Feb 20, 2024
2 of 3 checks passed
@taylorbell57 taylorbell57 deleted the exotic-ld_v3 branch February 20, 2024 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
Status: Done
Status: Done
Development

Successfully merging this pull request may close these issues.

[Bug]: Current code/documentation incompatible with exotic-ld v3
2 participants