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

Add URI widget #86

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add URI widget #86

wants to merge 2 commits into from

Conversation

tischi
Copy link

@tischi tischi commented Oct 26, 2024

No description provided.

@tischi
Copy link
Author

tischi commented Oct 26, 2024

Hi @imagejan et al.,

A challenge is that we would like to prepend the file: scheme for local paths. However, there were issues with making this work properly with the persists option. @imagejan can explain this better.

If we cannot find a good solution for automatically prepending the "file:", maybe we could also consider just not doing it.

It is important that down-stream code can deal with the URI. I wrote here some code to see whether one could live without the "file:" and it seems to be possible. Basically, the pattern would be that one defaults to "file:" whenever one cannot identify the scheme. Maybe that is fair?

@imagesc-bot
Copy link

This pull request has been mentioned on Image.sc Forum. There might be relevant details there:

https://forum.image.sc/t/fiji-friends-weekly-dev-update-thread/103718/8

@tischi
Copy link
Author

tischi commented Oct 27, 2024

ping @bogovicj ☝️

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.

2 participants