Skip to content

CICD testing integration#3

Merged
kvaikunthan merged 5 commits into
masterfrom
ci-cd-testing
Jul 3, 2026
Merged

CICD testing integration#3
kvaikunthan merged 5 commits into
masterfrom
ci-cd-testing

Conversation

@harshi-puli

Copy link
Copy Markdown
Contributor

Added python based testing units to test various aspects of the data generation pipeline.

The following changes were made:

  • ci.yml (outlines github actions)
  • test/test_generate.py (tests the actual generated output: valid yolo labelling, null cases)
  • test/test_overlay_single.y (tests if overlay abides by certain rules and if the functions are actually returning images/labels.) + Also creates the target images for the test_smoke.py.
  • test/smoke.py (tests the data generation pipeliine against a small dataset within test/fixtures)
  • test/fixtures/{backgrounds or targets} - the basic flat color images that are used to test the data generation pipeline in practice.

Process Testing:

  • GitHub actions worked within ci-cd-testing branch.

@kvaikunthan

Copy link
Copy Markdown
Contributor

Why are there edits to generate.py? Need to check if those are safe.

@kvaikunthan

Copy link
Copy Markdown
Contributor

Ok so I think you may have not pulled from main before making the changes 😭

@kvaikunthan kvaikunthan self-requested a review July 3, 2026 02:47

@kvaikunthan kvaikunthan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

@kvaikunthan kvaikunthan merged commit 7eae6c1 into master Jul 3, 2026
2 checks passed
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.

2 participants