drivendataorg / concept-to-clinic

ALCF Concept to Clinic Challenge
https://concepttoclinic.drivendata.org/
MIT License
367 stars 147 forks source link

Add Vuex state for managing the case beginning-to-end #263

Closed isms closed 6 years ago

isms commented 6 years ago

Remove view guards (at least for right now) - the plan is just not to show UI for elements based on what is in the global state (e.g. no case yet ⇒ don't show detect and select, no marked candidates yet ⇒ don't show annotate and segment, etc).

pjbull commented 6 years ago

LGTM, go go vuex store!

reubano commented 6 years ago

guess this addresses #264?

pjbull commented 6 years ago

@reubano @lamby More fun to review. This branch also still has a lot to be turned into issues. The goal here was to get most of the API calls into state.js except for on "Open Image", which needs to do a little extra work.

TODOs, which we'll probably turn some of these into issues.

reubano commented 6 years ago

@pjbull looking now. btw, I commented on #261, but since it was already merged, not sure if it sent you guys the notification. just fyi..

isms commented 6 years ago

output