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/extend ci with pipeline run #26

Merged
merged 1 commit into from
Apr 5, 2024
Merged

Conversation

redur
Copy link
Contributor

@redur redur commented Apr 4, 2024

Extended the CI to contain a pipeline run with a test borehole profile. I.e. the prod environment is now tested for errors.

Fixed some errors occurring in the prod environment. (Libraries and cases when there is no groundtruth or mlflow)
Switched to different opencv version as I struggled installing the right dependencies in the CI runner. The current version does not have any dependencies other than numpy. All binaries come shipped with the python package.

Note: There are a lot of commits in this PR that do not belong here. This is because this PR is based on #23 as we need some changes there to make this work.

It's best to merge #23 first. To review this PR have a look at the very last commit. Once #23 is ready, we can rebase this branch and then merge the changes contained in here to main.

@redur redur marked this pull request as draft April 4, 2024 13:29
@redur redur self-assigned this Apr 4, 2024
@redur redur force-pushed the feat/extend_CI_with_pipeline_run branch 2 times, most recently from 83db248 to d62c1f2 Compare April 5, 2024 05:55
@redur redur marked this pull request as ready for review April 5, 2024 06:04
@redur redur changed the base branch from main to feat/configure_pipeline_with_arguments_refactored April 5, 2024 07:15
@redur redur force-pushed the feat/configure_pipeline_with_arguments_refactored branch from d03d0b8 to 2166ad1 Compare April 5, 2024 07:34
@redur redur force-pushed the feat/extend_CI_with_pipeline_run branch from d62c1f2 to fa86d15 Compare April 5, 2024 07:57
Add missing dependency for prod environment.
use opencv version that does not require additional non python dependencies.
Catch the case when there is no ground truth file.
Base automatically changed from feat/configure_pipeline_with_arguments_refactored to feat/configure_pipeline_with_arguments April 5, 2024 09:39
Base automatically changed from feat/configure_pipeline_with_arguments to main April 5, 2024 09:48
@redur redur merged commit df7f9f6 into main Apr 5, 2024
3 checks passed
@redur redur deleted the feat/extend_CI_with_pipeline_run branch April 5, 2024 10:08
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