glue-viz / glue-astronomy

Plugin to add astronomy-specific functionality to glue
https://glue-astronomy.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
11 stars 13 forks source link

Switch CI to GitHub Actions #69

Closed dhomeier closed 2 years ago

dhomeier commented 2 years ago

Description

Experimental transition from Azure pipelines using OpenAstronomy workflows

codecov[bot] commented 2 years ago

Codecov Report

Merging #69 (e54c590) into main (fda396d) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #69   +/-   ##
=======================================
  Coverage   96.95%   96.95%           
=======================================
  Files          15       15           
  Lines        1182     1182           
=======================================
  Hits         1146     1146           
  Misses         36       36           

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 fda396d...e54c590. Read the comment docs.

dhomeier commented 2 years ago

Switching pyversion for Windows for now, as py37 fails to load on

INTERNALERROR>   File "D:\a\glue-astronomy\glue-astronomy\.tox\py37-test\lib\site-packages\numpy\ctypeslib.py", line 154, in load_library
INTERNALERROR>     raise OSError("no file with expected extension")
INTERNALERROR> OSError: no file with expected extension

(though it's using the same numpy version as Azure?).

dhomeier commented 2 years ago

Not sure if the experimental aarch64 is running natively or in Rosetta; it reports

pythonLocation: /Users/runner/hostedtoolcache/Python/3.10.3/x64

whether platform: [a64] is specified or not.

dhomeier commented 2 years ago

@astrofrog this is probably the most uncontroversial of the 4 repos in migration (at least it was as long as Actions were working...)

dhomeier commented 2 years ago

OK, thanks – in glue-viz/glue-jupyter#300 I have given the publishing its own job so it would run independently; gonna change that.