search

FZJ-INM1-BDA / siibralab

Matlab interface to the siibra toolsuite
Apache License 2.0
5 stars 2 forks source link

Error in walkthrough.mlx script line 121/122 #35

Open SBludau opened 1 year ago

SBludau commented 1 year ago

if I understand the error message correctly there seems to be a problem/change in the API?

Many greetings Sebastian

matlab command: v1l_region = julichbrain.decodeRegion("Area hOc1 (V1, 17, CalcS) left"); matlab command: receptorDensities = v1l_region.getReceptorDensities()

Error using matlab.internal.webservices.HTTPConnector/copyContentToByteArray The server returned the status 500 with message "Internal Server Error" in response to the request to URL https://siibra-api-stable.apps.hbp.eu/v2_0/atlases/juelich/iav/atlas/v1.0.0/1/parcellations/minds/core/parcellationatlas/v1.0.0/94c1125b-b87e-45e4-901c-00daee7f2579-290/regions/Area%20hOc1%20(V1,%2017,%20CalcS)%20left/features.

Error in readContentFromWebService (line 46) byteArray = copyContentToByteArray(connection);

Error in webread (line 125) [varargout{1:nargout}] = readContentFromWebService(connection, options);

Error in siibra.internal.API.doWebreadWithLongTimeout (line 23) result = webread( ...

Error in siibra.items.Region/getAllFeatures (line 79) features = siibra.internal.API.doWebreadWithLongTimeout( ...

Error in siibra.items.Region/getReceptorDensities (line 97) allFeatures = obj.getAllFeatures();

xgui3783 commented 1 year ago

Unfortunately receptor density is one of functionality that cannot be restored in v1 and v2 of API endpoints. (or at least not restored easily without significant work)

I would recommend convert to v3 of the endpoint

SBludau commented 1 year ago

Ok, then we should consider removing this part from the walkthrough or if it's not a big hassle, switch to the v3 API.

valentin-pinkau commented 1 year ago

I would suggest to remove this from the walkthrough script as adding support for the region features in v3 is involved and will be done in a separate PR.

dickscheid commented 4 months ago

In siibra-api V3, this should simply return a table (originating from python pandas dataframe), like all other data features which are not images. Suggest to migrate to api V3 to solve a range of issues.

vijayiyer05 commented 2 weeks ago

Checking in re the status of this PR. @SBludau, it's listed as occurring at line 121/122, but the walkthrough I currently see only has 75 lines.