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

adding back dandi_example to commits #45

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

stevevanhooser
Copy link
Collaborator

No description provided.

@stevevanhooser stevevanhooser added this to the v0.8.0 milestone Dec 18, 2023
@stevevanhooser
Copy link
Collaborator Author

Hi all -

These are the comments from pull request #34 that pertain to the file not merged.

In an effort to get out version 7 before Dec 15, @vijayiyer05 and I discussed setting the Dandi integration aside for a little bit.

In consultation with @vijayiyer05, the 50GB file is a bit unwieldy.

There are a few issues with downloading the dandi file.

  • For me, the file doesn't download, even if I call the dandi-cli tool from my terminal.
  • If we are going to ask the users to install the dandi-cli, then I think we ought to tell them how to do it so they don't have to read too much (easy as pip install dandi).
  • The code as it is now calls the dandi command from Matlab via the system command. However, it is not on my path, and I doubt it will be on the path of many users, given that it is in the Python library. So if we're going to do that we ought to have some good solution. Not sure if telling users how to add it to their Matlab path is easier, or walking them through a download on the command line.

Work can continue on the dandihub_work_0.8.0 branch, for which a draft pull request #45 has been issued.

Thanks
Steve

@jbentvelsen jbentvelsen linked an issue Dec 20, 2023 that may be closed by this pull request
@jbentvelsen
Copy link
Collaborator

Hi all -

These are the comments from pull request #34 that pertain to the file not merged.

In an effort to get out version 7 before Dec 15, @vijayiyer05 and I discussed setting the Dandi integration aside for a little bit.

In consultation with @vijayiyer05, the 50GB file is a bit unwieldy.

There are a few issues with downloading the dandi file.

* For me, the file doesn't download, even if I call the dandi-cli tool from my terminal.

* If we are going to ask the users to install the dandi-cli, then I think we ought to tell them how to do it so they don't have to read too much (easy as `pip install dandi`).

* The code as it is now calls the `dandi` command from Matlab via the `system` command. However, it is not on my path, and I doubt it will be on the path of many users, given that it is in the Python library. So if we're going to do that we ought to have some good solution. Not sure if telling users how to add it to their Matlab path is easier, or walking them through a download on the command line.

Work can continue on the dandihub_work_0.8.0 branch, for which a draft pull request #45 has been issued.

Thanks Steve

Hi Steve, thanks for testing this branch. Strange it does not download on your pc, do you get any specific error?

Furthermore, this branch was mainly intended to be used on dandi-hub. In that case, dandi-cli is already installed and on path. Also, downloading the 50GB file will take around 15 minutes. For users running locally, I believe there is a link to the installation instructions in the introduction text of the livescript.

Finally, if there are smaller datasets with raw data available, I agree we should use one of them.

@vijayiyer05 vijayiyer05 modified the milestones: v0.8.0, v0.9.0 May 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Finalize dandi example
3 participants