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

Fix argument name #67

Merged
merged 1 commit into from
Nov 30, 2022
Merged

Fix argument name #67

merged 1 commit into from
Nov 30, 2022

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