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

Internal server error #91

Closed
Tanaya9299 opened this issue Jun 12, 2020 · 5 comments
Closed

Internal server error #91

Tanaya9299 opened this issue Jun 12, 2020 · 5 comments
Assignees
Labels
bug Something isn't working

Comments

@Tanaya9299
Copy link

The problem

I tested the afids protocol and while using the validator I am getting internal error

Environment

  • Safari (13.1.1)

_

@kaitj kaitj added the bug Something isn't working label Jun 12, 2020
@kaitj kaitj self-assigned this Jun 12, 2020
@kaitj
Copy link
Collaborator

kaitj commented Jun 12, 2020

Hi there! Could you let us know which dropdown option was selected as well as an example of the .fcsv that was used?

@Tanaya9299
Copy link
Author

2009bMNI_T1_TM_01.txt

@kaitj
Copy link
Collaborator

kaitj commented Jun 12, 2020

@Tanaya9299 - Thank you. I took a look at the contents of the .txt file. It seems as if there is some mismatch/missing of information in the first two fiducials (AC, PC) which is causing the internal error. I've included below the two lines that are causing this issue.

Original:

vtkMRMLMarkupsFiducialNode_0,0.023,-8.165,-16.415,0.000,0.000,0.000,1.000,1,1,1,AC,,vtkMRMLScalarVolumeNode1
vtkMRMLMarkupsFiducialNode_1,0.536,-25.470,-2.099,0.000,0.000,0.000,1.000,1,1,1,PC,,vtkMRMLScalarVolumeNode1

Replace with:

vtkMRMLMarkupsFiducialNode_0,0.023,-8.165,-16.415,0.000,0.000,0.000,1.000,1,1,1,1,AC,vtkMRMLScalarVolumeNode1
vtkMRMLMarkupsFiducialNode_1,0.536,-25.470,-2.099,0.000,0.000,0.000,1.000,1,1,1,2,PC,vtkMRMLScalarVolumeNode1

Note the two commas following label description (AC, PC) and the missing label (1, 2). This should fix the file to get around the file.

@afids/afids-validator - note to discuss implementation of checking mismatched column contents to columns. This error bypasses current checks and throws an internal error - also replicated on Chrome (83.0.4103.97). There are correct number of columns, however label description (AC) was in the place of label, and description ended up blank.

@Tanaya9299
Copy link
Author

Thank you so much. It worked.

@kaitj
Copy link
Collaborator

kaitj commented Jun 19, 2020

Opening new issue to move dicussion of the handling of this error. Closing this issue as manually correcting error seen in the .fcsv resolves the server error.

@kaitj kaitj closed this as completed Jun 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants