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.

Sign up for GitHub

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

Jump to bottom

Timestamp in TIF metadata for data pipeline output #70

Closed
yellowcap opened this issue Dec 6, 2023 · 4 comments
Closed

Timestamp in TIF metadata for data pipeline output #70

yellowcap opened this issue Dec 6, 2023 · 4 comments
Assignees
@yellowcap
Labels
data-pipeline Pull Requests about the data pipeline
Milestone
v1 Release

Comments

@yellowcap
Copy link
Member

To facilitate its use and persist beyond the file name.

See also #54 (review)
@yellowcap yellowcap self-assigned this Dec 6, 2023
@yellowcap yellowcap mentioned this issue Dec 6, 2023
Merged
@yellowcap yellowcap added the data-pipeline Pull Requests about the data pipeline label Dec 6, 2023
yellowcap added a commit that referenced this issue Dec 6, 2023
@yellowcap
Add date to raster metadata
6c95b6a 
Closes #70
@yellowcap yellowcap mentioned this issue Dec 6, 2023
Merged
@weiji14
Copy link
Contributor

weiji14 commented Dec 6, 2023

Looking at whether there's a standard to encode the acquisition datetime in a GeoTIFF at https://gdal.org/drivers/raster/gtiff.html#metadata. A couple of options:

Both of these expect a datetime (with year, month, date, hour, minute second) though, but currently we are only storing YYYY-MM-DD. Which one should we go for?

yellowcap added a commit that referenced this issue Dec 7, 2023
@yellowcap
Add date to raster metadata
d82ff66 
Closes #70
yellowcap added a commit that referenced this issue Dec 7, 2023
@yellowcap
Add date to raster metadata
426aa06 
Closes #70
@yellowcap
Copy link
Member Author

Looking at whether there's a standard to encode the acquisition datetime in a GeoTIFF at https://gdal.org/drivers/raster/gtiff.html#metadata. A couple of options:

Sorry only saw this now. Went with simple date. But can adapt in v3.

@yellowcap yellowcap reopened this Dec 11, 2023
@weiji14 weiji14 added this to the v1 Release milestone Dec 17, 2023
@weiji14 weiji14 mentioned this issue Dec 19, 2023
Merged
4 tasks
brunosan pushed a commit that referenced this issue Dec 27, 2023
@yellowcap
Small pipeline fixes (#72)
018fcbc 
* Remove default for subset

This is easy to forget and then run with subset without the intention to actually subset

* Improve file name

Closes #69

- Zero padding for counter
- v before version number
- Underscores instead of hyphon separators
- Drop hyphons from date stamp

* Bump version to 02

* Make mgrs sample file external

Closes #71

* Add date to raster metadata

Closes #70

* Improve print statement
@weiji14 weiji14 mentioned this issue Jan 14, 2024
Merged
@yellowcap
Copy link
Member Author

Looking into this, changing this tag needs refactoring also in the data-pipeline, finding the simple "date" tag and replace it with the choice of use ACQUISITIONDATETIME or TIFFTAG_DATETIME.

@yellowcap
Copy link
Member Author

This is in the date tag in the tiff files for v0.1 and v0.2, and will move into the STAC Item / stacchip index for v1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yellowcap yellowcap

Labels
data-pipeline Pull Requests about the data pipeline
Projects
None yet
Milestone
v1 Release
Development

No branches or pull requests
2 participants
@yellowcap @weiji14