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

[FIX] Arrange scale between bbox preds and bbox targets in ATSS #1880

Merged
merged 1 commit into from
Mar 14, 2023

Conversation

jaegukhyun
Copy link
Contributor

Cause
Bbox encode or decode status is not matched between bbox preds and bbox targets in ATSS

Solution
Check self.reg_decoded_bbox value. If self.reg_decoded_bbox is True, then ATSS has to decode bbox_pred values since bbox_target values are not encoded

@jaegukhyun jaegukhyun requested a review from a team as a code owner March 14, 2023 06:03
Copy link
Contributor

@eunwoosh eunwoosh left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for your work :)

Copy link
Contributor

@sungmanc sungmanc left a comment

Choose a reason for hiding this comment

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

Could you follow the PR template?
e.g. openvinotoolkit/datumaro#854

@sungmanc sungmanc added this to the 1.1.0 milestone Mar 14, 2023
@eunwoosh
Copy link
Contributor

Isn't same bugfix needed to release branch?

Copy link
Contributor

@goodsong81 goodsong81 left a comment

Choose a reason for hiding this comment

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

I've left a comment. Please let me know your opinion.

@goodsong81 goodsong81 merged commit b76ee9f into develop Mar 14, 2023
@goodsong81 goodsong81 deleted the jg/atss-fix branch March 14, 2023 07:31
goodsong81 pushed a commit that referenced this pull request Mar 14, 2023
Arrange scale between bbox preds and bbox targets
goodsong81 added a commit that referenced this pull request Mar 14, 2023
* Fix ote_config -> otx_config

* [FIX] hang issue when tracing a stack in certain scenario (#1868)
fix: use primitive library

* [FIX][POT] Set stat_requests_number parameter to 1 (#1870)
Set POT stat_requests_number parameter to 1 in order to lower RAM footprint

* [FIX] Training error when batch size is 1 (#1872)
fix: drop last batch

* Recover detection num_workers=2

* Remove nbmake from base requirements

* Add py.typed in package

* [FIX] Arrange scale between bbox preds and bbox targets in ATSS (#1880)
Arrange scale between bbox preds and bbox targets

---------

Co-authored-by: Inhyuk Cho <andy.inhyuk.jo@intel.com>
Co-authored-by: Nikita Savelyev <nikita.savelyev@intel.com>
Co-authored-by: Jaeguk Hyun <jaeguk.hyun@intel.com>
goodsong81 added a commit that referenced this pull request Mar 17, 2023
* [RELEASE][DOC] Fix wrong info in documentation (#1849)
* updated dataset formats info. Fix for multilabel classification
* revert file
* revert file. minor
* added warning to instance segmentation
* revert changes

* [Enhance] Separate installation for each tasks on release task (#1869)
* separte_import
* align with pre commit
* update unit test code
* add separate task env pre  merge test & aplly it to github action
* add multiprocess to requirement

* [FIX][REL1.0] Fix Geti integration issues (#1885)
* Fix ote_config -> otx_config

* [FIX] hang issue when tracing a stack in certain scenario (#1868)
fix: use primitive library

* [FIX][POT] Set stat_requests_number parameter to 1 (#1870)
Set POT stat_requests_number parameter to 1 in order to lower RAM footprint

* [FIX] Training error when batch size is 1 (#1872)
fix: drop last batch

* Recover detection num_workers=2
* Remove nbmake from base requirements
* Add py.typed in package

* [FIX] Arrange scale between bbox preds and bbox targets in ATSS (#1880)
Arrange scale between bbox preds and bbox targets

* [FIX][RELEASE1.0] Remove cfg dump in ckpt (#1895)
* Remove cfg dump in ckpt
* Fix pre-commit

* Release v1.0.1

* [FIX] Prevent torch 2.0.0 installation (#1896)

* Add torchvision & torchtext in requirements/anomaly.txt with fixed version

* Update requirements/anomaly.txt

* Fix _model_cfg -> _recipe_cfg due to cfg merge

---------

Signed-off-by: Songki Choi <songki.choi@intel.com>
Co-authored-by: Prokofiev Kirill <kirill.prokofiev@intel.com>
Co-authored-by: Eunwoo Shin <eunwoo.shin@intel.com>
Co-authored-by: Inhyuk Cho <andy.inhyuk.jo@intel.com>
Co-authored-by: Nikita Savelyev <nikita.savelyev@intel.com>
Co-authored-by: Jaeguk Hyun <jaeguk.hyun@intel.com>
Co-authored-by: Jihwan Eom <jihwan.eom@intel.com>
@eunwoosh eunwoosh mentioned this pull request Mar 19, 2023
7 tasks
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