Skip to content
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

Tus for project dataset #4485

Merged
merged 5 commits into from
Mar 22, 2022
Merged

Tus for project dataset #4485

merged 5 commits into from
Mar 22, 2022

Conversation

klakhov
Copy link
Contributor

@klakhov klakhov commented Mar 17, 2022

Motivation and context

Continue TUS integration to provide chunk upload for large dataset files in projects

How has this been tested?

Checklist

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.
  • I have updated the license header for each file (see an example below)
# Copyright (C) 2022 Intel Corporation
#
# SPDX-License-Identifier: MIT

cvat-core/src/server-proxy.js Outdated Show resolved Hide resolved
cvat-core/src/server-proxy.js Outdated Show resolved Hide resolved
cvat-core/src/server-proxy.js Outdated Show resolved Hide resolved
@bsekachev
Copy link
Member

Generally patch works for me except of 2 moments:

  1. When I first time tried to import dataset, I get error with CSRF token (403), see gif above. I re-loginned and was not able to reproduce it anymore. So, maybe it was not related with this patch. Let's see if somebody meet the same issue.
  2. When error occurs, it does not show any errors, probably related with the error I commented in the code (generatedError is not raised)

error

@nmanovic nmanovic merged commit bceae22 into develop Mar 22, 2022
@nmanovic nmanovic deleted the kl/tus-for-dataset branch March 22, 2022 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants