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

Code styling consistency #252

Open
anilyil opened this issue Dec 15, 2022 · 0 comments
Open

Code styling consistency #252

anilyil opened this issue Dec 15, 2022 · 0 comments

Comments

@anilyil
Copy link
Contributor

anilyil commented Dec 15, 2022

Description of feature

Most of ADflow is in camelCase, but it is not completely consistent. I personally don't prefer using camelCase for all variables since it makes it difficult to read in my opinion. I also am not sure if this is consistent with pep8.

In #231 I introduced some changes that uses an underscore based naming change. I changed all my changes to camelCase in the python changes but left the fortran changes as is for now. Creating this issue so we can further discuss with @eirikurj.

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

No branches or pull requests

1 participant