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

[DISCUSSION] Naming - UDFs, App method names, DataFrame names, and filters. #468

Open
ruebot opened this issue May 27, 2020 · 3 comments
Open
Assignees
Labels

Comments

@ruebot
Copy link
Member

@ruebot ruebot commented May 27, 2020

Let's evaluate how we've named everything; UDFs, App method names, DataFrame names, and filters.

Are we all good with how we've named everything?

Do we need to change anything before a 1.0.0 release?

See also #467

@lintool
Copy link
Member

@lintool lintool commented May 29, 2020

My thoughts: In Python, I would default to PEP8 unless there's a convincing reason otherwise: https://www.python.org/dev/peps/pep-0008/

Apps are most like Classes, so I would go CamelCase. Note, it says:

Note: When using acronyms in CapWords, capitalize all the letters of the acronym. Thus HTTPServerError is better than HttpServerError.

UDFs are like methods, so should follow standard snake_case.

@ruebot
Copy link
Member Author

@ruebot ruebot commented May 29, 2020

@lintool anything on the Scala side? If not, I'll get some PRs ready for aut, documentation, and notebooks.

@ruebot
Copy link
Member Author

@ruebot ruebot commented May 29, 2020

  • ExtractPopularImages
  • WriteGexf
  • WriteGraphml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
1.0.0 Release of AUT
  
In Progress
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.