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

Got "Error loading project Invalid project file." #274

Closed
yarikoptic opened this issue Oct 29, 2021 · 8 comments · Fixed by #275
Closed

Got "Error loading project Invalid project file." #274

yarikoptic opened this issue Oct 29, 2021 · 8 comments · Fixed by #275

Comments

@yarikoptic
Copy link

this integration used to work but seems "has stopped" (I guess we should add some integration testing).

on DANDI archive (e.g. visit page https://gui.dandiarchive.org/#/dandiset/000023/0.210914.1900/files?location=sub-541516760%2F and click on three dots icon available per file) where we direct visitor to nwbexplorer, pointing to nwb file url on our api server which redirects to S3, e.g.

http://nwbexplorer.opensourcebrain.org/nwbfile=https://api.dandiarchive.org/api/assets/95b266d2-7746-4845-80bc-7dbea747d7c4/download/

clicking on such link now leads me to http://nwbexplorer.opensourcebrain.org/user/ba6f5b0f-68a2-4d6f-a9b5-eae2d61beb14/97d2b0043705/geppetto which now says

Error loading project
Invalid project file. Double check the information you have entered and try again.

in the top right pane. nwbfile in ipython shell is also not available.

@pgleeson
Copy link
Collaborator

@yarikoptic Thanks for reporting. Checking this I see that some of the links in Dandisets work fine, e.g. https://gui.dandiarchive.org/#/dandiset/000004 loads fine, so I think the root of the issue is that some nwb files still simply don't load correctly in NWBE. There is a new release planned by @filippomc, and some of these issues should get resolved.

But yes, some automated testing of this would be wise to explicitly see which ones work and which don't. The issue with that really until now had been that some Dandisets are just too big to be within the scope of NWBE. By the way, (also for @bendichter @satra @oruebel), we've not forgotten about the pynwb streaming option. It would require some planning to change our back end handling of NWB files, as the assumption with NWBE (and particularly OSBv2 workspaces) is that the files reside locally. We'll try to get back to you soon about that.

@satra
Copy link
Contributor

satra commented Oct 30, 2021

@pgleeson - just sent a PR. it may be as simple as that.

@filippomc
Copy link
Collaborator

filippomc commented Nov 3, 2021

Thanks @satra! Can test the new release candidate (including the pr above) here: http://35.226.194.155/

@satra
Copy link
Contributor

satra commented Nov 3, 2021

@filippomc - i tried to load a file on this but it seems to have crashed it, now i can't restart it.

@satra
Copy link
Contributor

satra commented Nov 3, 2021

@filippomc - the release candidate gives the same error on the url that @yarikoptic posted:

https://api.dandiarchive.org/api/assets/95b266d2-7746-4845-80bc-7dbea747d7c4/download/
https://dandiarchive.s3.amazonaws.com/blobs/4db/c6b/4dbc6bc8-f6cb-42db-ac6e-1426af22a5ce

both of the above are the same files. the first one is a redirect to the second.

@filippomc
Copy link
Collaborator

filippomc commented Nov 4, 2021

This is the error I can see in the logs:

ValueError: ('Error reading the NWB file.', ("Could not construct IntracellularElectrode object due to: IntracellularElectrode.init: incorrect type for 'description' (got 'bytes', expected 'str')",))

Looks like the file is not readable by pynwb. Will try update to pynwb 2.0.0 (now 1.3.0)

@filippomc
Copy link
Collaborator

filippomc commented Nov 4, 2021

Upgrade to 2.0.0 did fix that file, can try here: http://35.226.194.155/
The upgrade may have broken other files though, please let me know if you find anything weird

@oruebel
Copy link

oruebel commented Nov 4, 2021

ValueError: ('Error reading the NWB file.', ("Could not construct IntracellularElectrode object due to: IntracellularElectrode.init: incorrect type for 'description' (got 'bytes', expected 'str')",))

h5py changed its in behavior from reading strings as str in v2.x to reading strings as bytes in >v3.0. My guess is that this error could be due to the use of h5py 3.x with PyNWB 1.3. PyNWB 2.0 added support for hpy3.x.

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 a pull request may close this issue.

5 participants