-
Notifications
You must be signed in to change notification settings - Fork 79
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
lint graph/_*.py
#618
lint graph/_*.py
#618
Conversation
include those.
Whatever, this is fully private. Happy to stick to strict CamelCase if it lives in the private module anyway |
Codecov Report
@@ Coverage Diff @@
## main #618 +/- ##
=====================================
Coverage 84.4% 84.4%
=====================================
Files 139 139
Lines 14867 14865 -2
=====================================
+ Hits 12541 12543 +2
+ Misses 2326 2322 -4
|
swap it james. I dont like ignoring pep8 anyway. i'll be psyched once we finally get rid of those underscores in esda |
Co-authored-by: Martin Fleischmann <[email protected]>
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.
Two more notes, sorry
xref #589
This PR:
libpysal/graph/_*.py
_set_ops._Set_Mixin
to_set_ops.SetOpsMixin