mdanalysis icon indicating copy to clipboard operation
mdanalysis copied to clipboard

WIP, CI: 3.11rc2 check

Open tylerjereddy opened this issue 3 years ago • 6 comments
trafficstars

  • Python 3.11rc2 is guaranteed ABI stable with the final release per: https://www.python.org/downloads/release/python-3110rc2/

  • the final release is about two weeks away, so let's see what goes wrong with 3.11 a bit before it hits; most of the major dependencies have releasd PyPI wheels because of the above ABI stability guarantee

  • I believe the CI workflow I adjusted may use a mix of conda and pip, so I'm less confident of what will go wrong vs. pure PyPI workflow

tylerjereddy avatar Oct 10 '22 02:10 tylerjereddy

Setting up a pure pypi runner for this is going to take a bit more work I'm afraid (since we rely on mamba as our base installer for our normal actions CI). Might just be easier to put this in the azure pipelines that are pure pypi?

IAlibay avatar Oct 10 '22 02:10 IAlibay

Codecov Report

Base: 94.34% // Head: 94.22% // Decreases project coverage by -0.12% :warning:

Coverage data is based on head (baad2d0) compared to base (81ed5d0). Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3857      +/-   ##
===========================================
- Coverage    94.34%   94.22%   -0.13%     
===========================================
  Files          193      193              
  Lines        25007    25007              
  Branches      3373     3373              
===========================================
- Hits         23593    23562      -31     
- Misses        1365     1394      +29     
- Partials        49       51       +2     
Impacted Files Coverage Δ
package/MDAnalysis/due.py 46.66% <0.00%> (-30.00%) :arrow_down:
package/MDAnalysis/coordinates/chemfiles.py 92.30% <0.00%> (-4.95%) :arrow_down:
package/MDAnalysis/coordinates/H5MD.py 95.94% <0.00%> (-1.68%) :arrow_down:
package/MDAnalysis/converters/ParmEdParser.py 96.89% <0.00%> (-1.56%) :arrow_down:
package/MDAnalysis/converters/ParmEd.py 91.95% <0.00%> (-1.15%) :arrow_down:
package/MDAnalysis/analysis/hole2/utils.py 75.47% <0.00%> (-0.63%) :arrow_down:
package/MDAnalysis/analysis/hole2/hole.py 82.63% <0.00%> (-0.27%) :arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

codecov[bot] avatar Oct 10 '22 02:10 codecov[bot]

@tylerjereddy do you mind if I jump in here and test this out?

IAlibay avatar Oct 10 '22 21:10 IAlibay

@IAlibay yeah go for it, may tell you a bit about some useful changes that can be made soon if needed

tylerjereddy avatar Oct 10 '22 22:10 tylerjereddy

Just testing out this azp thing here now, then I'll probably open a separate PR to not spam too much. Easiest thing imho will be to just add a runner in cron CI that picks up the latest unstable version from https://github.com/actions/python-versions/blob/main/versions-manifest.json

IAlibay avatar Oct 10 '22 23:10 IAlibay

Ok MDA wheel build fine, but the presence of h5py in the azp script is messing with us. I'll switch to a new PR at this point to keep the commit noise down.

IAlibay avatar Oct 10 '22 23:10 IAlibay

Closing as superseded by #3862

IAlibay avatar Oct 26 '22 12:10 IAlibay