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

DOC: Update variables a and b to names consistent with comment documentation #60366

Open
1 task done
migelogali opened this issue Nov 20, 2024 · 1 comment · May be fixed by #60372
Open
1 task done

DOC: Update variables a and b to names consistent with comment documentation #60366

migelogali opened this issue Nov 20, 2024 · 1 comment · May be fixed by #60372
Assignees
Labels
Clean expressions pd.eval, query

Comments

@migelogali
Copy link

Pandas version checks

  • I have checked that the issue still exists on the latest versions of the docs on main here

Location of the documentation

https://github.com/pandas-dev/pandas/blob/main/pandas/core/computation/expressions.py

Documentation problem

Lines 234 and 235 explain what a and b are in detail (left and right operands), but there are many of those same variables earlier in the file, making it harder to understand what they represent.

Suggested fix for documentation

Assuming a and b represent right and left operands throughout each function, change these variable names to right_op and left_op instead throughout all functions to have more descriptive variable names

@migelogali migelogali added Docs Needs Triage Issue that has not been reviewed by a pandas team member labels Nov 20, 2024
@mingjji
Copy link

mingjji commented Nov 20, 2024

take

mingjji added a commit to mingjji/pandas that referenced this issue Nov 20, 2024
@rhshadrach rhshadrach added Clean expressions pd.eval, query and removed Docs Needs Triage Issue that has not been reviewed by a pandas team member labels Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Clean expressions pd.eval, query
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants