DataDreamer - v0.2.0 #68
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Co-authored-by: conorsim <[email protected]>
Feature/add dataset_plugin argument
* fix: replace typing with typing_extensions * fix: remove deprecated truncation argument * feature: add luxonis loader with plugin * feature: modify luxonis dataset with plugin * style: formatting * [Automated] Updated coverage badge * style: remove commented lines * fix: remove dataset_id from luxonis dataset args * fix: remove redundant env var check * [Automated] Updated coverage badge * fix: empty image_paths with luxonis dataset plugin * style: formatting * [Automated] Updated coverage badge * test: simple luxonis dataset test * fix: luxonis dataset converter missing attr * style: formatting * fix: luxonis dataset converter parent * [Automated] Updated coverage badge --------- Co-authored-by: GitHub Actions <[email protected]>
* chore: add gcsfs to requirements.txt * chore: fix dev image tag
* fix: bbox labels in LuxonisDatasetConverter * fix: convert split_ratios to tuple for luxonis-ml * [Automated] Updated coverage badge --------- Co-authored-by: GitHub Actions <[email protected]>
* fix: use plugin arg to load image paths * fix: pre-commit formatting * fix: pass sync_target_directory to loader
* fix: unet from config warning * feat: add logger * test: add utils tests * style: utils tests formatting * fix: args extenstion in merge dataset function * docs: docstrings and return types * [Automated] Updated coverage badge * fix: remove axes in bbox visualization * tests: improve image generation tests * [Automated] Updated coverage badge * docs: fix docstrings formatting * fix: type hints * tests: replace default ubuntu runner with buildjet runner * fix: type hint * test: modify memory computation * test: round up ram computation * test: disable output capturing * test: decrease required ram for demanding tests * test: 8vpcu buildjet runner * test: fix buildjet 8cpu runner * test: fix 8vcpu buildjet * test: divide tests into core and heavy * style: tests formatting * test: rename core tests * test: run core tests on pull to dev * test: fix config paths * [Automated] Updated coverage badge * test: update tests * [Automated] Updated coverage badge * test: run core tests on pr to main * test: rename heavy test scripts --------- Co-authored-by: GitHub Actions <[email protected]>
* chore: add manual GHCR publish trigger from any branch * chore: remove GAR publish on release * chore: rename GHCR on release publish action * chore: modify commit hash extraction
* Add safety features * Fix lm prompt testing * Correct lm prompt testing * [Automated] Updated coverage badge * Rework the safety features * Add truncation * Remove truncation * Update Profanity Filter * Profanity Filter refactor --------- Co-authored-by: GitHub Actions <[email protected]>
* Add Qwen2.5 LM as prompt generator * Format code * fix: prompt text visualization * fix: padding_side="left" in Qwen2.5 to remove warning * docs: update args description --------- Co-authored-by: Nikita Sokovnin <[email protected]>
Feat/add instance segmentation
sokovninn
requested review from
kozlov721,
klemen1999,
tersekmatija,
conorsim and
HonzaCuhel
☂️ Python Coverage
Overall Coverage
New Files
Modified Files
|
Test Results 5 files 5 suites 1h 19m 25s ⏱️ Results for commit 1914f7d. |
kozlov721
approved these changes
Oct 29, 2024
HonzaCuhel
approved these changes
Oct 30, 2024
There was a problem hiding this comment.
LGTM, but can we please first merge this PR as it contains a fix for when using DataDreamer to just annotate images which aren't necessarily all RGB.
* Fix: convert images to RGB * Change the source branch to install * [Automated] Updated coverage badge --------- Co-authored-by: GitHub Actions <[email protected]>
Fix: computation of x, y, w, h
* fix: add images with no annotations to LuxonisDataset * fix(workflow): use branch we run workflow from * test: confirm current branch in workflow * fix(workflows): correctly pass branch arg * fix(workflows): tags * [Automated] Updated coverage badge * chore: change min required luxonis-ml version to 0.5.0 --------- Co-authored-by: conorsim <[email protected]> Co-authored-by: GitHub Actions <[email protected]>
* Add option to keep images with no annotation, by default removing & refactor * [Automated] Updated coverage badge * Rename 'keep_empty_images' to 'keep_unlabeled_images' --------- Co-authored-by: GitHub Actions <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR includes: