-
-
Notifications
You must be signed in to change notification settings - Fork 17
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
Investigate combining datasets for dssterrapixel.plate and DSSPngL5to12_* #88
Comments
Quick update from reviewing my notes from inventorying the storage containers:
(Note that level N+1 is 4× as big as level N, so even though levels 0 to 7 are more than half of the levels, their total size is much smaller than level 12.) The file name format in |
Now that we can work with plate files in blob storage, I think we can close this issue for the time being. |
@pkgw Is the 4gb file size limit due to the fileformat itself? It would be great to combine them all into a single platefile |
I'm not sure. I thought it was due to filesystem issues, but from some quick Wikipedia-ing it looks like NTFS has supported files larger than 4 GiB for a long time? And I assume that would be the relevant filesystem. |
Hmm that could be interesting. Azure blobs support huge blobs as well (terabytes) so it could simplify plate files. That would really make it simple to have a data-driven endpoint. I'm going to reopen this to track if it's possible - probably longer term, so feel free to close again. |
Worth keeping in mind here that the DSS dataset is splintered into these sub-plate-files only because of the previous 4 GB file size limit. On the blob storage, the handling here could be simplified to pull data from one large pyramid, rather than splitting between the one file for levels 0-8 (
dssterrapixel.plate
) and the collection of plates for levels 5-12 (DSSpngL5to12_x{1}_y{2}.plate
).Relevant here is that I'm pretty sure that our
wwtfiles
blob container already contains the loose PNG files as a single 0-12 tile pyramid in thedsstp
container.Originally posted by @pkgw in #87 (comment)
The text was updated successfully, but these errors were encountered: