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
tickets/DM-34158 #26
tickets/DM-34158 #26
Conversation
7450904
to
92dd4da
Compare
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.
A few more minor comments/formatting/suggestions.
scarlet/lite/component.py
Outdated
The array of parameters to constrain. | ||
prox_step: `float` | ||
A scaling parameter used in some proximal operators, | ||
but ignored here. |
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.
Ok, looking at proxmin, I gather operators need to have x and step as the first two args, and can have any number after that? It should probably be a callable class or something then, but in the meantime, it would be useful to make a note here to that effect for a future clueless reader like myself.
max_prox_iter: `int` | ||
Maximum number of proximal iterations. | ||
factor: `int` | ||
The factor to scale the noise to set the |
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.
Ok, perhaps worth noting here then (adding a TODO if a ticket already exists).
This commit implements basic parametric models including PSF sources, sersic models, and general models that are parameterized by the radius and optionally elliptical orientation.
This commit implements basic parametric models including PSF sources, sersic models, and general models that are
parameterized by the radius and optionally elliptical orientation.