[cmalinmayor]

Proposed Change

Simplify both the actual URI (janelia:/linux/path) and the functions/modules/structure.

@msschwartz21 the to_janelia_uri will now accept/recognize any of the universal formats (linux path, smb://, windows UNC thing) and turn them into the janelia uri. Only ambiguous local mounts (/Volumes, X:/ on windows) require the share to be mounted to resolve the janelia uri, and will do nothing if you pass them though. Would you prefer a warning if you pass an unmounted share? It could feasibly be a non-janelia path that should be passed through untouched, but... then you can ignore the warning 😅 [edit: i added the warnings, can still revert if not needed]

[edit edit: as a bonus i added an argparser type thing so it can auto-convert janelia URIs to local paths!]

Unless we change the actual URI based on discussions at the parallel session, I think this is about ready for v1 release.

Checklist

Go through these things before merge. Actions should run automatically to test them, but for information on how to run locally, see CONTRIBUTING.md.

• I have added tests that prove that my feature works in various situations or tests the bugfix (if applicable).
• I have checked that the tests pass and I maintained or improved test coverage (just test-cov).
• I have written docstrings for any new public functions and classes.
• I have checked that linting and formatting pass (just lint).
• I have checked that mypy type checking passes (just typecheck).

[msschwartz21]

This is a good compromise! I approve 🍅