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

first changes to make the docker works with #5

Merged
merged 9 commits into from
Oct 30, 2024
Merged

first changes to make the docker works with #5

merged 9 commits into from
Oct 30, 2024

Conversation

MichelDaab
Copy link
Collaborator

lidar_selecter and on GPAO

README.md Outdated Show resolved Hide resolved
configs/configs_patchwork.yaml Outdated Show resolved Hide resolved
OUTPUT_FILE: null # path to the (resulting) file with added points.
INPUT_INDICES_MAP: null # path for the indices map reflecting the changes to the recipient
OUTPUT_INDICES_MAP: null
CSV_PATH: null # path to the csv file that log the lidar files to process with patchwork
# INPUT_DIRECTORY: null # directory for input (shapefile)
Copy link
Collaborator

Choose a reason for hiding this comment

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

code commenté à enlever

Copy link
Collaborator

Choose a reason for hiding this comment

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

up

Copy link
Collaborator

Choose a reason for hiding this comment

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

up

gpao_builder.py Outdated Show resolved Hide resolved
@leavauchier
Copy link
Collaborator

Ah et il manque la mise à jour du CHANGELOG

ther is, get the donor file matching the
recipient file from that csv file
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated
@@ -1,5 +1,9 @@
# CHANGELOG

## 1.1.0
- modification de chemin pour pouvoir passer dans la gpao
- patchwork vérifie maintenant s'il y a un ficheir csv en entrée. Si c'est le cas, le fichier donneur utilisé est celui qui correspond au fichier receveur dans le ficheir csv. S'il n'y a pas de fichier donneur correspondant, patchwork termine sans rien faire
Copy link
Collaborator

Choose a reason for hiding this comment

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

En attendant une meilleure solution (notamment une amélioration du fonctionnement lidarexpress, qui pour le moment a besoin d'avoir tous les fichiers las du projet dans le dossier d'entrée de chaque étape), j'ai l'impression qu'il serait préférable d'avoir l'option de faire une copie du fichier d'entrée vers le dossier de sortie quand on est hors du csv, (peut-être même en ajoutant le tag origin=0 (?), mais pour ça je suis moins sûre)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Je ne suis pas sûr de comprendre ce que tu veux. Tu souhaite que, si un fichier csv est fourni ET qu'il n'y a pas de fichier donneur associé -> plutôt que de ne rien faire, copier le fichier receveur de l'entrée vers la sortie ?

Copy link
Collaborator

Choose a reason for hiding this comment

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

c'est ça (peut-être à mettre en option pour le moment où on améliorera lidarexpress pour gérer le passage des fichiers en entrée d'une étape autrement qu'en lisant dans le dossier de sortie de l'étape précédente)

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 tu peux rajouter cette possibilité stp ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

c'est bon

patchwork.py Outdated Show resolved Hide resolved
patchwork.py Outdated Show resolved Hide resolved
patchwork.py Outdated Show resolved Hide resolved
OUTPUT_FILE: null # path to the (resulting) file with added points.
INPUT_INDICES_MAP: null # path for the indices map reflecting the changes to the recipient
OUTPUT_INDICES_MAP: null
CSV_PATH: null # path to the csv file that log the lidar files to process with patchwork
# INPUT_DIRECTORY: null # directory for input (shapefile)
Copy link
Collaborator

Choose a reason for hiding this comment

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

up

configs/configs_patchwork.yaml Outdated Show resolved Hide resolved
test/test_patchwork.py Outdated Show resolved Hide resolved
CHANGELOG.md Outdated
@@ -1,5 +1,9 @@
# CHANGELOG

## 1.1.0
- modification de chemin pour pouvoir passer dans la gpao
- patchwork vérifie maintenant s'il y a un ficheir csv en entrée. Si c'est le cas, le fichier donneur utilisé est celui qui correspond au fichier receveur dans le ficheir csv. S'il n'y a pas de fichier donneur correspondant, patchwork termine sans rien faire
Copy link
Collaborator

Choose a reason for hiding this comment

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

c'est ça (peut-être à mettre en option pour le moment où on améliorera lidarexpress pour gérer le passage des fichiers en entrée d'une étape autrement qu'en lisant dans le dossier de sortie de l'étape précédente)

OUTPUT_FILE: null # path to the (resulting) file with added points.
INPUT_INDICES_MAP: null # path for the indices map reflecting the changes to the recipient
OUTPUT_INDICES_MAP: null
CSV_PATH: null # path to the csv file that log the lidar files to process with patchwork
# INPUT_DIRECTORY: null # directory for input (shapefile)
Copy link
Collaborator

Choose a reason for hiding this comment

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

up

## 1.1.0
- modification de chemin pour pouvoir passer dans la gpao
- coupure des chemins de fichiers en chemins de répertoires/nom de fichiers pour pouvoir les utiliser sur docker + store
- patchwork vérifie maintenant s'il y a un ficheir csv en entrée. Si c'est le cas, le fichier donneur utilisé est celui qui correspond au fichier receveur dans le fichier csv. S'il n'y a pas de fichier donneur correspondant, patchwork termine sans rien faire
Copy link
Collaborator

Choose a reason for hiding this comment

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

typo :

Suggested change
- patchwork vérifie maintenant s'il y a un ficheir csv en entrée. Si c'est le cas, le fichier donneur utilisé est celui qui correspond au fichier receveur dans le fichier csv. S'il n'y a pas de fichier donneur correspondant, patchwork termine sans rien faire
- patchwork vérifie maintenant s'il y a un fichier csv en entrée. Si c'est le cas, le fichier donneur utilisé est celui qui correspond au fichier receveur dans le fichier csv. S'il n'y a pas de fichier donneur correspondant, patchwork termine sans rien faire

CHANGELOG.md Outdated
@@ -1,5 +1,9 @@
# CHANGELOG

## 1.1.0
- modification de chemin pour pouvoir passer dans la gpao
- patchwork vérifie maintenant s'il y a un ficheir csv en entrée. Si c'est le cas, le fichier donneur utilisé est celui qui correspond au fichier receveur dans le ficheir csv. S'il n'y a pas de fichier donneur correspondant, patchwork termine sans rien faire
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 tu peux rajouter cette possibilité stp ?

recipient without doing anything
@MichelDaab MichelDaab merged commit 4f9e334 into main Oct 30, 2024
1 check passed
@leavauchier leavauchier deleted the gpao branch October 30, 2024 10:44
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.

2 participants