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

Feature : Multimasks training for RAMP #240

Merged
merged 11 commits into from
Sep 3, 2024
Merged

Conversation

kshitijrajsharma
Copy link
Member

@kshitijrajsharma kshitijrajsharma commented Mar 24, 2024

What does this PR do ?

API

  • Adds API endpoint for training two support three new parameters input_contact_spacing , input_boundary_width and multimasks
  • Provides option in training to choose between binary masks and multimasks
  • Records multimasks extra param in description of training if multimasks is chosen

Frontend

Screenshots

image

How to test ?

  • Download preprocessed.tar.xz from training with multimasks enabled , You should have multimasks folder added
    image

  • When you load them in to QGIS you should see multiple bands for the masks like following . You can compare with the binary masks if you like
    image

  • While doing training you will notice multimasks folder being used if enabled.

  • if multimask feature is disabled then things should work as it is as used to before

  • Compare the prediction with and without the boundary option ( specially in dense area )

@kshitijrajsharma kshitijrajsharma changed the title Feature/multimask Feature : Multimasks training for RAMP Mar 24, 2024
@kshitijrajsharma kshitijrajsharma marked this pull request as ready for review March 25, 2024 06:06
@kshitijrajsharma kshitijrajsharma removed the request for review from omranlm April 3, 2024 12:22
@kshitijrajsharma kshitijrajsharma merged commit 1aa818c into master Sep 3, 2024
4 checks passed
@kshitijrajsharma kshitijrajsharma deleted the feature/multimask branch September 3, 2024 09:10
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.

1 participant