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

Optionally use the 'number' column to identify the segmentation value corresponding to the target object #23

Closed
wants to merge 4 commits into from

Conversation

NiallMac
Copy link

When creating the composite weight + segmentation mask, the code needs to assume which segmentation pixel value corresponds to the object in question. By default it checks the central pixels of the stamp. In some of our simulations this occasionally failed. This option allows you to instead use the get_number() method to get the appropriate segmentation pixel value.

@esheldon
Copy link
Owner

esheldon commented Jun 14, 2016

MEDS is designed to be a generic format. The number field may not always exist.

How about making the keyword be segid_column, and one could send "number". E.g.

useg=m.get_cweight_cutout_nearest(iobj, icutout, segid_column='number')

then internally it will use that

if segid_column is not None:
    segid=self[segid_column]
else:
    segid=coadd_seg[coadd_rowcen,coadd_colcen]

@beckermr
Copy link
Collaborator

beckermr commented Jul 3, 2019

This issue is worth closing now.

@esheldon esheldon closed this Mar 26, 2021
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.

3 participants