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

Add sources to YAML #10

Closed
wants to merge 3 commits into from
Closed

Add sources to YAML #10

wants to merge 3 commits into from

Conversation

linogaliana
Copy link
Collaborator

No description provided.

@linogaliana
Copy link
Collaborator Author

@TrystanViaud il n'y a pas les bassins de vie pour le moment sur la branche. C'est volontaire ? C'est-à-dire, avais tu validé ton ajout après la séance 10% ? Si oui, c'était sur cette branche ? Si non, est-ce que tu les retrouves en cliquant sur https://github.com/InseeFrLab/cartogether/edit/add_sources/resources/sources.yaml ? (parfois ils sont gardés en cache par github)

@TrystanViaud
Copy link
Contributor

@TrystanViaud il n'y a pas les bassins de vie pour le moment sur la branche. C'est volontaire ? C'est-à-dire, avais tu validé ton ajout après la séance 10% ? Si oui, c'était sur cette branche ? Si non, est-ce que tu les retrouves en cliquant sur https://github.com/InseeFrLab/cartogether/edit/add_sources/resources/sources.yaml ? (parfois ils sont gardés en cache par github)

Il me semble que les bassins de vie ont bien été ajoutés dans la section INSEE mais sous le nom "BV" (ligne 81 à la ligne 85). Effectivement le nom n'est pas très intuitif.

@linogaliana
Copy link
Collaborator Author

Ah parfait ! Désolé je l'avais pas vu, je craignais qu'une partie du travail ait disparu

@linogaliana
Copy link
Collaborator Author

@linogaliana linogaliana closed this Jan 4, 2024
@linogaliana linogaliana deleted the add_sources branch January 9, 2024 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants