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

Create AWS S3 Interface for TIFF Files #458

Draft
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

stuwilkins
Copy link
Contributor

@stuwilkins stuwilkins commented Sep 13, 2020

@MarkRivers .. This is more of an exploratory "DEMO" PR, I wanted to test using the AWS S3 storage directly from AD that we talked about over e-mail.

I have verified that this works at the moment writing to a MinIO object store at SDCC here at BNL but I have no reason it would not work directly to Amazon S3 object storage.

MinIO is here:

https://min.io/

This basically gets rid of the requirement to have a file system to write images to.

This is "super hacky" and is so brute force and ugly it might even be elegant.

With some changes to the base NDFileTIFF it would only be a small shim over the files.

Comments from all welcome...

P.S. I realize that there is copied code in there. I didn't want to play too much with the NDFileTIFF at this stage. However, one could separate this out a little bit and make the NSFileTIFFS3 really simple.

@coveralls
Copy link

coveralls commented Sep 13, 2020

Coverage Status

Coverage remained the same at 38.916% when pulling 1a6cc51 on stuwilkins:add_tiff_s3 into 1324d63 on areaDetector:master.

@tacaswell
Copy link

https://github.com/constantinpape/z5 may also be of interest to target more cloud-native data formats.

@stuwilkins
Copy link
Contributor Author

Thanks @tacaswell This is just a test, not sure I am married to TIFF (as you know!). But, by using the AWS S3 async upload I can get line rate out of my office computer.........

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.

3 participants