-
Notifications
You must be signed in to change notification settings - Fork 3
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
Issue 271 - reprise de la documentation #273
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je propose aussi un changement du paragraphe en anglais du readme :
This program validates a dataset (a folder containing geodata files and other documents) according to a schema describing the folder structure and the feature catalog. It outputs a report containing validation errors and warnings, and normalized data ready for database integration.
It has been originaly developped for the Géoportail de l'urbanisme to check urban planning documents against CNIG standards in order to produce a nationwide database.
* Pour un fichier `./document-test/TRONCON_ROUTE.shp`, on notera la présence d'un fichier CSV `./document-test/TRONCON_ROUTE.vrows` contenant le résultat de la conversion du Shapefile en CSV. | ||
* Pour un fichier GML `./document-test/GeoVendee_LeTallud-StGemme.gml` contenant plusieurs collections, on notera la présence d'un dossier `./document-test/GeoVendee_LeTallud-StGemme.vtabs` contenant le résultat de la conversion du GML en CSV. | ||
|
||
Ces conversions sont réalisées à l'aide de [ogr2ogr de GDAL](../dependencies/ogr2ogr.md). L'utilisation d'extension dédiées facilite principalement une nouvelle exécution du validateur sur un dossier. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Attention au lien si le doc ogr2ogr à part disparait ou est déplacé
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Vu, mais je pense qu'on sera potentiellement amené à détailler les commandes appelées
@@ -1,8 +1,8 @@ | |||
# geotools | |||
# GeoTools |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comme pour ogr2ogr, peut-être n'est il pas utile d'en faire un fichier à part et on peut le mettre dans un gros fichier de prérequis.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Vu, mais si ça te va, je conserve en l'état pour l'instant.
@slafayIGN merci pour la relecture! j'ai pris en compte la plupart des suggestions |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good job
|
||
You may run `bash .ci/build-openjdk11.sh` to ensure that source code is formatted, build code and run tests. | ||
|
||
## Create releases |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, i like that section
Je merge après avoir testé en local (j'ignore pourquoi le contrôle de formatage est en échec uniquement avec github actions, mais ce n'est pas lié à cette reprise de la documentation). |
Aperçu ici : https://github.com/IGNF/validator/tree/issue_271#validator
@slafayIGN Il restera à ajouter d'autres exemples pour la commande document_validator et faire une documentation dédié pour les plugins CNIG et DGPR, mais je pense qu'on peut merger ces premiers changements.
(closes #271)