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

silx.io.open: Added basic support for tiled URLs #4121

Draft
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

t20100
Copy link
Member

@t20100 t20100 commented May 3, 2024

Checklist:


This PR adds minimal support for tiled in silx.io.open through a silx.io.tiledh5 module.
As it stand, it only supports Container and ArrayClient.

There is a way to limit the number of entries retrieved for each group by setting silx.io.tiledh5.TiledH5.MAX_CHILDREN_PER_GROUP (default: all).

silx.io.open tries to open http(s) URLs with h5pyd and tiled when available and raises an IOError exception when both fail.
Other exceptions when accessing content (timeout, connection errors) are not catched.

There a shortcut in silx.io.open added in first poc when passing an URL with a tiled- prefix that I would prefer to remove now that the basic URL can be used. (Note: since first poc in this branch, I changed the prefix from tiled: to tiled- to avoid issues with URL parsing) attn @vasole

Related to #4106

@vasole
Copy link
Member

vasole commented May 5, 2024

I changed the prefix from tiled: to tiled- to avoid issues with URL parsing) attn @vasole

vasole/pymca#1075 should deal with it

@danielballan
Copy link

Hello! I am just writing to provide a heartbeat...we're very interested in this work. We have recently had our time consumed with some urgent operational tasks, and NSLS-II heads back into operations next week. We look forward to digging into this.

src/silx/io/tiledh5.py Show resolved Hide resolved
src/silx/io/tiledh5.py Show resolved Hide resolved
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.

4 participants