-
Notifications
You must be signed in to change notification settings - Fork 12
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
Update requirements.txt #57
Conversation
Why does coverage decrease by 9% when switching the required numpy version? This seems strange @swagnercarena |
paltas/MainDeflector/perturbers.py
Outdated
- z_lens - perturber redshift | ||
""" | ||
# Define the parameters we expect to find for the DG_19 model | ||
required_parameters = ('gamma','theta_E','e1','e2','center_x', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to update required parameters. You'll need to feed in new parameters for the SIS model.
paltas/MainDeflector/perturbers.py
Outdated
lens_class(model).param_names | ||
) | ||
model_kwargs = {} | ||
for param in p_names: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Be careful here. Because you have both a PEMD model and an SIS model, some parameter names are shared between the two models. You might have to re-write this to account for that. (ex: both main PEMD and perturber SIS will reqiure a 'theta_E' in the lenstronomy kwargs list)
Changed numpy dependency from >=1.13,<=1.24 to >=1.13,<=1.23 to avoid deprecation of distutils.