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

Expose link to viewer next to each image in dataset image-list page #527

Closed
tcompa opened this issue Jul 16, 2024 · 4 comments · Fixed by #542
Closed

Expose link to viewer next to each image in dataset image-list page #527

tcompa opened this issue Jul 16, 2024 · 4 comments · Fixed by #542

Comments

@tcompa
Copy link
Collaborator

tcompa commented Jul 16, 2024

This is an experimental feature, not to be merged.

The button would link to our own vizarr/data service (e.g. on port 3000), and use the image URL (aka absolute path).
Notice that we would need to map cluster paths into VM paths, but at the testing level we can work around this need in a few ways (e.g. by mounting the cluster share on the VM with the same absolute path, or using symlinks, ...).

@zonia3000
Copy link
Collaborator

In the link-to-viewer branch I added the new experimental link. The branch has been created from summer_improvements, so it contains also those commits.

The dataset page has a new "View" button that opens a link (in a new tab) to the vizarr viewer:

Screenshot from 2024-08-07 18-26-05

The viewer link can be configured using the PUBLIC_FRACTAL_VIZARR_VIEWER_URL environment variable. If the variable is not present the link is not displayed.

@jluethi
Copy link
Collaborator

jluethi commented Aug 8, 2024

Neat! That will be great to add to the Science Cluster deployment!

How does the PUBLIC_FRACTAL_VIZARR_VIEWER_URL need to be set? How does it handle the fact that the base of the path is different on the VM that streams the data from the cluster?

@zonia3000
Copy link
Collaborator

How does the PUBLIC_FRACTAL_VIZARR_VIEWER_URL need to be set?

It's like all the other variables that we are setting in the env file or via export when starting from the script.
It's read at runtime, when the fractal-web app start.

For local testing we can use:

PUBLIC_FRACTAL_VIZARR_VIEWER_URL=http://localhost:3000/vizarr

How does it handle the fact that the base of the path is different on the VM that streams the data from the cluster?

I'm not sure if I have understood the question correctly.

If you are talking about the paths of the files on disk I suppose that, as suggested by @tcompa in the first comment of this issue, we can "mount the cluster share on the VM with the same absolute path". That has to be tested.

Notice that, from an external point of view, the URL of fractal-web must have the same main domain of the URL of vizarr-viewer, otherwise cookies authentication will not work (more details on fractal-vizarr-viewer readme), so we added some proxy configuration (that is already in place).

@jluethi
Copy link
Collaborator

jluethi commented Aug 8, 2024

If you are talking about the paths of the files on disk I suppose that, as suggested by @tcompa in the first comment of this issue, we can "mount the cluster share on the VM with the same absolute path". That has to be tested.

Ah, makes sense. We haven't set that up yet, but going that route makes sense.

@tcompa tcompa changed the title Experimental: expose link to viewer next to each image in dataset image-list page Expose link to viewer next to each image in dataset image-list page Aug 26, 2024
@zonia3000 zonia3000 mentioned this issue Aug 26, 2024
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging a pull request may close this issue.

3 participants