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

Build CI #31

Merged
merged 14 commits into from
Jun 25, 2024
Merged

Build CI #31

merged 14 commits into from
Jun 25, 2024

Conversation

kmuehlbauer
Copy link
Collaborator

@kmuehlbauer kmuehlbauer commented Jun 25, 2024

This tries to build the docker image inside PR for better debugging.

  • base image is jammy (22.04)
  • python version is 3.11

It looks like baltrad hlhdf does not compile with python 3.11. I'm trying to investigate, but this seems to be a numpy compat issue, cc @andershenja.

Copy link

github-actions bot commented Jun 25, 2024

👋 Thanks for opening this PR! The Cookbook will be automatically built with GitHub Actions. To see the status of your deployment, click below.
🔍 Git commit SHA: 3d7946b
✅ Deployment Preview URL: https://openradar.github.io/erad2024/_preview/31

@kmuehlbauer
Copy link
Collaborator Author

@mgrover1 @DanielMichelson @andershenja

I'm not able to fix the issues with hlhdf. I've tried a bit in this fork https://github.com/openradar/hlhdf but I'm no expert at all in C/numpy expecially when it comes to heave use of macro's. So if there is someone capable of fixing these issues, just let us know.

I'll skip the baltrad install for now to see, if LROSE is working.

@kmuehlbauer kmuehlbauer changed the title Build ci Build CI Jun 25, 2024
@kmuehlbauer
Copy link
Collaborator Author

@mgrover1 Going to merge this. Checking build and push of the image to the ghcr.io.

@kmuehlbauer kmuehlbauer merged commit 4ea4347 into openradar:main Jun 25, 2024
2 checks passed
@kmuehlbauer kmuehlbauer deleted the build-ci branch June 25, 2024 09:55
github-actions bot pushed a commit that referenced this pull request Jun 25, 2024
@kmuehlbauer kmuehlbauer mentioned this pull request Jun 25, 2024
30 tasks
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.

1 participant