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

Fix comparison methods for python3 #1537

Conversation

nils-van-zuijlen
Copy link
Member

See https://portingguide.readthedocs.io/en/latest/comparisons.html#rich-comparisons for an explaination of why it was broken.

In particular, I was stuck with a bug in the _Event class because heaps are ordered and I wanted to add multiple events to it, which failed because they had no suitable comparison functions.

@peternewman
Copy link
Member

Thanks @nils-van-zuijlen . Initially it might be worth a quick read of #1514 and some of the discussion linked to from #1506 .

Do you fancy targetting our 0.10 branch?

I'd started trying to get Python 3 builds working in Travis, if you fancy continuing that as it still needs some work (working on the principal we should probably have some tests to catch these failures in future)...
https://github.com/OpenLightingProject/ola/compare/master...peternewman:0.10-python3?expand=1

@nils-van-zuijlen nils-van-zuijlen changed the base branch from master to 0.10 February 15, 2019 17:49
@nils-van-zuijlen nils-van-zuijlen changed the base branch from 0.10 to master February 15, 2019 17:50
functools.total_ordering is only available from python 3.2
fix flake8 formatting
@peternewman
Copy link
Member

All Mac builds are failing for some reason outside your control. 😞

@peternewman peternewman changed the title Fix comparaison methods for python3 Fix comparison methods for python3 Feb 15, 2019
@nils-van-zuijlen
Copy link
Member Author

Yes, I saw it. Apparently GitHub syncronisation failed 😢

@peternewman
Copy link
Member

Yes, I saw it. Apparently GitHub syncronisation failed

Looks like it's progressing third time around...

@nils-van-zuijlen
Copy link
Member Author

Now it works !

@nils-van-zuijlen
Copy link
Member Author

I rebased my commits for 0.10 in #1538

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.

2 participants