Add baseline LR advisor #29
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Generated automatically by `pip install -e .[baseline-lr] && make lock`
Co-authored-by: Alex Hedges <git@alexhedges.dev> Co-authored-by: Alexander Jephtha <ajep3697@uni.sydney.edu.au>
Without this change, importing any bot will cause a crash unless the optional dependency is installed.
To make builds easier, I decided to store the model weights directly in an OCI image. First, I downloaded the model weights and unzipped them into a directory named `lr_model/`. Then, I created a `Dockerfile` in the same directory that contained `lr_model/`: ```dockerfile FROM scratch COPY lr_model/ /lr_model/ ``` Finally, I ran the following commands to create and upload the OCI image containing only the model weights directory: ```shell docker build --platform linux/amd64 --tag ghcr.io/allan-dip/chiron-utils:baseline-lr-model-2025-01-14 . docker push ghcr.io/allan-dip/chiron-utils:baseline-lr-model-2025-01-14 ```
Co-authored-by: Alex Hedges <git@alexhedges.dev>
gale2307
approved these changes
Feb 5, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR contains the code changes from #19 and 53ba04a, which I then cleaned up and rebased against the more general changes I have already merged to
main. The bot itself was good, but I needed to modify the code so we can merge it without causing problems for any other bots.The code changes to these specific files should be pretty minimal, but they should still be tested and looked over to make sure there weren't any regressions.