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

Update documentation to avoid filters as flt, aggregations as aggs and spell them out instead #1638

Open
andeplane opened this issue Feb 21, 2024 · 1 comment
Assignees

Comments

@andeplane
Copy link
Contributor

Describe the bug
In many of the examples in the documentation, we have imports like ... filters as flt. It looks inspired from import matplotlib.pyplot as plt, but doesn't add much value imo, and seems like it might make AI confused in code generation (also me!)

I'd like to update the documentation to import them with filters and aggregations spelled out.

@andeplane andeplane added the bug label Feb 21, 2024
@andeplane andeplane self-assigned this Feb 21, 2024
@haakonvt
Copy link
Contributor

haakonvt commented Feb 21, 2024

@andeplane What about rather always importing them explicitly? I.e. change all to from ... import And, Equals

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants