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

Add class_names/class_property_name to dataset config file #139

Merged
merged 1 commit into from
Feb 19, 2025

Conversation

favyen2
Copy link
Collaborator

@favyen2 favyen2 commented Feb 18, 2025

These are for annotation / prediction layers to indicate which GeoJSON properties correspond to some notion of a class, or for non-image single-band GeoTIFFs to define what the different values in the band mean.

It could be useful in the future to do things like rasterize a vector layer to a corresponding raster layer; or we can move some similar definitions from model configuration file to dataset configuration file (e.g. make property_name/classes unnecessary in rslearn.train.tasks.classification.ClassificationTask). For now it will be used for ES Studio syncing.

these are for annotation / prediction layers to indicate which GeoJSON properties
correspond to some notion of a class, or for non-image GeoTIFFs to define what the
different values mean. it could be useful in the future to do things like rasterize
a vector layer to a corresponding raster layer. for now it will be used for es
studio syncing.
@favyen2 favyen2 requested a review from APatrickJ February 18, 2025 22:31
Copy link
Collaborator

@APatrickJ APatrickJ left a comment

Choose a reason for hiding this comment

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

This makes sense to me, including the per-band raster settings.

Thanks for doing this so quickly!

@favyen2 favyen2 merged commit 755094c into master Feb 19, 2025
4 checks passed
@favyen2 favyen2 deleted the favyen/add-classes-to-dataset-config branch February 19, 2025 16:34
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.

2 participants