Skip to content

Conversation

@itallix
Copy link
Contributor

@itallix itallix commented Feb 5, 2026

Summary

  • Added new job for dataset export
  • Refactored job implementations as executors
  • Added behave with bdd tests for job export
  • Added GHA step to run bdd tests

Checklist

  • The PR title and description are clear and descriptive
  • I have manually tested the changes
  • All changes are covered by automated tests
  • All related issues are linked to this PR (if applicable)
  • Documentation has been updated (if applicable)

Copilot AI review requested due to automatic review settings February 5, 2026 15:51
@itallix itallix requested review from a team as code owners February 5, 2026 15:51
@itallix itallix linked an issue Feb 5, 2026 that may be closed by this pull request
@github-actions github-actions bot added DEPENDENCY Any changes in any dependencies (new dep or its version) should be produced via Change Request on PM TEST Any changes in tests BUILD Geti Tune Backend Issues related to Geti Tune backend labels Feb 5, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR refactors job implementations into an executor pattern and adds dataset export functionality with comprehensive BDD testing.

Changes:

  • Refactored job implementations from a Trainer base class to a more generic Executor base class
  • Added dataset export job with support for COCO, YOLO, and DATUMARO_V2 formats
  • Implemented BDD tests using behave framework for dataset export scenarios

Reviewed changes

Copilot reviewed 50 out of 57 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
application/backend/app/executors/base.py Renamed Trainer to Executor and removed training-specific logic
application/backend/app/executors/dataset_export/exporter.py New dataset exporter implementation with multi-format support
application/backend/app/models/jobs/export_dataset_job.py New job model for dataset export operations
application/backend/app/api/routers/jobs.py Added endpoint handling for dataset export job submission
application/backend/app/lifecycle.py Registered dataset exporter in job controller factory
application/backend/tests/bdd/ BDD test infrastructure and dataset export scenarios
application/backend/app/repositories/filters.py Extracted common filter logic for reuse
application/backend/app/models/dataset_item.py Added REVIEWED_WITH_UNANNOTATED annotation status
.github/workflows/backend-lint-and-test.yaml Added BDD test execution step

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@itallix itallix force-pushed the vitalii/5271-export-job branch from 8ff6903 to a360e22 Compare February 5, 2026 15:53
@itallix itallix marked this pull request as draft February 5, 2026 15:58
@github-actions
Copy link

github-actions bot commented Feb 5, 2026

📊 Test coverage report

Metric Coverage
Lines 34.7%
Functions 74.1%
Branches 88.3%
Statements 34.7%

@github-actions
Copy link

github-actions bot commented Feb 5, 2026

Docker Image Sizes

CPU

Image Size
geti-tune-cpu:pr-5443 2.97G
geti-tune-cpu:sha-1752408 2.97G

GPU

Image Size
geti-tune-gpu:pr-5443 10.95G
geti-tune-gpu:sha-1752408 10.95G

XPU

Image Size
geti-tune-xpu:pr-5443 9.76G
geti-tune-xpu:sha-1752408 9.76G

@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@itallix itallix force-pushed the vitalii/5271-export-job branch from 51d5452 to 4963b12 Compare February 9, 2026 14:02
@itallix itallix force-pushed the vitalii/5271-export-job branch from 4963b12 to b1ae52b Compare February 10, 2026 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BUILD DEPENDENCY Any changes in any dependencies (new dep or its version) should be produced via Change Request on PM Geti Tune Backend Issues related to Geti Tune backend TEST Any changes in tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement job to export dataset

2 participants