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 jupyter style guide #5267

Merged
merged 6 commits into from
Dec 19, 2021
Merged

Conversation

OriolAbril
Copy link
Member

@OriolAbril OriolAbril commented Dec 16, 2021

Update the jupyter style guide to include explanation on the extra dependencies template introduced in pymc-devs/pymc-examples#249 (already used in the glm robust with outlier detection example)

@codecov
Copy link

codecov bot commented Dec 16, 2021

Codecov Report

Merging #5267 (fab2b66) into main (cce1613) will increase coverage by 0.01%.
The diff coverage is n/a.

❗ Current head fab2b66 differs from pull request most recent head 20f778b. Consider uploading reports for the commit 20f778b to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##             main    #5267      +/-   ##
==========================================
+ Coverage   79.17%   79.18%   +0.01%     
==========================================
  Files          88       88              
  Lines       14346    14353       +7     
==========================================
+ Hits        11359    11366       +7     
  Misses       2987     2987              
Impacted Files Coverage Δ
pymc/model.py 83.26% <0.00%> (ø)
pymc/distributions/transforms.py 92.55% <0.00%> (+0.59%) ⬆️

Copy link
Member

@ltoniazzi ltoniazzi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure you want to include that doc type references should use paths. Mentioning this as I struggled with it in PR 195 (see @OriolAbril reviewed on 4 Sep there), and I could not easily reach a page explaining this from the Jupyter Style Guide.

docs/source/contributing/jupyter_style.md Outdated Show resolved Hide resolved
docs/source/contributing/jupyter_style.md Outdated Show resolved Hide resolved
Copy link
Member

@michaelosthege michaelosthege left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merge when you think it's a good time

@OriolAbril OriolAbril merged commit c15af2a into pymc-devs:main Dec 19, 2021
@OriolAbril OriolAbril deleted the jupyter_style branch December 19, 2021 19:15
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.

Standardize names for common variables and quantities in the Jupyter Notebook Style Guide
3 participants