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

Ajouter les données pour la région de la Corse #2856

Open
Tracked by #2814
mxmeunier opened this issue Nov 6, 2024 · 5 comments · May be fixed by #2899
Open
Tracked by #2814

Ajouter les données pour la région de la Corse #2856

mxmeunier opened this issue Nov 6, 2024 · 5 comments · May be fixed by #2899
Assignees

Comments

@mxmeunier
Copy link
Collaborator

Story: #2814

Description

Ajouter un producer et importer le jeu de donnée 2022

@mxmeunier
Copy link
Collaborator Author

aller voir si on a pas les données 2023

@461OceanBd 461OceanBd self-assigned this Nov 18, 2024
@461OceanBd 461OceanBd moved this from To Do to In Progress in Data.Subvention Nov 18, 2024
461OceanBd added a commit that referenced this issue Nov 19, 2024
- Added headers path
- Added excluded BOM conf when parsing
461OceanBd added a commit that referenced this issue Nov 19, 2024
- Added headers path
- Added excluded BOM conf when parsing
461OceanBd added a commit that referenced this issue Nov 19, 2024
- Added trim() to all csv headers (remove line break chars)
461OceanBd added a commit that referenced this issue Nov 19, 2024
- Added columns callback tests
461OceanBd added a commit that referenced this issue Nov 19, 2024
- Added columns callback tests
@461OceanBd
Copy link
Collaborator

aller voir si on a pas les données 2023

pas de données 2023 a priori

@461OceanBd
Copy link
Collaborator

461OceanBd commented Nov 20, 2024

@VSerain @AlixBouillon 474 lignes non importées pour 3903 lignes. Dans tous les cas il s'agit d'un problème de SIRET manquant ou invalide (quelques SIRET font 9 chiffres seulement).

@VSerain
Copy link
Collaborator

VSerain commented Nov 20, 2024

@VSerain @AlixBouillon 474 lignes non importées pour 3903 lignes. Dans tous les cas il s'agit d'un problème de SIRET manquant ou invalide (quelques SIRET font 9 chiffres seulement).

9 chiffres ça veut dire que ce sont des siren ! @AlixBouillon je dirrait qu'on leurs renvoies mais qu'on intègre déjà les 3500 linges valkide ?

@461OceanBd
Copy link
Collaborator

Pour la très grande majorité se sont des SIRET manquant. Voici le rapport d'erreur :
s_open_data_subentions-2022.csv-errors.csv

@461OceanBd 461OceanBd moved this from In Progress to To Review in Data.Subvention Nov 21, 2024
@461OceanBd 461OceanBd linked a pull request Nov 25, 2024 that will close this issue
461OceanBd added a commit that referenced this issue Nov 27, 2024
- remove unnecessary trim
461OceanBd added a commit that referenced this issue Nov 27, 2024
- Added headers path
- Added excluded BOM conf when parsing
461OceanBd added a commit that referenced this issue Nov 27, 2024
- Added trim() to all csv headers (remove line break chars)
461OceanBd added a commit that referenced this issue Nov 27, 2024
- Added columns callback tests
461OceanBd added a commit that referenced this issue Nov 27, 2024
- remove unnecessary trim
461OceanBd added a commit that referenced this issue Nov 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: To Review
Development

Successfully merging a pull request may close this issue.

4 participants