-
Notifications
You must be signed in to change notification settings - Fork 7
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
Implemented updateAnnotationCount
mutation
#389
Conversation
Components will now call `updateAnnotationCount` mutation on `input` event of `v-select`
- `selectCategoricalOption`: If the empty option from the v-select is selected the rawValue is removed from `valueMap` - `updateAnnotationCount`: Modified conditions for incrementing the `annotationCount`
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I made a few small change requests, but more important is that you need to adjust the annot-categorical
and annot-continuous-values
component tests. They are currently not passing, and possibly also explains the github Cypress hang.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👨🍳
This PR includes:
updateAnnotationCount
mutationvalueMap
valueMap
is not emptytransformationHeuristic
transformationHeuristic
is notnull
updateAnnotationCount
unit-testselectCategoricalOption
mutation, if the empty option from the v-select is selectedrawValue
is removed from thevalueMap
Closes #388