-
Notifications
You must be signed in to change notification settings - Fork 866
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added undistort function #520
Conversation
Hi @dlion thank you for your contribution. A couple of comments.
Thanks! |
@deadprogram Ops, I missed that, fixed thanks! :) |
Codecov Report
@@ Coverage Diff @@
## dev #520 +/- ##
==========================================
+ Coverage 98.07% 98.07% +<.01%
==========================================
Files 17 17
Lines 2077 2079 +2
==========================================
+ Hits 2037 2039 +2
Misses 33 33
Partials 7 7
Continue to review full report at Codecov.
|
Sorry to be a pest @dlion can you please rebase against dev and also remove the commented out line I mentioned? Thank you. |
@deadprogram now should be fixed :) |
Seems like no change? |
@deadprogram I just force-pushed in my own branch, as you can see there aren't conflicts :) |
OK, merging. Thanks again @dlion for another contribution! |
No description provided.