Skip to content

Simplify kepler image#37

Merged
dommyrock merged 2 commits intomainfrom
simplify_kepler_image
Jun 18, 2025
Merged

Simplify kepler image#37
dommyrock merged 2 commits intomainfrom
simplify_kepler_image

Conversation

@dommyrock
Copy link
Member

@dommyrock dommyrock commented May 19, 2025

Simplify kepler image

Simplified kepler dockerfile and updated DOCS to use docker exec instead of creating new container and than running commands in it with docker run.

I ran some tests using these podman commands to make sure importing was still working the same way as before.

podman exec -it kepler kepler import_nist 2025 -d /data
for year in $(seq 2002 2025); do
  podman exec -it  kepler kepler import_nist $year -d /data
done

REFS

I have

  • run cargo fmt;
  • run cargo clippy;
  • run cargo testand all tests pass;
  • linked to the originating issue (if applicable).

@dommyrock
Copy link
Member Author

Cc @davidristov just want to make sure you're ok with these changes before merging ?

@dommyrock dommyrock merged commit e05e408 into main Jun 18, 2025
6 checks passed
@dommyrock dommyrock deleted the simplify_kepler_image branch June 18, 2025 06:42
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.

4 participants