Enhance PR comment trigger to include last commit message and workflow result #4316
Conversation
This file contains hidden or 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
eugene123tw
requested review from
samet-akcay,
kprokofi,
sovrasov,
Daankrol,
djdameln,
ashwinvaidya17,
rajeshgangireddy and
atwinand
as code owners
ashwinvaidya17
added a commit
that referenced
this pull request
Apr 9, 2025
* Update Documentation (#4294) * update docs * walk through docs * minor * modify docs * fix readme * update readme. Delete outdated sections * 🧹 Minor cleanup (#4299) Refactor import statements to use 'otx.algo.utils.utils' for InstanceData across multiple files * Promote/Demote Inst Seg Algo (#4301) Update model categories in Mask R-CNN templates for balance and accuracy * Reduce Integration Test Scope (#4303) * Reduce Integration Test Scope to 2-3 recipes per task * Add PR comment trigger workflow * Improve PR comment event trigger (#4307) * Add GitHub Actions comments for workflow run status in PR * Update permissions in PR comment trigger workflow (#4309) * Update PR comment trigger (#4312) Update PR comment trigger workflow to enhance permissions and improve PR description updates * Add workflow step to update PR description with run link and status (#4314) * Enhance PR comment trigger to include last commit message and workflow result (#4316) Enhance PR comment trigger to include last commit message and workflow run summary * Refactor PR comment trigger (#4318) Refactor PR comment trigger to include workflow status emoji and streamline run summary * Support linear classifier fine-tuning for classification models (#4298) * Backbone frozen in engine.py and head_only.yaml in multi_class_cls * update * Add : implemented linear fine-tuning for model layers * Updated frozen backbone settings/ * fixes from comments #4286 * Removed the coed comments #4298 * fixes from review #4298 * trailing space and removed head_only.yaml #4298 * Resolve the Dependency Inversion Problem#4298 * removed white space #4298 Update engine.py * removed white space #4298 * pre-commit * Added Unit test for frrze_backbone * update precommit * modified test_freeze_backbone in test_deit_tiny.py * modified test_freeze_backbone in test_deit_tiny.py * modified test_freeze_backbone in test_deit_tiny.py * run pre-commit --------- Co-authored-by: Vladislav Sovrasov <[email protected]> * 🧹 Refactor detection entity (#4322) * Remove detection entity --------- Signed-off-by: Ashwin Vaidya <[email protected]> * Update integration test commands to include run-category-only flag options (#4325) * Bump rich from 13.9.4 to 14.0.0 (#4324) Bumps [rich](https://github.com/Textualize/rich) from 13.9.4 to 14.0.0. - [Release notes](https://github.com/Textualize/rich/releases) - [Changelog](https://github.com/Textualize/rich/blob/master/CHANGELOG.md) - [Commits](Textualize/rich@v13.9.4...v14.0.0) --- updated-dependencies: - dependency-name: rich dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> --------- Signed-off-by: Ashwin Vaidya <[email protected]> Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Prokofiev Kirill <[email protected]> Co-authored-by: Eugene Liu <[email protected]> Co-authored-by: limgyuil <[email protected]> Co-authored-by: Vladislav Sovrasov <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
ashwinvaidya17
added a commit
that referenced
this pull request
Apr 14, 2025
* Update Documentation (#4294) * update docs * walk through docs * minor * modify docs * fix readme * update readme. Delete outdated sections * 🧹 Minor cleanup (#4299) Refactor import statements to use 'otx.algo.utils.utils' for InstanceData across multiple files * Promote/Demote Inst Seg Algo (#4301) Update model categories in Mask R-CNN templates for balance and accuracy * Reduce Integration Test Scope (#4303) * Reduce Integration Test Scope to 2-3 recipes per task * Add PR comment trigger workflow * Improve PR comment event trigger (#4307) * Add GitHub Actions comments for workflow run status in PR * Update permissions in PR comment trigger workflow (#4309) * Update PR comment trigger (#4312) Update PR comment trigger workflow to enhance permissions and improve PR description updates * Add workflow step to update PR description with run link and status (#4314) * Enhance PR comment trigger to include last commit message and workflow result (#4316) Enhance PR comment trigger to include last commit message and workflow run summary * Refactor PR comment trigger (#4318) Refactor PR comment trigger to include workflow status emoji and streamline run summary * Support linear classifier fine-tuning for classification models (#4298) * Backbone frozen in engine.py and head_only.yaml in multi_class_cls * update * Add : implemented linear fine-tuning for model layers * Updated frozen backbone settings/ * fixes from comments #4286 * Removed the coed comments #4298 * fixes from review #4298 * trailing space and removed head_only.yaml #4298 * Resolve the Dependency Inversion Problem#4298 * removed white space #4298 Update engine.py * removed white space #4298 * pre-commit * Added Unit test for frrze_backbone * update precommit * modified test_freeze_backbone in test_deit_tiny.py * modified test_freeze_backbone in test_deit_tiny.py * modified test_freeze_backbone in test_deit_tiny.py * run pre-commit --------- Co-authored-by: Vladislav Sovrasov <[email protected]> * 🧹 Refactor detection entity (#4322) * Remove detection entity --------- Signed-off-by: Ashwin Vaidya <[email protected]> * Update integration test commands to include run-category-only flag options (#4325) * Bump rich from 13.9.4 to 14.0.0 (#4324) Bumps [rich](https://github.com/Textualize/rich) from 13.9.4 to 14.0.0. - [Release notes](https://github.com/Textualize/rich/releases) - [Changelog](https://github.com/Textualize/rich/blob/master/CHANGELOG.md) - [Commits](Textualize/rich@v13.9.4...v14.0.0) --- updated-dependencies: - dependency-name: rich dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Fix integration test commands run-category-only flag (#4329) * updated repo with mobilenet_v4 (#4321) * updated repo with mobilenet_v4 * Introduce MobileNetV4 for Improved Performance * Introduced MobileNetV4 recipe for Improved Performance --------- Co-authored-by: Vladislav Sovrasov <[email protected]> * Update documentation and references to Open-Edge Platform (#4331) * Update documentation and references to Open-Edge Platform- Updated dataset format links in auto_configuration.rst, models_optimization.rst, and various algorithm documentation to point to the Open-Edge Platform repository. - Changed references from openvinotoolkit to open-edge-platform in multiple files including anomaly detection, classification, object detection, segmentation, and product design documentation. - Modified installation instructions and CLI command references to reflect the new repository URL. - Adjusted the help formatter to display the updated GitHub repository link. - Ensured all links to Datumaro and related resources are consistent with the new repository structure. * 🧹 Refactor Instance Segmentation Entity (#4327) Signed-off-by: Ashwin Vaidya <[email protected]> * 🧹 Refactor Anomaly Entities (#4328) * Remove anomaly entities Signed-off-by: Ashwin Vaidya <[email protected]> --------- Signed-off-by: Ashwin Vaidya <[email protected]> * Add a contribution guide (#4335) * Add a contribution guide * Fix a typo * Apply grammar suggestions Co-authored-by: Rajesh Gangireddy <[email protected]> --------- Co-authored-by: Rajesh Gangireddy <[email protected]> * Bug Fix : Convert labels for multi-label Classification into LongTensor before one_hot encoding (#4336) * Update classification.py * make ruff happy * make it simpler * Update src/otx/core/data/dataset/classification.py Co-authored-by: Daan Krol <[email protected]> --------- Co-authored-by: Daan Krol <[email protected]> * Bump OV and NNCF to 2025.1 (#4334) * Bump OV to 2025.1 * Bump NNCF * Update changelog * Update imports --------- Signed-off-by: Ashwin Vaidya <[email protected]> Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Prokofiev Kirill <[email protected]> Co-authored-by: Eugene Liu <[email protected]> Co-authored-by: limgyuil <[email protected]> Co-authored-by: Vladislav Sovrasov <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Gaurav Singh <[email protected]> Co-authored-by: Rajesh Gangireddy <[email protected]> Co-authored-by: Daan Krol <[email protected]>
sovrasov
added a commit
that referenced
this pull request
Jun 25, 2025
* Update feature branch (#4333) * Update Documentation (#4294) * update docs * walk through docs * minor * modify docs * fix readme * update readme. Delete outdated sections * 🧹 Minor cleanup (#4299) Refactor import statements to use 'otx.algo.utils.utils' for InstanceData across multiple files * Promote/Demote Inst Seg Algo (#4301) Update model categories in Mask R-CNN templates for balance and accuracy * Reduce Integration Test Scope (#4303) * Reduce Integration Test Scope to 2-3 recipes per task * Add PR comment trigger workflow * Improve PR comment event trigger (#4307) * Add GitHub Actions comments for workflow run status in PR * Update permissions in PR comment trigger workflow (#4309) * Update PR comment trigger (#4312) Update PR comment trigger workflow to enhance permissions and improve PR description updates * Add workflow step to update PR description with run link and status (#4314) * Enhance PR comment trigger to include last commit message and workflow result (#4316) Enhance PR comment trigger to include last commit message and workflow run summary * Refactor PR comment trigger (#4318) Refactor PR comment trigger to include workflow status emoji and streamline run summary * Support linear classifier fine-tuning for classification models (#4298) * Backbone frozen in engine.py and head_only.yaml in multi_class_cls * update * Add : implemented linear fine-tuning for model layers * Updated frozen backbone settings/ * fixes from comments #4286 * Removed the coed comments #4298 * fixes from review #4298 * trailing space and removed head_only.yaml #4298 * Resolve the Dependency Inversion Problem#4298 * removed white space #4298 Update engine.py * removed white space #4298 * pre-commit * Added Unit test for frrze_backbone * update precommit * modified test_freeze_backbone in test_deit_tiny.py * modified test_freeze_backbone in test_deit_tiny.py * modified test_freeze_backbone in test_deit_tiny.py * run pre-commit --------- Co-authored-by: Vladislav Sovrasov <[email protected]> * 🧹 Refactor detection entity (#4322) * Remove detection entity --------- Signed-off-by: Ashwin Vaidya <[email protected]> * Update integration test commands to include run-category-only flag options (#4325) * Bump rich from 13.9.4 to 14.0.0 (#4324) Bumps [rich](https://github.com/Textualize/rich) from 13.9.4 to 14.0.0. - [Release notes](https://github.com/Textualize/rich/releases) - [Changelog](https://github.com/Textualize/rich/blob/master/CHANGELOG.md) - [Commits](Textualize/rich@v13.9.4...v14.0.0) --- updated-dependencies: - dependency-name: rich dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> --------- Signed-off-by: Ashwin Vaidya <[email protected]> Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Prokofiev Kirill <[email protected]> Co-authored-by: Eugene Liu <[email protected]> Co-authored-by: limgyuil <[email protected]> Co-authored-by: Vladislav Sovrasov <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Move engine to native Signed-off-by: Ashwin Vaidya <[email protected]> * Fix tests Signed-off-by: Ashwin Vaidya <[email protected]> * Add readme Signed-off-by: Ashwin Vaidya <[email protected]> * ⛙ Merge develop to feature (#4338) * Update Documentation (#4294) * update docs * walk through docs * minor * modify docs * fix readme * update readme. Delete outdated sections * 🧹 Minor cleanup (#4299) Refactor import statements to use 'otx.algo.utils.utils' for InstanceData across multiple files * Promote/Demote Inst Seg Algo (#4301) Update model categories in Mask R-CNN templates for balance and accuracy * Reduce Integration Test Scope (#4303) * Reduce Integration Test Scope to 2-3 recipes per task * Add PR comment trigger workflow * Improve PR comment event trigger (#4307) * Add GitHub Actions comments for workflow run status in PR * Update permissions in PR comment trigger workflow (#4309) * Update PR comment trigger (#4312) Update PR comment trigger workflow to enhance permissions and improve PR description updates * Add workflow step to update PR description with run link and status (#4314) * Enhance PR comment trigger to include last commit message and workflow result (#4316) Enhance PR comment trigger to include last commit message and workflow run summary * Refactor PR comment trigger (#4318) Refactor PR comment trigger to include workflow status emoji and streamline run summary * Support linear classifier fine-tuning for classification models (#4298) * Backbone frozen in engine.py and head_only.yaml in multi_class_cls * update * Add : implemented linear fine-tuning for model layers * Updated frozen backbone settings/ * fixes from comments #4286 * Removed the coed comments #4298 * fixes from review #4298 * trailing space and removed head_only.yaml #4298 * Resolve the Dependency Inversion Problem#4298 * removed white space #4298 Update engine.py * removed white space #4298 * pre-commit * Added Unit test for frrze_backbone * update precommit * modified test_freeze_backbone in test_deit_tiny.py * modified test_freeze_backbone in test_deit_tiny.py * modified test_freeze_backbone in test_deit_tiny.py * run pre-commit --------- Co-authored-by: Vladislav Sovrasov <[email protected]> * 🧹 Refactor detection entity (#4322) * Remove detection entity --------- Signed-off-by: Ashwin Vaidya <[email protected]> * Update integration test commands to include run-category-only flag options (#4325) * Bump rich from 13.9.4 to 14.0.0 (#4324) Bumps [rich](https://github.com/Textualize/rich) from 13.9.4 to 14.0.0. - [Release notes](https://github.com/Textualize/rich/releases) - [Changelog](https://github.com/Textualize/rich/blob/master/CHANGELOG.md) - [Commits](Textualize/rich@v13.9.4...v14.0.0) --- updated-dependencies: - dependency-name: rich dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Fix integration test commands run-category-only flag (#4329) * updated repo with mobilenet_v4 (#4321) * updated repo with mobilenet_v4 * Introduce MobileNetV4 for Improved Performance * Introduced MobileNetV4 recipe for Improved Performance --------- Co-authored-by: Vladislav Sovrasov <[email protected]> * Update documentation and references to Open-Edge Platform (#4331) * Update documentation and references to Open-Edge Platform- Updated dataset format links in auto_configuration.rst, models_optimization.rst, and various algorithm documentation to point to the Open-Edge Platform repository. - Changed references from openvinotoolkit to open-edge-platform in multiple files including anomaly detection, classification, object detection, segmentation, and product design documentation. - Modified installation instructions and CLI command references to reflect the new repository URL. - Adjusted the help formatter to display the updated GitHub repository link. - Ensured all links to Datumaro and related resources are consistent with the new repository structure. * 🧹 Refactor Instance Segmentation Entity (#4327) Signed-off-by: Ashwin Vaidya <[email protected]> * 🧹 Refactor Anomaly Entities (#4328) * Remove anomaly entities Signed-off-by: Ashwin Vaidya <[email protected]> --------- Signed-off-by: Ashwin Vaidya <[email protected]> * Add a contribution guide (#4335) * Add a contribution guide * Fix a typo * Apply grammar suggestions Co-authored-by: Rajesh Gangireddy <[email protected]> --------- Co-authored-by: Rajesh Gangireddy <[email protected]> * Bug Fix : Convert labels for multi-label Classification into LongTensor before one_hot encoding (#4336) * Update classification.py * make ruff happy * make it simpler * Update src/otx/core/data/dataset/classification.py Co-authored-by: Daan Krol <[email protected]> --------- Co-authored-by: Daan Krol <[email protected]> * Bump OV and NNCF to 2025.1 (#4334) * Bump OV to 2025.1 * Bump NNCF * Update changelog * Update imports --------- Signed-off-by: Ashwin Vaidya <[email protected]> Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Prokofiev Kirill <[email protected]> Co-authored-by: Eugene Liu <[email protected]> Co-authored-by: limgyuil <[email protected]> Co-authored-by: Vladislav Sovrasov <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Gaurav Singh <[email protected]> Co-authored-by: Rajesh Gangireddy <[email protected]> Co-authored-by: Daan Krol <[email protected]> * commit changes Signed-off-by: Ashwin Vaidya <[email protected]> * Fix tests Signed-off-by: Ashwin Vaidya <[email protected]> * Restructure folder Signed-off-by: Ashwin Vaidya <[email protected]> * update imports Signed-off-by: Ashwin Vaidya <[email protected]> * Delete mobilenetv4 Signed-off-by: Ashwin Vaidya <[email protected]> * [feature-v3] Add OV Engine (#4374) * add OV engine as a backend * update engine * move backend * test works * add predict, delete OVModels * fix optimize * add unit test for OVEngine * fix ruff * fix mypy issues * update changes from feature branch * some fixes * added API tests with ov_engine * reply to comments, update docstrings * inherit OVEngine from Engine * fix API tests * fix tests * fix rest_engine * try to delete test to check: * [feature-v3] Move rest of OTX under native backend. Change structure of the folders. (#4408) * move all folders under native * add missing files * delete for developers * add annotations to types * update __init__ * align AnomalyDataset with OTXDataset * move cspnext to detection to avoid circular error * fix linter * move all unit test * fix linter * return __init__ to openvino backend tests * reply comments * fix unit tests. Fix class patch * change copyright * fix recipe path * fix inst seg. Move samplers * minor fix from copilot * [feature-v3] Remove T_* data entities (#4412) * remove T_* entities * fix linter * fix API test * resolve conflicts * fix typing * remove all ignores * minor fix * fix duplication * [feature-v3] Integrate "create_engine" to tests and polish engine interface (#4414) * align engines * engine.data instead of engine.datamodule * add missing files * fix all issues with get_engine() * add missing files * fix linter * delete task from Engine * simplify the logic of auto-configurator * decrease bum of epochs * remove task from all configs * fix converter test * fix linter * small fix to unit test * fix integration tests and delete unnecessary auto configuration test * add yaml to configs * small fix for linter * fix unit tests * fix integration * fix bandit * fix anomaly tests| --------- Signed-off-by: Ashwin Vaidya <[email protected]> Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Prokofiev Kirill <[email protected]> Co-authored-by: Eugene Liu <[email protected]> Co-authored-by: limgyuil <[email protected]> Co-authored-by: Vladislav Sovrasov <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Gaurav Singh <[email protected]> Co-authored-by: Rajesh Gangireddy <[email protected]> Co-authored-by: Daan Krol <[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.
…w run summary
Summary
Enhance PR comment trigger to include last commit message and workflow result
How to test
Checklist
License
Feel free to contact the maintainers if that's a concern.