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

Convert to Python 3 due to Python 2 Sunset #818

Open
jgandt opened this issue Jul 19, 2020 · 9 comments · May be fixed by #820
Open

Convert to Python 3 due to Python 2 Sunset #818

jgandt opened this issue Jul 19, 2020 · 9 comments · May be fixed by #820

Comments

@jgandt
Copy link

jgandt commented Jul 19, 2020

Given the decision and these comments were made three years ago, I would like to propose an initiative to migrate this codebase from Python 2 to Python 3.

Python 2 is officially sunset and no longer maintained. This alone should light a fire under our butts to begin conversion.

I realize this is a large effort and potentially disruptive, but I believe it is the correct path moving forward. If we remain in Python 2, the project is more likely to languish. In addition, we would be better Python citizens and denizens if we upgrade to 3.

The original reason for remaining on Python 2 was that Python 3's binary compilation support was lacking. I believe this situation has drastically changed since the decision was made.

@davidelang
Copy link
Contributor

davidelang commented Jul 20, 2020 via email

@BarbourSmith
Copy link
Member

I would support a migration to python 3 if someone is willing to take it on. As far as I know the only change should be to switch a couple print "string" to print("string"). The bigger issue will be making sure all the dependencies in the build process are working again which is the real issue. I would release a version pointing people to web control if it were easy to build, but the build process needs a lot of love.

@mdelmans
Copy link

mdelmans commented Nov 1, 2020

@davidelang I think you should put this message in bold in README of this repository. It took me several days to set GroundControl on Raspberry Pi, and now only through this issue I'm learning there is a WebControl that is not a part of MaslowCNC and has all attention.

@BarbourSmith
Copy link
Member

Thanks for the suggestion, I've added a note to the top of the readme letting folks know that Web Control is worth checking out.

@mdelmans
Copy link

mdelmans commented Nov 4, 2020

I've looked at WebControl and I don't like it. I want to control the machine from a Raspberry + Raspberry screen and I don't like the feel of WebControl running through the browser (even in fullscreen mode). Furthermore, all this complication with split Python/js code, which I think is unnecessary.

I think GroundControl is far from ideal but it is more suited for the Raspberry setup. I have translated GroundControl to Python 3, added poetry support and turned it into a proper python package. Works on Mac, haven't tested on Raspberry yet.

Worth sharing ?

@BarbourSmith
Copy link
Member

Absolutely!

Have yous seen our forums? I think people would be very impressed :)

forums.maslowcnc.com/

@davidelang
Copy link
Contributor

davidelang commented Nov 4, 2020 via email

@mdelmans mdelmans linked a pull request Nov 8, 2020 that will close this issue
6 tasks
@mdelmans
Copy link

mdelmans commented Nov 8, 2020

See the merge request, let me know what do you think.

@davidelang not sure I can afford to be a maintainer but I am happy to share all the work I'm doing on it.

@BarbourSmith
Copy link
Member

This is looking fantastic!

You are doing an awesome job and porting to Python3 is obviously the thing to do now.

My only reservation would be that I don't currently have a machine set up with the right configuration to test this. I think that is OK since if we aren't planning to do a release the master branch can support python 3 while the last release where folks can download the ready to install versions for Windows and OS X will point to the thoroughly tested version.

Does anyone have any reservations? If not I think this is fantastic.

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 a pull request may close this issue.

4 participants