MLM bugfixes#1556
Conversation
|
I continued working on my (synced) fork, so this PR also lists commits that have already been merged a few weeks ago... Sorry for the mess... |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1556 +/- ##
==========================================
+ Coverage 92.38% 92.51% +0.13%
==========================================
Files 55 55
Lines 8374 8575 +201
Branches 965 1037 +72
==========================================
+ Hits 7736 7933 +197
- Misses 453 454 +1
- Partials 185 188 +3 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
gadomski
left a comment
There was a problem hiding this comment.
Thanks for the contribution and fixes! I can review the code, but I don't have much visibility into how the MLM extension works. Could you possibly suggest two or three folks that might be able to review this PR from for content and usability?
|
It might also make sense to cherry-pick the new commits onto a fresh branch off origin:main so that the diff is easier to parse. |
|
continuing this in #1571 |
**Description:
Fixed some bugs in mlm extension (see changelog for details)
PR Checklist:
pre-commit run --all-files)pytest)