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

Isosurface route does not support agglomerate mapping #4509

Closed
philippotto opened this issue Mar 31, 2020 · 1 comment · Fixed by #4531
Closed

Isosurface route does not support agglomerate mapping #4509

philippotto opened this issue Mar 31, 2020 · 1 comment · Fixed by #4531
Labels

Comments

@philippotto
Copy link
Member

philippotto commented Mar 31, 2020

The front-end sends requests like this:

{"position":[200700,194940,3580],"cubeSize":[260,260,260],"zoomStep":1,"segmentId":1483607019,"mapping":"agglomerate_view_65","voxelDimensions":[4,4,4]}

and the back-end throws an error, since it probably expects a JSON mapping. Ask me for a link to reproduction.

@fm3
Copy link
Member

fm3 commented Apr 7, 2020

Do you think the frontend could be modified so that it also supplies the mapping type (hdf5 vs json) in the request? Of course we could check which exists in the backend, but that would be another IO operation (or an additional cache) so it may impact performance. If this is a simple change in the frontend, I’d think it’s the better solution

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

2 participants