search

FZJ-INM1-BDA / siibra-configurations

1 stars 1 forks source link

Add bbox field to bigbrain sections #45

Open AhmetNSimsek opened 5 months ago

AhmetNSimsek commented 5 months ago

Please see https://github.com/FZJ-INM1-BDA/siibra-python/pull/583 for the intended use.

EDT: Need to recalculate after https://github.com/FZJ-INM1-BDA/siibra-python/pull/591

AhmetNSimsek commented 5 months ago

@xgui3783 This PR aims to speed up the spatial queries. Timo has suggested we should also add bounding boxes to maps and spaces. This, however, bloats the maps with many volumes. Any alternative ideas on how to resolve this?

xgui3783 commented 5 months ago

This, however, bloats the maps with many volumes. Any alternative ideas on how to resolve this?

The downside of using the static typing approach, I suppose.

the alternative is to add fetching of bounding box to warm cache