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

Feat/configure pipeline with arguments #21

Merged
merged 20 commits into from
Apr 5, 2024

Conversation

redur
Copy link
Contributor

@redur redur commented Apr 3, 2024

Created a CLI interface to run our main script.

We now have the option to define input and output directories.
We can specify single files for the pipeline to be run on.
We can specify what drawings are performed (bounding boxes for predictions and lines).

@redur redur self-assigned this Apr 3, 2024
Copy link
Contributor

@stijnvermeeren-swisstopo stijnvermeeren-swisstopo left a comment

Choose a reason for hiding this comment

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

A few ideas/comments.

Also check the additional proposal/comments in #22

pyproject.toml Outdated Show resolved Hide resolved
src/stratigraphy/main.py Outdated Show resolved Hide resolved
src/stratigraphy/main.py Outdated Show resolved Hide resolved
@redur redur force-pushed the feat/configure_pipeline_with_arguments branch from 9679ad8 to 672a894 Compare April 5, 2024 06:30
Copy link
Contributor

@stijnvermeeren-swisstopo stijnvermeeren-swisstopo left a comment

Choose a reason for hiding this comment

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

Looks good (on condition that #23 is approved and merged as well) apart from these minor details.

README.md Outdated Show resolved Hide resolved
src/stratigraphy/main.py Outdated Show resolved Hide resolved
src/stratigraphy/main.py Outdated Show resolved Hide resolved
@redur redur merged commit f86e4d6 into main Apr 5, 2024
2 checks passed
@redur redur deleted the feat/configure_pipeline_with_arguments branch April 5, 2024 09:48
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