Closed vmoens closed 2 years ago
@SepehrAkhavan has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
@vmoens has updated the pull request. You must reimport the pull request before landing.
@vmoens has updated the pull request. You must reimport the pull request before landing.
@vmoens has updated the pull request. You must reimport the pull request before landing.
@vmoens has updated the pull request. You must reimport the pull request before landing.
Merging #91 (f031947) into main (4992731) will increase coverage by
0.14%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## main #91 +/- ##
==========================================
+ Coverage 98.25% 98.39% +0.14%
==========================================
Files 6 6
Lines 229 249 +20
==========================================
+ Hits 225 245 +20
Misses 4 4
Flag | Coverage Δ | |
---|---|---|
unittests | 98.39% <100.00%> (+0.14%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
Impacted Files | Coverage Δ | |
---|---|---|
tests/test_bijector.py | 100.00% <100.00%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 4992731...f031947. Read the comment docs.
@stefanwebb has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
LGTM! Thanks for taking care of this
Adds a new InverseBijector layer class. Instances can be created by calling
bij.invert()
orInverseBijector(bij)
. We make sure thatbij.invert().invert() is bij
returnsTrue
. Suggested by #90