Skip to content
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

Consider using division operator from future module #1507

Closed
talumbau opened this issue Aug 12, 2017 · 2 comments
Closed

Consider using division operator from future module #1507

talumbau opened this issue Aug 12, 2017 · 2 comments
Labels

Comments

@talumbau
Copy link
Member

Regarding the small differences between Python 2 and Python 3, the only thing that occurs to me is that there may be one or two operations occurring somewhere where the / operator is doing truncating integer division for Python 2 and float division for Python 3. It would be an interesting experiment to put from __future__ import division in all source files and see if the Py2 vs Py3 discrepancy is resolved. Making a note here to try myself, but whoever tries first should just report back results and if it resolves the findings in #1506

@martinholmer
Copy link
Collaborator

One experiment along the lines suggest in issue #1507 was conducted as part of the work involved in preparing merged pull request #1508. Adding the from __future__ import division statement to several files did not change any test_cpscsv.py results under Python 2.7.

@martinholmer
Copy link
Collaborator

@talumbau found the source of the Python 2.7 versus Python 3.x differences described in #1506.
The results of his investigation are in issue #1522.
So, the now obsolete issue #1507 is now being closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants