-
Notifications
You must be signed in to change notification settings - Fork 39
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
Use Julian style of code? #89
Comments
I would say that the only convention I am not using is the function names I will change this, you are right |
You are also not using the same style for the names of keyword arguments, which typically are |
I do find this pretty jarring too, changing it would simplify getting into the code (although it's your code, so you do what you want with it!). Also, indentation is off (julia packages usually have 4 spaces instead of tabs). https://github.com/domluna/JuliaFormatter.jl can possibly help (although I personally hate much of their decisions, so you have to select only the parts you want...) |
I will comply! I just have a bigger project to commit before making these changes. |
Hi, The last version should address most of this |
The dev is up to date:
https://bifurcationkit.github.io/BifurcationKitDocs.jl/dev/
I will tag a new version of the doc soon
… On Sep 23, 2023, at 10:47 PM, Carter Hinsley ***@***.***> wrote:
Hello, please note that the documentation does not reflect this change (neither docstrings nor the tutorial sample code online). This is confusing for new users, though not difficult to fix.
—
Reply to this email directly, view it on GitHub <#89 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAMEUBF3FPDFZCQOIZAPL7TX35DHDANCNFSM6AAAAAAWVIFWEY>.
You are receiving this because you commented.
|
Is it more up to the standards now? We have been working a lot on this. |
Feel free to re-open if you spot some non Julian choice |
I've noticed that most names/options in BifurcationKit.jl does not follow the Julia style guide when it comes to naming: https://docs.julialang.org/en/v1/manual/style-guide/#Use-naming-conventions-consistent-with-Julia-base/
If anything, it seems to be a style of a different language altogether, as there is no point where
snakeCase
should be used (according to the official guide). This raises a conceptual disconnect when using this package. Using the style guide comes with a lot of benefits, see e.g. slide number 11 here: https://github.com/JuliaDynamics/GoodScientificCodeWorkshop/blob/main/block2_clearcode/block2_slides.pdfIs there a solid reason that all keyword arguments are
snakeCase
instead oflow_case_with_underscores
? If not, perhaps it is worth considering changing these things in a 1.0 release.The text was updated successfully, but these errors were encountered: