dandi / dandi/dandi-cli

working with video files

Open
#918 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
28
Forks
37
Avg merge
1d 17h
Merged PRs (30d)
9

Description

Related to the recent merge: #841 There are two things that I would like to discuss:

1. Currently to validate and upload video files we need to specify the option `allow-any-path`. Since we now explicitly organize video files with `dandi organise` I dont think this options is a good idea as the user may be able to upload other unwanted files along with videos. Additionally, for this option to work the `DANDI_DEVEL` env variable needs to exist so any user would need to do: `DANDI_DEVEL=1 dandi validate --allow-any-path`. This is rather un-user-friendly for common users. (edit by @yarikoptic - dedicated https://github.com/dandi/dandi-cli/issues/920 )
2. Need to validate video files:
- Currently we support the organization of `".mp4", ".avi", ".wmv", ".mov", ".flv"` formats. Validation could enforce these extensions. While this list of file containers are only a start, we will build a better list once things get rolling and we encounter different video formats used in animal experiments.
- It is theoretically possible for acquisition systems to output corrupted videos: have dropped frames, no frames. Ideally its the users responsibility to correct this so may not be something for DANDI to worry about. These errors could be checked by opening the file using opencv within a `try/except` block. I am not sure if this happens often enough to make it a necessary check everytime.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.