Skip to content

Conversation

sudarsan2k5
Copy link
Contributor

Fixes #64

  • renamed f_node to f_apply
  • corrected doc string
  • verified all the related tests

Here are a few important guidelines and requirements to check before your PR can be merged:

  • There is an informative high-level description of the changes.
  • The description and/or commit message(s) references the relevant GitHub issue(s).
  • pre-commit is installed and set up.
  • The commit messages follow these guidelines.
  • The commits correspond to relevant logical changes, and there are no commits that fix changes introduced by other commits in the same branch/BR.
  • There are tests covering the changes introduced in the PR.

@ferrine
Copy link
Member

ferrine commented Nov 30, 2022

Let's wait tests pass and merge! Thank you!

@twiecki twiecki merged commit 4077c47 into pymc-devs:main Nov 30, 2022
@twiecki
Copy link
Member

twiecki commented Nov 30, 2022

Thanks @sudarsan2k5

@sudarsan2k5 sudarsan2k5 deleted the fix_argument_name branch November 30, 2022 17:26
@twiecki twiecki mentioned this pull request Nov 30, 2022
6 tasks
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.

Argument name is incorrect

3 participants