radical-cybertools / radical.pilot

RADICAL-Pilot
http://radical-cybertools.github.io/radical-pilot/index.html
Other
54 stars 23 forks source link

move back to python3.7 #2959

Closed AymenFJA closed 1 year ago

AymenFJA commented 1 year ago

fix issue #2958

codecov[bot] commented 1 year ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 41.45%. Comparing base (cc56f41) to head (f7aa9cb). Report is 2623 commits behind head on devel.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## devel #2959 +/- ## ======================================= Coverage 41.45% 41.45% ======================================= Files 95 95 Lines 10481 10481 ======================================= Hits 4345 4345 Misses 6136 6136 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

eirrgang commented 1 year ago

Would 3.8.8 work?

E.g. module load gcc/9.2.0 openmpi/3.1.6 python/3.8.8

AymenFJA commented 1 year ago

Would 3.8.8 work?

E.g. module load gcc/9.2.0 openmpi/3.1.6 python/3.8.8

Is 3.8.8 a hard requirement for you?

eirrgang commented 1 year ago

Would 3.8.8 work? E.g. module load gcc/9.2.0 openmpi/3.1.6 python/3.8.8

Is 3.8.8 a hard requirement for you?

The more recent the version, the more projects will be able to use the same python version for RP and for task execution. 3.9+ would be best, but rivanna does not have a module for 3.9 or 3.10

shantenujha commented 1 year ago

Would it not be better to ask Rivanna admins to update to a more "acceptable" version?

AymenFJA commented 1 year ago

@shantenujha @eirrgang, we have a ticket with them already (RIV-11250) regarding that issue. In the meantime, we switched to python3.7 as it serves the Cylon team perfectly. Once python3.11 is fixed we will move back to that version.