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.

Sign up for GitHub

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

Jump to bottom

package lsst.para file for distribution #44

Open
drewoldag opened this issue May 15, 2024 · 2 comments · May be fixed by #52
Open

package lsst.para file for distribution #44

drewoldag opened this issue May 15, 2024 · 2 comments · May be fixed by #52
Assignees
@raphaelshirley

Comments

@drewoldag
Copy link
Collaborator

We want users to have immediate access to lsst.para after pip installing rail_lephare.
@drewoldag
Copy link
Collaborator Author

This file needs to be here because the default stage param reads this file. Otherwise the user wouldn't be interacting directly with this file.

@drewoldag drewoldag assigned drewoldag and unassigned drewoldag May 16, 2024
@raphaelshirley
Copy link
Collaborator

Since this issue was created we decided to implement the default COSMOS config in the main lephare package. Because of this we then simply overide the values from this default in the main Python file. Likewise users who want to implement another overiding config can simply do that. this issue should therefore be closed by deleting the lsst.para and removing all references to it.

@raphaelshirley raphaelshirley self-assigned this Jun 12, 2024
@raphaelshirley raphaelshirley linked a pull request Jun 12, 2024 that will close this issue
Open
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@raphaelshirley raphaelshirley

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Removed lsst para file LSSTDESC/rail_lephare
2 participants
@raphaelshirley @drewoldag