Kitware / UPennContrast

UPenn ?
https://upenn-contrast.netlify.com/
Apache License 2.0
8 stars 6 forks source link

I'm wondering: does the JSON import code use the new multiple property value endpoint? #752

Closed arjunrajlab closed 4 months ago

arjunrajlab commented 4 months ago

I'm not sure I fully understand this code, but it seems like it uploads the property values one at a time? That would be very slow, I think.

https://github.com/Kitware/UPennContrast/blob/f7408f7811d437464d4cbbdbe0b73f2ab3b4df62/src/components/AnnotationBrowser/AnnotationImport.vue#L312C13-L317C1

bruyeret commented 4 months ago

Yes it doesn't use the new endpoint, I am fixing it 👍