-
Notifications
You must be signed in to change notification settings - Fork 12
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
Fix marker duplication #59
Conversation
7d1d85e
to
60e9ed8
Compare
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
- Also clean up python specifiers Signed-off-by: Dan Ryan <dan@danryan.co>
- Also add magic methods to MetaSet object - Super fast and efficient now due to set comparisons - Old method had sometimes thousands of the same pyspecs - This consolidates as it builds Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co> Add set operations and comparison methods to metasets Signed-off-by: Dan Ryan <dan@danryan.co> Add abstract extraction methods for marker cleanup Signed-off-by: Dan Ryan <dan@danryan.co> Add intersection method for PySpecs and smarter unions and creation Signed-off-by: Dan Ryan <dan@danryan.co> Simplify and cleanup metadata implementation Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
60e9ed8
to
4d72b7a
Compare
.gitignore
Outdated
# mypy | ||
.mypy_cache/ | ||
.dmypy.json | ||
dmypy.json |
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.
Why does every PR has this. Maybe commit this first and base all PRs after it?
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
Signed-off-by: Dan Ryan <dan@danryan.co>
Codecov Report
@@ Coverage Diff @@
## add-tests-frost #59 +/- ##
===================================================
- Coverage 56.35% 54.94% -1.42%
===================================================
Files 42 42
Lines 2335 2701 +366
Branches 388 475 +87
===================================================
+ Hits 1316 1484 +168
- Misses 915 1102 +187
- Partials 104 115 +11
Continue to review full report at Codecov.
|
#55 (4/4) deduplicate and merge markers using set logic (probably needs some cleanup but it mostly gets the job done now)