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

Consider renaming package to adhere to julia package naming conventions #8

Closed
dgkf-roche opened this issue Nov 24, 2023 · 3 comments · Fixed by #9
Closed

Consider renaming package to adhere to julia package naming conventions #8

dgkf-roche opened this issue Nov 24, 2023 · 3 comments · Fixed by #9
Assignees

Comments

@dgkf-roche
Copy link
Collaborator

Specifically this rule linked in the developer guide:

Package Naming Guidelines

[...]

  1. Err on the side of clarity, even if clarity seems long-winded to you.

RandomMatrices is a less ambiguous name than RndMat or RMT, even though the latter are shorter.


Sorry for jumping in with a nit-pick. It will take me some time to get up-to-speed with the project, just something that jumped out at me from the start.

@danielinteractive
Copy link
Collaborator

Thanks @dgkf-roche, good point. @brockk not sure if we could do BayesSafetySignalDetect or something like that?

@brockk
Copy link
Collaborator

brockk commented Nov 27, 2023

That's fine by me. I haven't submitted the abstract yet so can change it to whatever. Your suggestion is fine @danielinteractive

@brockk
Copy link
Collaborator

brockk commented Nov 28, 2023

Maybe SafetySignalDetection.jl
?

The Bayesian bit is probably OK to drop.

@brockk brockk closed this as completed in #9 Nov 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants