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

Error occurs when using validation mode to open point clouds task #6383

Closed
Kuroneko089 opened this issue Jun 27, 2023 · 5 comments · Fixed by #6507
Closed

Error occurs when using validation mode to open point clouds task #6383

Kuroneko089 opened this issue Jun 27, 2023 · 5 comments · Fixed by #6507
Assignees
Labels
bug Something isn't working ui/ux

Comments

@Kuroneko089
Copy link

Steps to Reproduce (for bugs)

  1. Create a new task, upload point cloud files.
  2. select validation mode and open the task.

Expected Behaviour

Validation mode should works on point cloud task too.

Current Behaviour

When choosing validation mode to open a point cloud task, the error occurs. The error page says "Oops, something went wrong".
The error messages are:

TypeError
e.html(...).removeEventListener is not a function

TypeError: e.html(...).removeEventListener is not a function
at gne.componentWillUnmount (http://localhost:8080/assets/cvat-ui.266a8357a2f055a978b8.min.js:2:9749696)
at http://localhost:8080/assets/cvat-ui.266a8357a2f055a978b8.min.js:2:4654400
at sr (http://localhost:8080/assets/cvat-ui.266a8357a2f055a978b8.min.js:2:4654441)
at fr (http://localhost:8080/assets/cvat-ui.266a8357a2f055a978b8.min.js:2:4656273)
at vs (http://localhost:8080/assets/cvat-ui.266a8357a2f055a978b8.min.js:2:4669727)
at n.unstable_runWithPriority (http://localhost:8080/assets/cvat-ui.266a8357a2f055a978b8.min.js:2:4807636)
at Wo (http://localhost:8080/assets/cvat-ui.266a8357a2f055a978b8.min.js:2:4615024)
at gs (http://localhost:8080/assets/cvat-ui.266a8357a2f055a978b8.min.js:2:4667587)
at ls (http://localhost:8080/assets/cvat-ui.266a8357a2f055a978b8.min.js:2:4663741)
at http://localhost:8080/assets/cvat-ui.266a8357a2f055a978b8.min.js:2:4615315

Context

I hope to evaluate annotation quality with the validation mode, both on image tasks and point cloud tasks.

Your Environment

  • Docker version docker version (e.g. Docker 17.0.05): Docker Desktop 4.17.0 (99724), Engine Version: 20.10.23.
  • Are you using Docker Swarm or Kubernetes? No.
  • Operating System and version (e.g. Linux, Windows, MacOS): Windows 10
@Kuroneko089
Copy link
Author

Browser version: Google Chrome Version 114.0.5735.134 (Official Build) (64-bit)

CVAT version:
Server: 2.5
Core: 9.2.0
Canvas: 2.16.7
UI: 1.52.1

@zhiltsov-max zhiltsov-max added bug Something isn't working ui/ux 3D lidar/radar labels Jun 27, 2023
@zhiltsov-max
Copy link
Contributor

zhiltsov-max commented Jun 27, 2023

Hi, I confirm this is a bug. Thanks for reporting!

@bsekachev, please take a look.

@bsekachev
Copy link
Member

@Kuroneko089

Validation mode is not supported for 3D point clouds.
How was you able to enable it?

image

@bsekachev
Copy link
Member

Also how did you get 2.5 server version if the latest official release is 2.4.8?

@bsekachev bsekachev added need info Need more information to investigate the issue and removed 3D lidar/radar bug Something isn't working ui/ux labels Jun 27, 2023
@bsekachev
Copy link
Member

Okay, sorted out how to reproduce the error.
But basically, as it was said review mode for 3D tasks is not supported.

@bsekachev bsekachev added bug Something isn't working ui/ux and removed need info Need more information to investigate the issue labels Jun 27, 2023
@bsekachev bsekachev reopened this Jun 27, 2023
@bsekachev bsekachev self-assigned this Jul 11, 2023
bsekachev added a commit that referenced this issue Jul 18, 2023
<!-- Raise an issue to propose your change
(https://github.com/opencv/cvat/issues).
It helps to avoid duplication of efforts from multiple independent
contributors.
Discuss your ideas with maintainers to be sure that changes will be
approved and merged.
Read the [Contribution
guide](https://opencv.github.io/cvat/docs/contributing/). -->

<!-- Provide a general summary of your changes in the Title above -->

### Motivation and context
Resolved #6383

### How has this been tested?
<!-- Please describe in detail how you tested your changes.
Include details of your testing environment, and the tests you ran to
see how your change affects other areas of the code, etc. -->

### Checklist
<!-- Go over all the following points, and put an `x` in all the boxes
that apply.
If an item isn't applicable for some reason, then ~~explicitly
strikethrough~~ the whole
line. If you don't do that, GitHub will show incorrect progress for the
pull request.
If you're unsure about any of these, don't hesitate to ask. We're here
to help! -->
- [x] I submit my changes into the `develop` branch
- [x] I have added a description of my changes into the
[CHANGELOG](https://github.com/opencv/cvat/blob/develop/CHANGELOG.md)
file
- [ ] I have updated the documentation accordingly
- [ ] I have added tests to cover my changes
- [x] I have linked related issues (see [GitHub docs](

https://help.github.com/en/github/managing-your-work-on-github/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword))
- [ ] I have increased versions of npm packages if it is necessary

([cvat-canvas](https://github.com/opencv/cvat/tree/develop/cvat-canvas#versioning),

[cvat-core](https://github.com/opencv/cvat/tree/develop/cvat-core#versioning),

[cvat-data](https://github.com/opencv/cvat/tree/develop/cvat-data#versioning)
and

[cvat-ui](https://github.com/opencv/cvat/tree/develop/cvat-ui#versioning))

### License

- [x] I submit _my code changes_ under the same [MIT License](
https://github.com/opencv/cvat/blob/develop/LICENSE) that covers the
project.
  Feel free to contact the maintainers if that's a concern.
mikhail-treskin pushed a commit to retailnext/cvat that referenced this issue Oct 25, 2023
<!-- Raise an issue to propose your change
(https://github.com/opencv/cvat/issues).
It helps to avoid duplication of efforts from multiple independent
contributors.
Discuss your ideas with maintainers to be sure that changes will be
approved and merged.
Read the [Contribution
guide](https://opencv.github.io/cvat/docs/contributing/). -->

<!-- Provide a general summary of your changes in the Title above -->

### Motivation and context
Resolved cvat-ai#6383

### How has this been tested?
<!-- Please describe in detail how you tested your changes.
Include details of your testing environment, and the tests you ran to
see how your change affects other areas of the code, etc. -->

### Checklist
<!-- Go over all the following points, and put an `x` in all the boxes
that apply.
If an item isn't applicable for some reason, then ~~explicitly
strikethrough~~ the whole
line. If you don't do that, GitHub will show incorrect progress for the
pull request.
If you're unsure about any of these, don't hesitate to ask. We're here
to help! -->
- [x] I submit my changes into the `develop` branch
- [x] I have added a description of my changes into the
[CHANGELOG](https://github.com/opencv/cvat/blob/develop/CHANGELOG.md)
file
- [ ] I have updated the documentation accordingly
- [ ] I have added tests to cover my changes
- [x] I have linked related issues (see [GitHub docs](

https://help.github.com/en/github/managing-your-work-on-github/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword))
- [ ] I have increased versions of npm packages if it is necessary

([cvat-canvas](https://github.com/opencv/cvat/tree/develop/cvat-canvas#versioning),

[cvat-core](https://github.com/opencv/cvat/tree/develop/cvat-core#versioning),

[cvat-data](https://github.com/opencv/cvat/tree/develop/cvat-data#versioning)
and

[cvat-ui](https://github.com/opencv/cvat/tree/develop/cvat-ui#versioning))

### License

- [x] I submit _my code changes_ under the same [MIT License](
https://github.com/opencv/cvat/blob/develop/LICENSE) that covers the
project.
  Feel free to contact the maintainers if that's a concern.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ui/ux
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants