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 Neurosift service for AVI files #1983

Merged
merged 1 commit into from
Aug 20, 2024
Merged

Conversation

magland
Copy link
Contributor

@magland magland commented Jul 20, 2024

This is a small change to add an "Open With -> Neurosift" entry for .avi files.

DANDI supports uploading of .avi files, but currently there is no way to preview/stream those files in the browser (browsers do not support AVI).

Neurosift now provides a workaround by using Dendro to precompute .mp4 files associated with portions of the .avi files.

For example:

https://dandiarchive.org/dandiset/001084/draft/files?location=sub-DL18%2Fsub-DL18_ses-211110_image%2Bophys&page=1

With this PR you would be able to "Open With" Neurosift and be directed to

https://neurosift.app/?p=/avi&url=https://api.dandiarchive.org/api/assets/3d760886-c1ac-467d-bd87-3dfd71a5cb65/download/&dandisetId=001084&dandisetVersion=draft

At that link, you should be able to view the first X minutes of that video because the Dendro job had already been submitted and completed. For other AVI files, the .mp4 file might not be prepared, and the Dendro job would need to be submitted. Dendro is still at an early stage and we are working on documenting how to obtain a Dendro API key, etc, to be able to submit those jobs. But it would be a great to establish this connection between the archive and Neurosift even during the development phase.

tag: @bendichter @yarikoptic

@yarikoptic
Copy link
Member

@waxlamp if no objections, I will merge this PR tomorrow.

@waxlamp
Copy link
Member

waxlamp commented Aug 20, 2024

@waxlamp if no objections, I will merge this PR tomorrow.

This serves an immediate need (thank you, @magland!) so you're good to merge, but I still see a path to a DANDI-centric solution (as outlined in #1626 (comment) and following comments) that should come in time.

@yarikoptic yarikoptic added the enhancement New feature or request label Aug 20, 2024
@yarikoptic yarikoptic merged commit 62442c9 into dandi:master Aug 20, 2024
4 checks passed
@dandibot
Copy link
Member

🚀 PR was released in v0.3.95 🚀

@dandibot dandibot added the released This issue/pull request has been released. label Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants