search

uscbp / bodb

Brain Operation DataBase (BODB)
0 stars 0 forks source link

finish API #5

Closed uscbp closed 10 years ago

uscbp commented 10 years ago

wrap up and test api for initial release.

TODOS: add xml to report dropdown add documentation test, test, test

uscbp commented 10 years ago

as per meeting with jimmy today: references/literature on ssr - fixed model bug - fixed figures for ssrs - fixed architecture in the models: inputs, outputs, and states now implemented (but not grouped) added fields and order to all sub seds

TODOS

BrainImagingSerializer - needs to properly handle coordinate spaces and data fields Figures - need to show pk, title, caption, order, and link

then I think the api is ready for documenting and release

jbonaiuto commented 10 years ago

Here's a list of issues I found: All document-derived models:

BOP

Model

All SED-derived models

Brain Imaging SED

ERP SED

Brain Region

mwinter80 commented 10 years ago

great. thanks for the thorough review. sometimes it gets hard to see these things in the trenches. I have my work cut out for me tomorrow.

-Mike

mwinter80 commented 10 years ago

I believe all the issues above should be resolved. It was a lot of work and now I am certainly a rest framework ninja. There is a lot that could be improved still. E.g., naming conventions are flexible (among many other things that I now know how to implement). Let me know if you want me to change anything.

Jimmy, can you retest and make sure thing are ok (especially with erp things). related_names were added to fields in the following models: SEDS ERPComponent SEDCoord TestSEDSSR

SSRS Prediction

jbonaiuto commented 10 years ago

Almost there!

mwinter80 commented 10 years ago

all these issues except tags should be fixed. please check esp. erpsed stuff. I will figure out tags soon I hope!