Running docker build only for changed methods #192
Merged
Conversation
This file contains hidden or 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
commit c588872 Author: gAldeia <guilherme.seidyo@gmail.com> Date: Sun Oct 6 22:45:32 2024 -0300 pandas dataframe for `model`, numpy array for `fit` commit 0c1f8e4 Author: gAldeia <guilherme.seidyo@gmail.com> Date: Sun Oct 6 18:56:02 2024 -0300 Misplaced values commit 61ea4d4 Author: gAldeia <guilherme.seidyo@gmail.com> Date: Sun Oct 6 18:48:11 2024 -0300 Using numpy arrays instead of dataframes commit f3cafc7 Author: gAldeia <guilherme.seidyo@gmail.com> Date: Sun Oct 6 13:34:57 2024 -0300 Fixed methods that does not take X=None as argument in model function commit a9e3f12 Author: gAldeia <guilherme.seidyo@gmail.com> Date: Sun Oct 6 13:15:23 2024 -0300 Fix pysr commit db26000 Author: gAldeia <guilherme.seidyo@gmail.com> Date: Sun Oct 6 13:14:01 2024 -0300 Fix rils-rols commit bc7a17c Author: gAldeia <guilherme.seidyo@gmail.com> Date: Sun Oct 6 13:13:21 2024 -0300 Fix tir commit 54c5ac4 Author: gAldeia <guilherme.seidyo@gmail.com> Date: Sun Oct 6 10:02:18 2024 -0300 Missing X argument in model commit 8db4acf Author: gAldeia <guilherme.seidyo@gmail.com> Date: Sat Oct 5 09:31:02 2024 -0300 Passing training data to algorithm.model in test_population commit 62f6549 Author: gAldeia <guilherme.seidyo@gmail.com> Date: Fri Oct 4 18:59:11 2024 -0300 Stop pushing image commit 508a771 Author: gAldeia <guilherme.seidyo@gmail.com> Date: Fri Oct 4 18:49:40 2024 -0300 Removed incomplete line in test_population commit b2c33ed Author: gAldeia <guilherme.seidyo@gmail.com> Date: Fri Oct 4 18:16:40 2024 -0300 Fixed checkout code step commit 801ac46 Author: gAldeia <guilherme.seidyo@gmail.com> Date: Fri Oct 4 18:04:16 2024 -0300 Removed loging in into dockerhub This is probably something we want to make a conditional check to run when pushing to master branch, but not docker-compose. Right now, as we do not have plans on merging this branch, I think we can leave as it is. I will open an issue regarding this. commit 5b943e9 Author: gAldeia <guilherme.seidyo@gmail.com> Date: Fri Oct 4 17:47:15 2024 -0300 Subsampling in test_population y_train is a np.array, we can pass a list of indexes directly to the operator[]. X_train is a dataframe, and our sample_idx is a list of randomly sampled integers between [0, len(X_train)), so we should operate with iloc.
…into docker-compose-ci
Member
|
@gAldeia is this ready to be merged? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Trying to avoid re-building all algorithms when a submission is changed