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

DM-30108: Implement merging sources and flagging saturated #50

Merged
merged 4 commits into from May 18, 2022

Conversation

fred3m
Copy link
Contributor

@fred3m fred3m commented Apr 11, 2022

No description provided.

@fred3m fred3m force-pushed the tickets/DM-30108 branch 2 times, most recently from 4cace12 to 893b2cf Compare April 11, 2022 18:45
@fred3m fred3m force-pushed the tickets/DM-30108 branch 2 times, most recently from a2e5dfe to d2bb696 Compare April 26, 2022 18:10
Copy link

@taranu taranu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mostly minor nitpicks, but hopefully more useful than annoying.

python/lsst/meas/extensions/scarlet/scarletDeblendTask.py Outdated Show resolved Hide resolved
python/lsst/meas/extensions/scarlet/scarletDeblendTask.py Outdated Show resolved Hide resolved
python/lsst/meas/extensions/scarlet/scarletDeblendTask.py Outdated Show resolved Hide resolved
python/lsst/meas/extensions/scarlet/scarletDeblendTask.py Outdated Show resolved Hide resolved
python/lsst/meas/extensions/scarlet/scarletDeblendTask.py Outdated Show resolved Hide resolved
python/lsst/meas/extensions/scarlet/scarletDeblendTask.py Outdated Show resolved Hide resolved
python/lsst/meas/extensions/scarlet/scarletDeblendTask.py Outdated Show resolved Hide resolved
python/lsst/meas/extensions/scarlet/source.py Outdated Show resolved Hide resolved
python/lsst/meas/extensions/scarlet/source.py Outdated Show resolved Hide resolved
python/lsst/meas/extensions/scarlet/source.py Outdated Show resolved Hide resolved
@fred3m fred3m force-pushed the tickets/DM-30108 branch 2 times, most recently from 854fd37 to 0c44373 Compare May 17, 2022 21:11
@fred3m fred3m merged commit 3415c8a into main May 18, 2022
@fred3m fred3m deleted the tickets/DM-30108 branch May 18, 2022 16:53
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.

None yet

3 participants