-
Notifications
You must be signed in to change notification settings - Fork 5
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
Docs: Revised basic tutorials #584
Conversation
small point, but per our code style guidelines, notebooks should be "Camel case with spaces for Jupyter notebooks: Example Notebook.ipynb" ie, no dashes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi Chad, for the newest version of the core which doesn't include aliases (and as such doesn't have the .model
on the state – instead requiring you to use .models[-1]
), there are some changes I'd suggest. These are in #587 which will merge onto this PR.
Also, to get this to run, you need to be running the newest pre-release of the synthetic abstract experiment runner – v0.0.8 (do pip install autora-synthetic-abstract-equation==0.0.8
) – it includes support for random_state
in the experiment_runner
…ions-john 569 revise tutorial suggestions john
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same here; looks good!
Description
Completed the basic tutorials with the latest AutoRA version to be released.
If the issue is on Github, simply link to it using the '#' symbol; otherwise, provide a notion page link):
Type of change
Delete as appropriate: