Skip to content

Switch to using TrueDivide instead of Divide everywhere - #813

Merged
filmor merged 1 commit into
masterfrom
use-truedivide-everywhere
Feb 25, 2019
Merged

filmor merged 1 commit into
masterfrom
use-truedivide-everywhere

Conversation

@filmor

@filmor filmor commented Feb 2, 2019

Copy link
Copy Markdown
Member

Fixes #802.

TrueDivide is supported in both Python 2 and 3 while Divide exists only in Python 2. This does mean that a simple division of numbers will behave differently in C# vs Python for Python 2, but it will at least
behave the same way in C# independent of the Python version being used.

Fixes #802.

TrueDivide is supported in both Python 2 and 3 while Divide exists only
in Python 2. This does mean that a simple division of numbers will
behave differently in C# vs Python for Python 2, but it will at least
behave the same way in C# independent of the Python version being used.
@filmor filmor added this to the 2.4.0 milestone Feb 5, 2019
@filmor

filmor commented Feb 25, 2019

Copy link
Copy Markdown
Member Author

The failing tests are completely unrelated, I don't have the time right now to fix the Travis build and it ran through on AppVeyor, so I'm going to merge this now.

@filmor
filmor merged commit 7bc2495 into master Feb 25, 2019
@filmor
filmor deleted the use-truedivide-everywhere branch March 6, 2019 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant