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

Airphen: add new dataset #1803

Merged
merged 3 commits into from
Jan 26, 2024
Merged

Conversation

adamjstewart
Copy link
Collaborator

@adamjstewart adamjstewart commented Jan 11, 2024

Adds a new GeoDataset for the Hiphen Airphen drone camera.

I'm not allowed to share any of the drone imagery publicly, so you'll have to take my word for it that plotting works.

@adamjstewart adamjstewart added this to the 0.6.0 milestone Jan 11, 2024
@github-actions github-actions bot added documentation Improvements or additions to documentation datasets Geospatial or benchmark datasets testing Continuous integration testing labels Jan 11, 2024
@@ -1,5 +1,6 @@
Dataset,Type,Source,License,Size (px),Resolution (m)
`Aboveground Woody Biomass`_,Masks,"Landsat, LiDAR","CC-BY-4.0","40,000x40,000",30
`Airphen`_,Imagery,Airphen,-,"1,280x960",0.047--0.09
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

License is up to the user who publishes their imagery.

Images are usually stitched together to create larger mosaics.

Resolution depends on customizable focal length and drone height, above values are for an altitude of 100 m.

Comment on lines +44 to +47
# Each camera measures a custom set of spectral bands chosen at purchase time.
# Hiphen offers 8 bands to choose from, sorted from short to long wavelength.
all_bands = ["B1", "B2", "B3", "B4", "B5", "B6", "B7", "B8"]
rgb_bands = ["B4", "B3", "B1"]
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This one is weird. Basically anyone who uses this dataset should really subclass it to specify which bands their data includes, such as:

class MyAirphen(Airphen):
    all_bands = ["B1", "B3", "B4"]

Copy link
Member

@calebrob6 calebrob6 left a comment

Choose a reason for hiding this comment

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

Seems good and TIL Airphen! I take it you are using this imagery in your work :)

torchgeo/datasets/airphen.py Outdated Show resolved Hide resolved
torchgeo/datasets/airphen.py Show resolved Hide resolved
@adamjstewart
Copy link
Collaborator Author

I take it you are using this imagery in your work :)

Yes, same as PRISMA. Expect to see a few more obscure MSI/HSI satellites in the future.

torchgeo/datasets/airphen.py Show resolved Hide resolved
@adamjstewart adamjstewart merged commit d998cb7 into microsoft:main Jan 26, 2024
21 checks passed
@adamjstewart adamjstewart deleted the datasets/airphen branch January 26, 2024 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
datasets Geospatial or benchmark datasets documentation Improvements or additions to documentation testing Continuous integration testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants