Skip to content

Conversation

mariyayb
Copy link
Contributor

@mariyayb mariyayb commented Jul 9, 2022

What is this PR about?
update docstring as in issue for Bernoulli, OrderedProbit, ZeroInflatedBinomial, ZeroInflatedNegativeBinomial classes
#DataUmbrellaPyMCSprint
Cc: @reshamas
Cc: @saurbhc

Checklist

Major / Breaking Changes

  • None

Bugfixes / New features

  • None

Docs / Maintenance

Reference

Towards #5459

@michaelosthege
Copy link
Member

Thanks, this looks good!

@reshamas
Copy link
Member

reshamas commented Jul 9, 2022

@mariyayb
Would you be able to remove "#5459" from the title of the PR and add it to the description?
Example:

### Reference
Towards #5459 

@reshamas
Copy link
Member

reshamas commented Jul 9, 2022

@mariyayb Thanks for the PR. If you are working with a pair partner can you add their GitHub handle to the above description?
Example:

cc:  @xxxxx

@codecov
Copy link

codecov bot commented Jul 9, 2022

Codecov Report

Merging #5961 (4eb73de) into main (966aabf) will increase coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #5961      +/-   ##
==========================================
+ Coverage   89.41%   89.44%   +0.03%     
==========================================
  Files          73       73              
  Lines       13292    13292              
==========================================
+ Hits        11885    11889       +4     
+ Misses       1407     1403       -4     
Impacted Files Coverage Δ
pymc/distributions/discrete.py 99.21% <ø> (ø)
pymc/step_methods/hmc/base_hmc.py 90.55% <0.00%> (+0.78%) ⬆️
pymc/parallel_sampling.py 86.79% <0.00%> (+0.99%) ⬆️

@mariyayb mariyayb changed the title Bernoulli docstring update as in #5459 Bernoulli, OrderedProbit, ZeroInflatedBinomial, ZeroInflatedNegativeBinomial docstring update Jul 9, 2022
@aloctavodia aloctavodia merged commit 7be9a19 into pymc-devs:main Jul 13, 2022
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.

4 participants