Skip to content

Remove dist calculation duplicate#3322

Open
novikov-alexander wants to merge 1 commit intoopen-mmlab:mainfrom
novikov-alexander:novikov-alexander-dist-duplicate
Open

Remove dist calculation duplicate#3322
novikov-alexander wants to merge 1 commit intoopen-mmlab:mainfrom
novikov-alexander:novikov-alexander-dist-duplicate

Conversation

@novikov-alexander
Copy link
Copy Markdown

@novikov-alexander novikov-alexander commented Jan 19, 2026

Motivation

Remove redundant calculation and make calculation safe

Modification

Removes redundant duplicate of dist calculation.
Adds safe check for division

Before PR:

  • I have read and followed the workflow indicated in the CONTRIBUTING.md to create this PR.
  • Pre-commit or linting tools indicated in CONTRIBUTING.md are used to fix the potential lint issues.
  • Bug fixes are covered by unit tests, the case that causes the bug should be added in the unit tests.
  • New functionalities are covered by complete unit tests. If not, please add more unit test to ensure the correctness.
  • The documentation has been modified accordingly, including docstring or example tutorials.

After PR:

  • If the modification has potential influence on downstream or other related projects, this PR should be tested with some of those projects, like MMDet or MMCls.
  • CLA has been signed and all committers have signed the CLA in this PR.

Removes redundant duplicate of dist calculation.
Adds safe check for division
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jan 19, 2026

CLA assistant check
All committers have signed the CLA.

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