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

Auto-annotation support for jobs #6276

Merged
merged 298 commits into from
Jun 16, 2023
Merged

Auto-annotation support for jobs #6276

merged 298 commits into from
Jun 16, 2023

Conversation

zhiltsov-max
Copy link
Contributor

@zhiltsov-max zhiltsov-max commented Jun 9, 2023

Motivation and context

Depends on #6204

  • Added an option to run auto-annotation on jobs
  • Added API schema definitions for offline lambda call endpoints

How has this been tested?

Checklist

  • I submit my changes into the develop branch
  • I have added a description of my changes into the CHANGELOG file
  • I have updated the documentation accordingly
  • I have added tests to cover my changes
  • I have linked related issues (see GitHub docs)
  • I have increased versions of npm packages if it is necessary
    (cvat-canvas,
    cvat-core,
    cvat-data and
    cvat-ui)

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.

try:
task = Task.objects.get(id=task)
except Task.DoesNotExist as ex:
raise ValidationError(str(ex))

Check warning

Code scanning / CodeQL

Information exposure through an exception

[Stack trace information](1) flows to this location and may be exposed to an external user.
try:
task = Task.objects.get(id=task_id)
except Task.DoesNotExist as ex:
raise ValidationError(str(ex))

Check warning

Code scanning / CodeQL

Information exposure through an exception

[Stack trace information](1) flows to this location and may be exposed to an external user.
try:
report = QualityReport.objects.get(id=report)
except QualityReport.DoesNotExist as ex:
raise ValidationError(str(ex))

Check warning

Code scanning / CodeQL

Information exposure through an exception

[Stack trace information](1) flows to this location and may be exposed to an external user.
serializer = RqIdSerializer({"rq_id": rq_id})
return Response(serializer.data, status=status.HTTP_202_ACCEPTED)
except qc.QualityReportUpdateManager.QualityReportsNotAvailable as ex:
raise ValidationError(str(ex))

Check warning

Code scanning / CodeQL

Information exposure through an exception

[Stack trace information](1) flows to this location and may be exposed to an external user.
@zhiltsov-max zhiltsov-max changed the title [Dependent] Job auto-annotation Auto-annotation support for jobs Jun 12, 2023
@codecov
Copy link

codecov bot commented Jun 12, 2023

Codecov Report

Merging #6276 (1af9867) into develop (f578e47) will decrease coverage by 25.92%.
The diff coverage is n/a.

❗ Current head 1af9867 differs from pull request most recent head 63b9b99. Consider uploading reports for the commit 63b9b99 to get more accurate results

@@             Coverage Diff              @@
##           develop    #6276       +/-   ##
============================================
- Coverage    78.09%   52.18%   -25.92%     
============================================
  Files          320      145      -175     
  Lines        37783    16934    -20849     
  Branches      6957     3285     -3672     
============================================
- Hits         29507     8837    -20670     
- Misses        7963     8091      +128     
+ Partials       313        6      -307     
Components Coverage Δ
cvat-ui 52.18% <ø> (-21.30%) ⬇️
cvat-server ∅ <ø> (∅)

cvat/schema.yml Outdated Show resolved Hide resolved
cvat/schema.yml Show resolved Hide resolved
@zhiltsov-max zhiltsov-max merged commit f6b0801 into develop Jun 16, 2023
@bsekachev bsekachev deleted the zm/job-autoannotation branch June 28, 2023 08:31
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.

4 participants