Supprime les dates de publication, notes et références sans valeurs.

[eraviart]

• Changement mineur.
• Périodes concernées : toutes.
• Zones impactées : openfisca_france/parameters.
• Détails :
  • Supprime, dans tous les paramètres de type valeur (et donc ni les nœuds, ni les barèmes), les parties des métadonnées "reference", "notes" et "official_journal_date" dont la date ne correspond à aucune valeur.

[eraviart]

Vu l'ampleur des suppressions, il y a peut-être des parties de l'arborescence à préserver ?

[Sasha-Laniece]

Effectivement, puisque le postulat de base est que ces références sont "en trop" car ajoutées lors de l'harmonisation (suite au découpage de fichiers contenant plusieurs paramètres), peut-être qu'on peut limiter cette PR aux dossiers touchés par l'harmonisation ?

J'en profite pour rappeler que cette PR est le résultat d'une demande apparue ici : https://github.com/openfisca/openfisca-france/pull/1761

[benjello]

@eraviart @Sasha-Laniece : La revue va être assez compliquée ? Déjà est-ce que l'on peut vérifier que l'on n'effacer jamais une entrée définitivement ? Après est-ce que l'on peut procéder par sous -arbre ?

Preneur de toutes vos suggestions pour essayer de ne rien perdre dans ce processus.

[eraviart]

Je viens de mettre à jour ce PR en utilisant une nouvelle stratégie : Au lieu de supprimer toutes les référence, notes etofficial_journal_date pour lesquelles il n'existe pas de valeur correspondant à leur date, le script supprime uniquement celles qui existent dans un autre paramètre (avec une valeur correspondant à leur date). Ainsi, une référence sans valeur n'est jamais supprimée si elle n'est pas utilisée correctement ailleurs. Et si une référence sans valeur est supprimée, c'est qu'elle figure ailleurs dans un endroit a priori proche dans la hiérarchie, donc pas trop difficile à retrouver. Cette technique permet de conserver plus de 900 référence, notes etofficial_journal_date sans valeur correspondant à leur date.

J'ai séparé cette PR en 2 commits :

• le premier trie les champs des paramètres sans rien enlever
• le second supprime les référence, notes etofficial_journal_date sans valeurs correspondantes

[benjello]

Je valide sachant que ce processus a correctement fonctionné sur les barèmes IPP.

[clallemand]

Je valide sachant que ce processus a correctement fonctionné sur les barèmes IPP.

Que signifie "ce processus a correctement fonctionné sur les barèmes" ?

[benjello]

Je valide sachant que ce processus a correctement fonctionné sur les barèmes IPP.

Que signifie "ce processus a correctement fonctionné sur les barèmes" ?

On l'a passé sur les barèmes IPP, vérifié à l'oeil au maximum et régénéré les pages du site web de l'IPP. Et fusionné !

[Sasha-Laniece]

♥♥♥