allow dataset_y updates with multiple values - #49
Conversation
|
@Lance-Drane I have also needed to make some changes to benchmarks, to get them to work on my end. They are entangled to some other changes that mostly affect the sable backend here: dc372f2 |
|
This looks fine and I'm happy to approve (also think it'll be good to get this in before develop -> main), could you add that commit to the update_fix branch? Alternatively you can just change the target from |
- bump the SABLE version to get the newest features - use the hyperparameter prior_std instead of alpha - transition the sable backend and clients to use that feature, and clean up specification of unnecessary hyperparameters - fix and adapt benchmarks to the new vesion
|
With the addition of torch as a dependency to Sable, we probably need to go ahead and make Sable an optional dependency. You'll need over 4G of python dependencies alone to build it now. I am going to go ahead and make this change on another branch based off of yours, if you are okay with that I think we should merge this. This will go under In UV (for development) you install this via In Pip (for our users) you'd use |
Signed-off-by: Lance-Drane <Lance-Drane@users.noreply.github.com>
Signed-off-by: Lance-Drane <Lance-Drane@users.noreply.github.com>
Signed-off-by: Lance-Drane <Lance-Drane@users.noreply.github.com>
Signed-off-by: Lance-Drane <Lance-Drane@users.noreply.github.com>
2 participants
Lance, that is a fix that is needed for the recent merge that allows for multiple y values. I do not know when we plan to merge to main, but I believe this is small enough to go in right now. Let me know if you see any issues.