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

Add gh-action to create a release on v*.*.* tags #13

Merged
merged 2 commits into from
Jun 25, 2024
Merged

Conversation

leavauchier
Copy link
Contributor

@leavauchier leavauchier commented Jun 24, 2024

Create github release

  • publish package on pypi

environment.yml Show resolved Hide resolved
@leavauchier leavauchier merged commit 961ae62 into main Jun 25, 2024
2 checks passed
@leavauchier leavauchier deleted the auto-release branch June 25, 2024 13:45


def test_main():
ini_las = "test/data/4_6.las"
dsm_dimension = "dsm_marker"
dtm_dimension = "dtm_marker"
with tempfile.NamedTemporaryFile(suffix="_mark_points_output.las") as las_output:
main(ini_las, las_output.name, dsm_dimension, dtm_dimension, "", "")
mark_points_to_use_for_digital_models_with_new_dimension(
ini_las, las_output.name, dsm_dimension, dtm_dimension, "", ""

Choose a reason for hiding this comment

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

niveau lecture, on sait pas trop à quoi ça correspond "", ""

ce serait plus lisible avec les noms des parametres

output_dsm=""

output_dtm=""

@@ -1,13 +1,19 @@
[project]
name = "pdal_ign_macro"
dynamic = ["version"]
readme = "README.md"
description = "Macros and pipelines for Pdal based on the https://github.com/IGNF/pdal-ign-plugin plugin"

Choose a reason for hiding this comment

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

Comme c'est la description du package python, j'aurai mis un truc qui fait la difference entre le python et le c++, comme ça :

"Python examples to use C++ PDAL plugin https://github.com/IGNF/pdal-ign-plugin"

Enfin c'est qu'une suggestion, c'est comme tu préfères.

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