-
Notifications
You must be signed in to change notification settings - Fork 4
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
Random updates #152
base: main
Are you sure you want to change the base?
Random updates #152
Conversation
Can we integrate this into #122 ? |
Works for me. Just for context, what's here is a side-effect of me working on some commercial code. As we go, I'm finding pieces of it that aren't core IP and feeding them back upstream to MeasureBase and MeasureTheory. Moving any of the MB-upstreamed stuff into #122 is fine, as long as it doesn't hold up the company work. Also, there are some other things that aren't here yet that could go into MB. Let's chat about those when you're available and we can make a plan. |
Oh, ok - then let's get it in directly and I'll rebase #122. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #152 +/- ##
==========================================
- Coverage 56.32% 54.03% -2.29%
==========================================
Files 44 43 -1
Lines 1321 1327 +6
==========================================
- Hits 744 717 -27
- Misses 577 610 +33 ☔ View full report in Codecov by Sentry. |
Just a placeholder for now, some of these changes could be rolled back or moved to MeasureTheory