Python 3 compatibility?

Question

Python 3 compatibility?

Closed this issue 7 years ago · 17 comments

ocgis is supporting Python 3.5+ in the next release. I naively tested against an icclim 4.2.5 version built with Python 3.6 and encountered issues. Does icclim have plans to support Python 3? Python 2.7 will still be supported in ocgis. I made an attempt to fix some of the basic syntax issues but received some errors deep in the calculation code which I couldn't parse easily.

Answer 1 · 2017-04-03T14:35:10.000Z

icclim will not be updated in the short term to support Python 3.5+, but I expect that starting in May I will have time to work on it.

Answer 2 · 2017-04-03T14:58:38.000Z

Thanks for the update @pagecp. I don't expect it will be too bad.

Answer 3 · 2017-08-25T10:21:23.000Z

There is a new python3 branch for icclim. It is not all fully-tested version yet, but it works for a few test files for now. I am working on this branch right now, so many git push are being done attm. Maybe you would like to test it.

…

On 01 Apr 2017, at 20:09, Ben Koziol ***@***.***> wrote: ocgis is supporting Python 3.5+ in the next release. I naively tested against an icclim 4.2.5 version built with Python 3.6 and encountered issues. Does icclim have plans to support Python 3? Python 2.7 will still be supported in ocgis. I made an attempt to fix some of the basic syntax issues but received some errors deep in the calculation code which I couldn't parse easily. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#26>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AG-dymzHTOaC2QH3LQhNKgV5_fBxhhmgks5rrpLegaJpZM4MwkR7>.

Answer 4 · 2017-08-25T16:50:22.000Z

Nice! I will give it a go. What is the preferred method for running tests? I have not added this to the Anaconda build we use. Just execute https://github.com/cerfacs-globc/icclim/blob/master/icclim/test-cases/icclim-test-wrapper.py? Thanks!

Answer 5 · 2017-08-28T07:26:39.000Z

This was the initial test script, but one collaborator implemented the Robot Framework, in another branch, which I need to finalise the tests and merge. ===== 1) checkout the branch "pybot-testsuite" 2) install the robot framework, e.g., "pip install robotframework" 3) if necessary, update the "test-cases/default_variables.txt"-file (or at least link the input folder name so it is available in the root folder) 4) execute "robot test-cases" and it should get going (note that if using any version prior to 3.0 of the framework the syntax is "pybot test-cases"). The "robot"-command recursively traverses the "test-cases/"-folder and executes all the test case files found, currently, * test-cases/test-cases-360-days/robot-360-days.txt * test-cases/test-cases-opendap/robot-opendap.txt * test-cases/test-cases-read-the-docs/robot-read-the-docs.txt As can be seen from these files the current test procedure is quite simple, simply execute and check the exit status, this could of course be changed if we want a more detailed criteria for test success, e.g., comparison of output files with earlier runs. Output from the running the tests is a summary to stdout + xml/html-files + text files with the test case "stdout_*", "stderr_*" ==== The icclim-test-wrapper.py can still be used, but the test data is not available anymore.

…

On 25 Aug 2017, at 18:50, Ben Koziol ***@***.***> wrote: Nice! I will give it a go. What is the preferred method for running tests? I have not added this to the Anaconda build we use. Just execute https://github.com/cerfacs-globc/icclim/blob/master/icclim/test-cases/icclim-test-wrapper.py <https://github.com/cerfacs-globc/icclim/blob/master/icclim/test-cases/icclim-test-wrapper.py>? Thanks! — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#26 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AG-dymTPzfnUoazodGxdfUTwKMOYSpFKks5sbvtPgaJpZM4MwkR7>.

Answer 6 · 2018-02-27T01:30:47.000Z

My sincerest apologies for the delay on getting back to you on this. I have a quick PR to address a couple Python 3 build issues: #32.

Answer 7 · 2018-03-07T15:57:02.000Z

Sorry for the delay, I was still on holidays. I merged the PR into the main branch, but I also included some changes in the netcdf3 branch.

…

On 27 Feb 2018, at 02:30, Ben Koziol ***@***.***> wrote: My sincerest apologies for the delay on getting back to you on this. I have a quick PR to address a couple Python 3 build issues: #32 <#32>. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#26 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AG-dyghP_aigcAsdu4Fv-PvVWG3q5Z8aks5tY1rHgaJpZM4MwkR7>.

Answer 8 · 2018-03-07T16:08:46.000Z

I think all issues are resolved now. Ben can you confirm?

Answer 9 · 2018-03-07T17:39:42.000Z

Sorry for the delay, I was still on holidays.

No worries. I'm just jealous. 😄

I think all issues are resolved now. Ben can you confirm?

Clean on my end: https://travis-ci.org/NCPP/ocgis/jobs/350427172

We'll be hosting ICCLIM Python 2.7, 3.5, and 3.6 builds in the NESII Anaconda channel for Linux and OSX: https://anaconda.org/nesii/icclim

There is also going to be an OCGIS release in the next few weeks, and I'd like to point to the most recent ICCLIM version. Is it easy to tag a bug fix release with the Python 3 fixes?

Answer 10 · 2018-03-08T07:38:26.000Z

Hi Ben!

Sorry for the delay, I was still on holidays. No worries. I'm just jealous. 😄

😊🙃

I think all issues are resolved now. Ben can you confirm? Clean on my end: https://travis-ci.org/NCPP/ocgis/jobs/350427172 <https://travis-ci.org/NCPP/ocgis/jobs/350427172>

Nice!

We'll be hosting ICCLIM Python 2.7, 3.5, and 3.6 builds in the NESII Anaconda channel for Linux and OSX: https://anaconda.org/nesii/icclim <https://anaconda.org/nesii/icclim>

Great! :) :)

There is also going to be an OCGIS release in the next few weeks, and I'd like to point to the most recent ICCLIM version. Is it easy to tag a bug fix release with the Python 3 fixes?

Are you using the master branch or the netcdf3 branch? I will tag the proper one. Christian

Answer 11 · 2018-03-09T15:49:31.000Z

Are you using the master branch or the netcdf3 branch? I will tag the proper one.

I've been using the master branch. Would you recommend a different branch?

Answer 12 · 2018-03-09T15:57:30.000Z

I had made all the changes for python 3.6.x+ in the netcdf3 branch, but if you can already use the master branch with python3 you can stay with the master branch.

…

On 9 Mar 2018, at 16:49, Ben Koziol ***@***.***> wrote: Are you using the master branch or the netcdf3 branch? I will tag the proper one. I've been using the master branch. Would you recommend a different branch? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#26 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AG-dyiNoqzIZh07BSy8c-PhESjO5RCZ6ks5tcqSMgaJpZM4MwkR7>.

Answer 13 · 2018-03-09T16:00:31.000Z

Got it. Either branch is fine with me, but I have been testing against master...

Answer 14 · 2018-03-09T16:01:26.000Z

ok let's go for the master branch, and I will tag it now

…

On 9 Mar 2018, at 17:00, Ben Koziol ***@***.***> wrote: Got it. Either branch is fine with me. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#26 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AG-dyhrDEQyYmbjBQ07WQcXC_d3QR4ndks5tcqcggaJpZM4MwkR7>.

Answer 15 · 2018-03-09T16:07:07.000Z

👍 I'll close this issue too. Thank you!

Answer 16 · 2018-04-24T12:07:49.000Z

Sorry guys I know this issue is closed but I didn't want to open a new issue. I was just wondering, is Python 3 supported in the latest version of ICCLIM? @pagecp

Answer 17 · 2018-05-03T14:01:14.000Z

Hi,
The branch netcdf3 is python 3 supported, but the goal is to merge it with the default branch asap.
Some users have reported problems with python 3 on both netcdf3 and default branches, which I was not able to reproduce yet.
Your experience could be valuable...
Christian