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

some small improvements to get everything ready for the sprint #5461

Merged
merged 3 commits into from
Feb 15, 2022

Conversation

OriolAbril
Copy link
Member

@codecov
Copy link

codecov bot commented Feb 10, 2022

Codecov Report

Merging #5461 (a562321) into main (e018309) will increase coverage by 42.36%.
The diff coverage is n/a.

❗ Current head a562321 differs from pull request most recent head d288f0d. Consider uploading reports for the commit d288f0d to get more accurate results

Impacted file tree graph

@@             Coverage Diff             @@
##             main    #5461       +/-   ##
===========================================
+ Coverage   37.86%   80.23%   +42.36%     
===========================================
  Files          82       82               
  Lines       14215    13941      -274     
===========================================
+ Hits         5382    11185     +5803     
+ Misses       8833     2756     -6077     
Impacted Files Coverage Δ
pymc/sampling_jax.py 0.00% <0.00%> (-98.31%) ⬇️
pymc/variational/stein.py 38.00% <0.00%> (-10.34%) ⬇️
pymc/variational/approximations.py 30.06% <0.00%> (-5.65%) ⬇️
pymc/variational/opvi.py 30.29% <0.00%> (-5.48%) ⬇️
pymc/variational/flows.py 31.55% <0.00%> (-5.24%) ⬇️
pymc/distributions/mixture.py 19.76% <0.00%> (-1.42%) ⬇️
pymc/variational/operators.py 48.78% <0.00%> (-1.22%) ⬇️
pymc/variational/callbacks.py 46.93% <0.00%> (-1.07%) ⬇️
pymc/plots/__init__.py 95.65% <0.00%> (-0.19%) ⬇️
pymc/step_methods/hmc/nuts.py 95.00% <0.00%> (+6.18%) ⬆️
... and 55 more


Tags can be anything, but we ask you to try to use [existing tags](https://github.com/pymc-devs/pymc/wiki/Categories-and-Tags-for-PyMC-Examples)
to avoid the tag list from getting too long.

Each notebook should have a single category indicating the level of the notebook.
Choose a category from [existing categories](https://github.com/pymc-devs/pymc/wiki/Categories-and-Tags-for-PyMC-Examples#categories).
Each notebook should have a one or two categories indicating:
Copy link
Member

@canyon289 canyon289 Feb 11, 2022

Choose a reason for hiding this comment

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

I think we should make it two so its not ambiguous , also so its very clear what category a document is in

Copy link
Member Author

Choose a reason for hiding this comment

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

The problem is that most notebooks are a mix of different types, so forcing the two categories already is a huge effort. My idea was to use that for new notebooks, for notebooks that already match a type and for notebooks we want to split/combine/restructure

Copy link
Member

Choose a reason for hiding this comment

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

My suggestion is force two moving forward and we can fix the older ones over time. This way at least at some point in the future all notebooks will have both labels

Copy link
Member Author

Choose a reason for hiding this comment

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

sounds good, I have updated the proposal template in pymc-devs/pymc-examples#281. I'd still keep it like this here for now because >95% of the changes that we have to do (and 99.9% of the changes done by newcomers) are updating existent notebooks, not writing new ones

@OriolAbril OriolAbril merged commit 7d974d1 into pymc-devs:main Feb 15, 2022
@OriolAbril OriolAbril deleted the pre_sprint_checks branch February 15, 2022 23:05
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.

2 participants