-
Notifications
You must be signed in to change notification settings - Fork 54
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
Towards integration on manifolds #628
Conversation
Codecov Report
@@ Coverage Diff @@
## master #628 +/- ##
========================================
Coverage 99.20% 99.21%
========================================
Files 106 106
Lines 10215 10340 +125
========================================
+ Hits 10134 10259 +125
Misses 81 81
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here is a few comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here is two more remarks.
In general this is missing 4 lines of coverage, that would be nice to check.
I do not understand the theory here completely, but noticed that already quite a while ago I checked most files as “viewed” - so I think past-me found them ok.
So besides test coverage, this is fine I think.
Co-authored-by: Ronny Bergmann <[email protected]>
I've started some work on integration on manifolds, mostly with exponential-wrapped distributions and kernel density estimation in mind. It is primarily inspired by http://arxiv.org/abs/2009.01983 . @sethaxen I think the
volume_density
function may be interesting for you 🙂 . It connects integration on a manifold with integration in normal coordinates.