Megvii-BaseDetection / Megvii-BaseDetection/DisAlign
Undocumented multiplication by num_classes
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 118
- Forks
- 10
- PR merge metrics
- No merged PRs in 30d
Description
Hi,
thank you first of all for the great paper and for publishing the code.
I was wondering why you are multiplying the class_weight by num_classes here:
I cannot find this multiplication in the equation of the paper:
Maybe it is a misunderstanding on my end. Thank you in advance!
Patrick
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start at semantic_seg/disalign/models/losses/grw_cross_entropy_loss.py line 35 and compare the class_weight calculation with the equation in the cited paper. Determine whether the multiplication by num_classes is intentional, then document the rationale near the implementation or in the relevant project documentation so the code and paper are unambiguous.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- machine-learning
- Issue type
- Documentation
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100