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

Is file size needed? #9

Open
cholmes opened this issue Jun 15, 2017 · 2 comments
Open

Is file size needed? #9

cholmes opened this issue Jun 15, 2017 · 2 comments

Comments

@cholmes
Copy link
Contributor

cholmes commented Jun 15, 2017

One change I made when shifting from OIN to this updated spec was to remove file size.

I definitely do not think it belongs in the abstract spec - if it is to power API's then there are several API's that serve imagery that do not know the file size ahead of time (like Planet - assets are generated on the fly).

I could see an argument for the JSON + Cloud Optimized GeoTIFF case, as it can be useful for clients to know. But it seems like it should be possible to just query the image file itself.

@tombh
Copy link

tombh commented Jun 19, 2017

I think it would be useful to at least have some indication of the size of the asset. So what about making it optional? There is an implied file size from the calculus of footprint * gsd. So instead of 'optional', what about, 'not guaranteed to be actual size on disk'?

@matthewhanson
Copy link
Collaborator

I agree file size is best as an optional parameter. The main reason is that metadata generation doesn't necessarily need, or have, access to the original data file. For our NASA work there are several NASA datasets where metadata is generated by lower level metadata files and don't ever look at the actual data files.
Another issue here is that a "scene" might actually be made up of multiple files, so a single file size at the top metadata level wouldn't make sense.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants