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

ci: add python 3.10, gcc-10 option #1795

Merged
merged 4 commits into from
Nov 10, 2021
Merged

ci: add python 3.10, gcc-10 option #1795

merged 4 commits into from
Nov 10, 2021

Conversation

georgebisbas
Copy link
Contributor

Tested python 3.10 via docker seems ok, just spinning a python 3.10 CI

@codecov
Copy link

codecov bot commented Nov 8, 2021

Codecov Report

Merging #1795 (f598b2f) into master (bd5eaac) will decrease coverage by 2.17%.
The diff coverage is n/a.

❗ Current head f598b2f differs from pull request most recent head ed7ac88. Consider uploading reports for the commit ed7ac88 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1795      +/-   ##
==========================================
- Coverage   89.55%   87.37%   -2.18%     
==========================================
  Files         209      204       -5     
  Lines       34062    33584     -478     
  Branches     4418     4413       -5     
==========================================
- Hits        30504    29345    -1159     
- Misses       3062     3729     +667     
- Partials      496      510      +14     
Impacted Files Coverage Δ
devito/arch/compiler.py 47.45% <ø> (-9.42%) ⬇️
tests/test_gpu_openmp.py 4.51% <0.00%> (-93.79%) ⬇️
tests/test_adjoint.py 26.66% <0.00%> (-73.34%) ⬇️
examples/seismic/tti/operators.py 53.94% <0.00%> (-43.16%) ⬇️
examples/seismic/self_adjoint/operators.py 58.46% <0.00%> (-41.54%) ⬇️
examples/seismic/viscoacoustic/operators.py 59.49% <0.00%> (-40.51%) ⬇️
examples/seismic/tti/wavesolver.py 68.33% <0.00%> (-30.00%) ⬇️
examples/seismic/elastic/elastic_example.py 48.57% <0.00%> (-28.58%) ⬇️
...mples/seismic/viscoelastic/viscoelastic_example.py 48.57% <0.00%> (-28.58%) ⬇️
examples/seismic/self_adjoint/wavesolver.py 75.43% <0.00%> (-24.57%) ⬇️
... and 55 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bd5eaac...ed7ac88. Read the comment docs.

@mloubout
Copy link
Contributor

mloubout commented Nov 8, 2021

scipy doesn't support 3.10 and numpy only in some case so until these move towards it I wouldn't even look at it.

@georgebisbas
Copy link
Contributor Author

makes sense

@mloubout
Copy link
Contributor

mloubout commented Nov 8, 2021

Looks liek scipy got a new release 3 days ago that support 3.10

@mloubout mloubout reopened this Nov 8, 2021
- name: pytest-docker-py36-gcc-omp
python-version: 3.6
os: ubuntu-18.04
- name: pytest-docker-py310-gcc-omp
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wouldn't change the docker build we wanna keep a safe standard version for that one.

@mloubout mloubout added dependencies Pull requests that update a dependency file testing labels Nov 8, 2021
@georgebisbas georgebisbas changed the title ci: test python 3.10 ci: add python 3.10, gcc-10 option Nov 9, 2021
]
set: [base, adjoint]
include:
- name: pytest-ubuntu-py37-gcc5-omp
python-version: 3.7
python-version: '3.7'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why do we need the '...' now?

os: ubuntu-20.04
arch: "gcc-9"
language: "openmp"
sympy: 1.8

- name: pytest-ubuntu-py310-gcc10-omp
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

adding a new one, but not dropping an old one? is this PR replacing a job, or adding a new one?

Copy link
Contributor Author

@georgebisbas georgebisbas Nov 10, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We discused in the meeting yesterday to maybe replace a 3.7 job, ' ' are needed from now on for github actions.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@FabioLuporini
Copy link
Contributor

why is the docker build also changing?

@georgebisbas
Copy link
Contributor Author

why is the docker build also changing?

It had wrong name before... now it is fixed. It is using 3.7

Copy link
Contributor

@FabioLuporini FabioLuporini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LTGM

@mloubout mloubout force-pushed the python3.10 branch 2 times, most recently from 81b921e to 3d00e0b Compare November 10, 2021 18:10
@mloubout mloubout merged commit 6159e32 into master Nov 10, 2021
@mloubout mloubout deleted the python3.10 branch November 10, 2021 19:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants