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

Vector instead of Dict performance #5

Open
chakravala opened this issue Dec 9, 2022 · 1 comment
Open

Vector instead of Dict performance #5

chakravala opened this issue Dec 9, 2022 · 1 comment

Comments

@chakravala
Copy link

if you're using sequential integers as keys, you might as well use a Vector with sequential integer indices, as that has faster

https://github.com/Eben60/Mendeleev.jl/blob/main/src/data.jl/ionization_data.jl

@Eben60
Copy link
Owner

Eben60 commented Dec 9, 2022

chakravala, thank you for looking onto my source code - I know you are a highly competent Julia developer.

I must however say, I didn't at all try to optimize the package for the speed, not even tested. Actually I can't imagine any use case where the speed can be relevant.

So while in principle you are right, I'd in the moment leave it as it is. I'd also leave the issue open, but as low priority for the code refactoring some day.

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

2 participants