blackjax icon indicating copy to clipboard operation
blackjax copied to clipboard

Fix "Langeving" typo in sampling.rst

Open matsen opened this issue 3 years ago • 1 comments

Thank you for opening a PR!

A few important guidelines and requirements before we can merge your PR:

  • [ ] If I add a new sampler, there is an issue discussing it already;
  • [ ] We should be able to understand what the PR does from its title only;
  • [ ] There is a high-level description of the changes;
  • [ ] There are links to all the relevant issues, discussions and PRs;
  • [ ] The branch is rebased on the latest main commit;
  • [ ] Commit messages follow these guidelines;
  • [ ] The code respects the current naming conventions;
  • [ ] Docstrings follow the numpy style guide
  • [ ] pre-commit is installed and configured on your machine, and you ran it before opening the PR;
  • [ ] There are tests covering the changes;
  • [ ] The doc is up-to-date;
  • [ ] If I add a new sampler* I added/updated related examples

Consider opening a Draft PR if your work is still in progress but you would like some feedback from other contributors.

matsen avatar Sep 21 '22 20:09 matsen

Thanks for contributing! Could you rebase your branch on main? There's a commit there fixing the error we're seeing in CI.

rlouf avatar Sep 21 '22 21:09 rlouf

Codecov Report

Merging #331 (df2d54c) into main (8428583) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #331   +/-   ##
=======================================
  Coverage   98.68%   98.68%           
=======================================
  Files          44       44           
  Lines        1894     1894           
=======================================
  Hits         1869     1869           
  Misses         25       25           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codecov[bot] avatar Sep 24 '22 12:09 codecov[bot]