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

fairgraph File query fails with code=500 'Internal Server Error' #36

Closed
mih opened this issue Dec 9, 2022 · 5 comments · Fixed by #60
Closed

fairgraph File query fails with code=500 'Internal Server Error' #36

mih opened this issue Dec 9, 2022 · 5 comments · Fixed by #60

Comments

@mih
Copy link
Member

mih commented Dec 9, 2022

Report is here #35 (comment)

I just reconfirmed that with my account things are still working as documented in that PR.

@adswa
Copy link
Member

adswa commented Dec 9, 2022

It looks like a failure in the middle of a successful process to me. The dataset did get quite a bit of history

adina@muninn in /tmp/kgdstryout on git:master!
❱ git log --oneline     
5a86cfa (HEAD -> master, tag: v2.6) In this version, the maximum probability map is comprised of 119 brain regions in each hemisphere and available for the following coordinate spaces: + MNI ICBM 152 (2009c Nonlinear Asymmet[...]
9ebd888 (tag: v2.5) In this version, the maximum probability map is comprised of 119 brain regions in each hemisphere and available for the following coordinate spaces: + MNI ICBM 152 (2009c Nonlinear Asymmetric) + Colin [...]
adef09b (tag: v2.4) In this version, the maximum probability map is comprised of 120 brain regions in each hemisphere and available for the following coordinate spaces: + MNI ICBM 152 (2009c Nonlinear Asymmetric) + Colin [...]
a898d5f (tag: v2.2) In this version, the maximum probability map is comprised of 120 brain regions in each hemisphere and available for the following coordinate spaces: + MNI ICBM 152 (2009c Nonlinear Asymmetric) + Colin [...]
ae29844 (tag: v1.18) In this version, the maximum probability map is comprised of 106 brain regions in each hemisphere and available for the following coordinate spaces: + MNI ICBM 152 (2009c Nonlinear Asymmetric) + Colin [...]
a326f45 (tag: v1.13) This is the first version of this dataset. In this version, the maximum probability map is comprised of 77 brain regions in each hemisphere and available for the following coordinate spaces: + Colin 27[...]
f473170 [DATALAD] new dataset

Its current state is this, it seems all contained files were deleted

❱ git status
On branch master
Changes not staged for commit:
  (use "git add/rm <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
	deleted:    MPM-collections/26/Licence%20%28CCBY-NC-SA%29.pdf
	deleted:    MPM-collections/26/ListOfPMFiles_icbm152casym.csv
	deleted:    MPM-collections/26/MPM/JulichBrain_MPMAtlas_l_N10_nlin2Stdicbm152asym2009c_publicDOI_3f5ec6016bc2242769c41befdbc1b2e0.nii.gz
	deleted:    MPM-collections/26/MPM/JulichBrain_MPMAtlas_l_N10_nlin2Stdicbm152asym2009c_publicDOI_3f5ec6016bc2242769c41befdbc1b2e0.xml
	deleted:    MPM-collections/26/MPM/JulichBrain_MPMAtlas_r_N10_nlin2Stdicbm152asym2009c_publicDOI_453c4c95827e38dcce5372b200065cba.nii.gz
	deleted:    MPM-collections/26/MPM/JulichBrain_MPMAtlas_r_N10_nlin2Stdicbm152asym2009c_publicDOI_453c4c95827e38dcce5372b200065cba.xml
	deleted:    MPM-collections/26/jubrain-ontology_26.json
	deleted:    version

If I restore those deletions, I can get files fine:

(ebrains) adina@muninn in /tmp/kgdstryout on git:master
❱ datalad get MPM-collections/26/jubrain-ontology_26.json
get(ok): MPM-collections/26/jubrain-ontology_26.json (file) [from web...]                                                                                                                                                        
(ebrains) adina@muninn in /tmp/kgdstryout on git:master
❱ datalad get MPM-collections/26/Licence%20%28CCBY-NC-SA%29.pdf
get(ok): MPM-collections/26/Licence%20%28CCBY-NC-SA%29.pdf (file) [from web...]    

As a small side note, the file names have some occasional encoding issues (e.g., Licence%20%28CCBY-NC-SA%29.pdf).

@mih
Copy link
Member Author

mih commented Jan 23, 2023

Today it happened again. There was no code change since the last successful run on the very same dataset.

@mih
Copy link
Member Author

mih commented Jan 23, 2023

The error seems to be specific to the particular query being made. Here is the line where it fails

>>> ds = omcore.Dataset.list(self.client, versions=dv)
-> crash
>>> type(dv)
<class 'fairgraph.openminds.core.products.dataset_version.DatasetVersion'>
# no obvious issues with that instance
>>> self.client.user_info()
User(alternate_name='mih', name='...', email=None, given_name='...', family_name='...', identifiers=['fc583e67-...'])

So the general connection and basic authentication seems to be working still in this error case.

@mih
Copy link
Member Author

mih commented Jan 24, 2023

One day later, things are back working. No change in code or account status.

@mih
Copy link
Member Author

mih commented Mar 7, 2023

A fix is coming as discussed in #58

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.

2 participants