-
Notifications
You must be signed in to change notification settings - Fork 0
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
Ch/dev #1
Conversation
The mask sorting was performed according to the mask area, hidding the smaller mask in the mask visualisation. The option reverse has been set to True in the save_masks function
The simplified geometry parameter is added in an odd way. It has to be done in a cleaner way
Update the README and the script segment_images.py to match the changes in swiss-territorial-data-lab/segment-geospatial#1
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.
I tried my best to understand, but it is still hard to understand the code enough to be sure that I get all the implications and the usefulness of the proposed modifications.
It looks like it makes sense, so I trust you.
@@ -251,12 +256,11 @@ def save_masks( | |||
dtype = np.uint16 | |||
else: | |||
dtype = np.uint32 | |||
|
|||
# Generate a mask of objects with unique values | |||
if unique: | |||
# Sort the masks by area in ascending order |
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.
"descending order" since you switched the sorting order.
What difference does it make?
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.
If the masks are sorted in ascending order by area, the larger ones are at the end of the list. When visualising the masks, the smaller ones were hidden by the larger ones. This caused problems when vectorising.
Small changes to the library
segment-geospatial
were made for the project.