deepinsight / deepinsight/insightface
Useless code in verification.py?
Open
- Dominant language
- Python
- Stars
- 29.7k
- Forks
- 6.1k
- PR merge metrics
- No merged PRs in 30d
Description
As `embeddings` is normed in line 269, why is it needed to norm it in line 265 at first?
https://github.com/deepinsight/insightface/blob/c3f2e2c70672a4656a76e667029a98000e56059d/recognition/arcface_torch/eval/verification.py#L264-L269
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.