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

Utilisation de micromamba pour installer et tester le package #102

Merged
merged 119 commits into from
May 2, 2024

Conversation

FlxPo
Copy link
Contributor

@FlxPo FlxPo commented Mar 19, 2024

La version actuelle de l'action github utilise anaconda et ne fonctionne pas. Je propose d'essayer d'utiliser micromamba pour corriger le bug et alléger l'action.

@FlxPo
Copy link
Contributor Author

FlxPo commented Mar 22, 2024

@Mind-the-Cap j'ai testé pas mal de choses pour arriver à faire tourner les tests. Pour le moment il y a encore quelques échecs qui semblent dus à la gestion de paths de certains fichiers. Mais sinon le package s'installe bien ! Et tous les tests passent en local chez moi (en 10 min environ...).

@FlxPo
Copy link
Contributor Author

FlxPo commented Apr 9, 2024

Nouveaux commits, pour repartir sur une nouvelle base de tests plus adaptés aux évolution des fonctionnalités du package. Les tests passent en 14 minutes, en local (avec un cache de données d'entrée complètement vide). Le taux de couverture est de 88 %.

FlxPo and others added 26 commits May 2, 2024 15:11
… them to match the transport zones boundary, extract only relevant ways for a given mode, merge the cropped and filtered files
…tion (data folders, proxy url, ...) + integration of dodgr
…s scripts called from mobility.setup + distribute a forked version of the osmdata R package as a binary to speed up OSM data parsing + switch to pyproject.toml
…icTravelCosts (all public transport modes) + create a GTFS class to handle download / aggregation / GTFS router object preparation
@FlxPo FlxPo force-pushed the gh-action-install-with-micromamba branch from 549be92 to a038319 Compare May 2, 2024 13:12
@FlxPo FlxPo merged commit a91d48e into main May 2, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants