-
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
Fixed resources import #5909
Fixed resources import #5909
Conversation
/check |
❌ Some checks failed |
/check |
❌ Some checks failed |
bfeae9a
to
55d4ce7
Compare
@Marishka17, AFAIK, the problem is also actual for interrupted backup uploads. |
55d4ce7
to
549911b
Compare
@zhiltsov-max, Yes, I know, this PR should fix that too. |
/check |
❌ Some checks failed |
tests/python/rest_api/test_tasks.py
Outdated
task.import_annotations(self.format, filename) | ||
self._check_annotations(task_id) | ||
|
||
@pytest.mark.timeout(70) |
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.
Probably, it makes sense to reduce the delay down to a couple of seconds. We totally don't want unit tests running more than a minute, unless they are in a special test suite.
Please resolve merge conflicts and add a changelog entry. |
Motivation and context
Fixed:
Resolves #5773
Resolves #5563
How has this been tested?
Manually
Checklist
develop
branch- [ ] I have increased versions of npm packages if it is necessary(cvat-canvas,
cvat-core,
cvat-data and
cvat-ui)
License
Feel free to contact the maintainers if that's a concern.