Skip to content

add back documentation and images#1394

Open
kaligrafy wants to merge 1 commit intochairemobilite:mainfrom
kaligrafy:addDoc
Open

add back documentation and images#1394
kaligrafy wants to merge 1 commit intochairemobilite:mainfrom
kaligrafy:addDoc

Conversation

@kaligrafy
Copy link
Collaborator

the images and documentation for object diagrams, POIs analysis and population synthesis was still in transition-legacy. This documentation will be very useful for poi analysis, population synthesis and OD trips analysis and simulations in transition

the images and documentation for object diagrams, POIs analysis and population synthesis was still in transition-legacy. This documentation will be very useful for poi analysis, population synthesis and OD trips analysis and simulations in transition
@kaligrafy kaligrafy requested a review from tahini May 28, 2025 19:23
Copy link
Collaborator

@tahini tahini left a comment

Choose a reason for hiding this comment

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

Je crois que ça vient directement de transition-legacy, sans avoir été mis à jour. Probablement que certains imports sont déjà implémentés via les scripts d'import de données pour l'algo génétique. Il faudrait soit revoir la doc en question, ou alors ajouter un gros warning dans les fichiers .md pour dire que cette information date de 202x? 201x? et qu'elle devrait être revisée (au cas où qqn prendrait ça pour du cash).


## Importation des données

1. [Importer les données d'enquêtes Origine-Destination](#importODSurveyData)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Est-ce que ces liens fonctionnent? D'ailleurs probablement qu'une partie de ces imports ont déjà été implémentés via les scripts d'import de données, tels que décrits dans la section algo génétique de la doc. Il faudrait mettre à jour.

Copy link
Collaborator

@greenscientist greenscientist left a comment

Choose a reason for hiding this comment

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

Je vais avoir un avis similaire à Geneviève ici.
Je ne pense pas qu'on devrait importer ceci en block. Il y a probablement pleins de choses désuette et des trucs qui ne sont pas implémenté.

Si il y a des trucs pertinent, je crois qu'on devrait les importer à la pièce, en petit morceau, pour qu'on puisse bien les réviser.

Si quelqu'un a besoin de l'info qui est dans legacy, on peut juste pointer au repo legacy directement.

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.

3 participants