-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Project task subsets #2774
Project task subsets #2774
Conversation
@vnishukov Please, test the PR and review the code. |
cvat-core/src/api-implementation.js
Outdated
@@ -222,30 +216,26 @@ | |||
owner: isString, | |||
search: isString, | |||
status: isEnum.bind(TaskStatus), | |||
without_tasks: isBoolean, |
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.
Why snake case here?
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.
Due to we rely on fields in server requests params. Added replacing.
* Added task project subsets * Added components list key * Added subset field resetting and subset header * Added CHANGELOG and increased npm package version * Added replacing camelcase to snake
Motivation and context
This PR provide new field in task such as
subset
to divide project tasks in some subsets f.e. train/test/val will be. This field will be used in the project dump in future project feature implementation.How has this been tested?
Checklist
develop
branch- [ ] I have updated the documentation accordingly- [ ] I have added tests to cover my changes- [ ] I have linked related issues (read github docs)cvat-core, cvat-data and cvat-ui)
License
Feel free to contact the maintainers if that's a concern.