Skip to content

Fix bug in VIFp#210

Merged
snk4tr merged 5 commits intomasterfrom
bug/vifp
Jan 20, 2021
Merged

Fix bug in VIFp#210
snk4tr merged 5 commits intomasterfrom
bug/vifp

Conversation

@jzakirov
Copy link
Collaborator

@jzakirov jzakirov commented Jan 20, 2021

Closes #208

Proposed Changes

  • Add conversion to [0, 255] range for input tensors
  • Refactor code to remove prediction and target names.
  • Add tests

After fix I got
PIQ VIFp SRCC 0.6102, KRCC 0.4579
Paper values are SRCC 0.610 KRCC 0.457
Exact match!

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@codecov
Copy link

codecov bot commented Jan 20, 2021

Codecov Report

Merging #210 (98fa0bd) into master (25b6efd) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #210      +/-   ##
==========================================
- Coverage   95.35%   95.33%   -0.02%     
==========================================
  Files          26       26              
  Lines        1785     1780       -5     
==========================================
- Hits         1702     1697       -5     
  Misses         83       83              
Flag Coverage Δ
unittests 95.33% <100.00%> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
piq/vif.py 100.00% <100.00%> (ø)

Copy link
Contributor

@snk4tr snk4tr left a comment

Choose a reason for hiding this comment

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

Nice one, @zakajd! Feel free to merge.
However, I noticed that you changed the version of the package. Please make sure to add new tag and release notes. I can do that as well if you lack time.

@jzakirov
Copy link
Collaborator Author

@snk4tr Yes, please create tag and release notes.
I'm currently working on README re-design.

@snk4tr snk4tr merged commit 60a7b12 into master Jan 20, 2021
@snk4tr snk4tr deleted the bug/vifp branch January 20, 2021 16:57
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.

Bug in VIFp implementation

2 participants