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

Update mean cov GP notebook #301

Merged
merged 4 commits into from
Nov 5, 2022
Merged

Update mean cov GP notebook #301

merged 4 commits into from
Nov 5, 2022

Conversation

canyon289
Copy link
Member

@canyon289 canyon289 commented Mar 24, 2022

Update mean and cov notebook

Just update image size so it doesn't require scrolling

Questions and answers

Helpful links

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@canyon289
Copy link
Member Author

Where do I put diataxis type?
image

@review-notebook-app
Copy link

View / edit / reply to this conversation on ReviewNB

OriolAbril commented on 2022-03-31T21:30:26Z
----------------------------------------------------------------

The myst target (the text inside the()= ) should be something that is specific to this notebook, for example (and preferably to avoid confusions) the file name of the notebook


@OriolAbril
Copy link
Member

Where do I put diataxis type?

There can be multiple categories like there are multiple tags: :category: intermediate, reference

@review-notebook-app
Copy link

View / edit / reply to this conversation on ReviewNB

aloctavodia commented on 2022-04-01T05:52:23Z
----------------------------------------------------------------

Line #10.    %matplotlib inline

Do we need this?


@review-notebook-app
Copy link

View / edit / reply to this conversation on ReviewNB

aloctavodia commented on 2022-04-01T05:52:24Z
----------------------------------------------------------------

Instead of FIGSIZE = (10, 4)

you can do

plt.rcParams["figure.figsize"] = (10,4)

and then all figures in the session will have the desired figsize, without needing to manually pass FIGSIZE


@OriolAbril OriolAbril merged commit e332822 into main Nov 5, 2022
@OriolAbril OriolAbril deleted the update_mean_cov branch November 5, 2022 19:47
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.

Update Cov and Means GP notebook
2 participants