-
Notifications
You must be signed in to change notification settings - Fork 176
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
Misc fixes #679
Misc fixes #679
Conversation
Closes mpmath#267
Closes mpmath#274
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.
This looks good to me!
The build
packaging dependency probably should've been a different PR: I've never used that before, but I looked it up and it seems reasonable to me.
https://pypi.org/project/build/ <-- "A simple, correct Python build frontend."
return (self.__rows == other.__rows and self.__cols == other.__cols | ||
and self.__data == other.__data) | ||
except AttributeError: | ||
return NotImplemented |
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.
I'd not seen this before, but I skimmed https://stackoverflow.com/questions/40780004/returning-notimplemented-from-eq
I also tested this, and confirm that the A9 == None
test below.
Closes mpmath#428
LGTM! |
mpf('.00')
like float constructor does, closes from_str: wrong parsing with underscores after decimal #377