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

Replace seed with key #1145

Open
wants to merge 18 commits into
base: main
Choose a base branch
from
Open

Conversation

Tomas542
Copy link

@Tomas542 Tomas542 commented Nov 30, 2024

Replace seed with key in all files. Add key=None error for 2 optimizers

UPD: Forgot, added example in add_noise docstring

@Tomas542
Copy link
Author

Tomas542 commented Nov 30, 2024

It seems, I've failed in some tests (maybe in rewritten there is some error), need time to set up Linux VM to run bash scripts...

UPD: oh, there are logs!

@Tomas542
Copy link
Author

messed with commit text... removed random word from error

@Tomas542
Copy link
Author

this test will fail due to assertions about tree not equal in key values. I will fix it later. Now I just reducing errors in logs

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