Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

correct the docstring parameter names to be the same as the variable names #73

Merged

Conversation

VladimirFokow
Copy link
Contributor

@VladimirFokow VladimirFokow commented Jul 20, 2023

  • The actual parameter name is edge_length but the dosctring had edge_lengths
    (maybe instead should rename the variable edge_length to edge_lengths? - but then it should be renamed everywhere)
  • corrected tol in the docstring. It was tolerance

@VladimirFokow VladimirFokow changed the title correct the docstring parameter name to be the same as the variable name correct the docstring parameter names to be the same as the variable names Jul 20, 2023
@paulbrodersen paulbrodersen merged commit 6eab68c into paulbrodersen:master Aug 7, 2023
@paulbrodersen
Copy link
Owner

Thanks for the PR and sorry for the late reply -- I was on vacation, and did not check my github. If you come across any other issues or have any suggestions, do let me know.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants