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

[Crash} Corrige le chargement du chemin du fichier dans le package #192

Merged
merged 2 commits into from
Nov 27, 2023

Conversation

Allan-CodeWorks
Copy link
Contributor

Description

Nous avons introduit un bug dans la PR Corrige warning deprecated path() dans la réforme epci_test_factory #190.

En effet, le chemin utilisé pour charger le fichier des epcis n'est valable que dans se dépôt là, si on appelle la réforme dans un autre projet, le fichier n'est pas trouvé car le chemin est en dure.

@Allan-CodeWorks Allan-CodeWorks requested a review from a team November 23, 2023 14:52
@baptou12
Copy link
Contributor

@Allan-CodeWorks aie, tu t'en es rendu compte comment ? Faudrait comprendre pourquoi et corriger la non detection ?

@Shamzic
Copy link
Contributor

Shamzic commented Nov 24, 2023

@Allan-CodeWorks aie, tu t'en es rendu compte comment ? Faudrait comprendre pourquoi et corriger la non detection ?

En attendant la réponse d'Allan, de mon côté, je m'en suis rendu compte en utilisant localement la version d'openfisca-france-local.
Avec le simulateur par exemple, on obtient l'erreur suivante :

.../aides-jeunes/.venv/lib/python3.9/site-packages/pandas/io/common.py", line 865, in get_handle
    handle = open(handle, ioargs.mode)
FileNotFoundError: [Errno 2] No such file or directory: 'openfisca_france_local/epcicom2020.xlsx'

Je te rejoins sur la question : est-ce que c'est normal que la CI ne teste pas ce cas mais aussi est-ce que c'est techniquement possible actuellement ?

Copy link
Contributor

@Shamzic Shamzic left a comment

Choose a reason for hiding this comment

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

Corrige bien le problème, fonctionnel de mon côté

@Allan-CodeWorks
Copy link
Contributor Author

@Allan-CodeWorks aie, tu t'en es rendu compte comment ? Faudrait comprendre pourquoi et corriger la non detection ?

Je pensais avoir répondu :/
Effectivement c'est en testant le répo AJ que ça à crash...
Je ne sais pas trop comment tester ça, je suis d'accord que faire un test une bonne chose mais en attendant, cette version d'openfisca crash donc je propose de merge et je me fais un ticket pour ecrire le teste ultérieurement.
C'est ok pour toi @baptou12 ?

@Allan-CodeWorks Allan-CodeWorks merged commit cca352c into master Nov 27, 2023
9 checks passed
@Allan-CodeWorks Allan-CodeWorks deleted the Fix_epci_reform_path branch November 27, 2023 08:54
@guillett guillett added this to the Passé milestone Mar 22, 2024
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.

4 participants