-
Notifications
You must be signed in to change notification settings - Fork 5
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
Implement local version of vdim #82
base: main
Are you sure you want to change the base?
Conversation
c9180d7
to
d880f12
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #82 +/- ##
===========================================
- Coverage 79.55% 57.98% -21.58%
===========================================
Files 25 25
Lines 2950 3342 +392
===========================================
- Hits 2347 1938 -409
- Misses 603 1404 +801 ☔ View full report in Codecov by Sentry. |
…for better conditioning
7190b9e
to
8877785
Compare
… efficient Polynomial evaluation
Possible performance improvements left, taken from a quick look through profiling:
|
Feature (not performance) TODO before ready to merge:
|
This stabilizes VDIM in the regime when hk is bounded strictly away from zero.
Experimental branch/PR to play around and track progress on ideas related to a local version of VDIM, which is given in schematic form below.