geopandas / pyogrio

Vectorized vector I/O using OGR
https://pyogrio.readthedocs.io
MIT License
272 stars 22 forks source link

CI: use setup-micromamba action #329

Closed martinfleis closed 9 months ago

martinfleis commented 9 months ago

provision-with-micromamba has been deprecated for some time. setup-micromamba is the new action to be used.

martinfleis commented 9 months ago

I got no clue why Python 3.10 does not work when loaded from matrix but does with the one include run...

theroggy commented 9 months ago

I don't remember anymore why exactly, but you need to use >- instead of | with create-args:

          create-args: >-
            python=${{ matrix.python }}
            ${{ matrix.extra }}

instead of

          create-args: |
            python=${{ matrix.python }}
            ${{ matrix.extra }}
martinfleis commented 9 months ago

@theroggy nope, those seem to be equal. The command is built correctly from create-args, it just that mamba does not find 3.10...

theroggy commented 9 months ago

@martinfleis I think python==${{ matrix.python }} should be python=${{ matrix.python }} , so only one =?

martinfleis commented 9 months ago

It required consistent use of >- everywhere in the yaml.

Now we have a segfault coming from reading GPKG with mask on macOS under Python 3.12. And that is something I will happily leave to someone else to look at as I know near nothing on how to fix that one.

martinfleis commented 9 months ago

Solved by re-running the env. All green!

jorisvandenbossche commented 9 months ago

And FWIW, for other people wondering what's the difference between | or >- -> https://yaml-multiline.info/ (essentially we needed a single line command here, instead of executing as multiple lines)