-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enable configurable set of artifacts #384
Comments
The KVis subcontractor, David Mackey, completed an initial implementation a few weeks back. When the PR was reviewed, Zach had some style requests -- related to how to initialize the new Django models. David tried, but was not able to implement all of Zach's suggestions successfully. The PR (#574) has been updated to partially address the request. After about two weeks of engineering attempt, I directed Dave to continue with the needs for the Lung Cancer application, instead. |
Thanks Curt. I can take over the branch if it's something we want to get merged. |
Yes, we would like to have this feature. |
@zachmullen , yes, please take a look. David ran into some issues where the artifacts weren't initialized early enough in startup for the Vue UI to be responsive. Maybe you can incorporate this more quickly. |
An update, we have fallen back to the approach that worked for us and will revisit this issue when we have a consistent "annotation version" of MIQA. We can then consider what parts of this solution we want to include through a PR. |
No description provided.
The text was updated successfully, but these errors were encountered: