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

Memoise read_config_file()? #166

Open
annakrystalli opened this issue Nov 20, 2024 · 0 comments
Open

Memoise read_config_file()? #166

annakrystalli opened this issue Nov 20, 2024 · 0 comments

Comments

@annakrystalli
Copy link
Member

The function read_config_file() can be called numerous times during validation. Although it's fast, should we nevertheless memoise it for efficiency?

It's a re-export from hubUtils so probably need to also memoise it there. In general, need to double check whether the fact it's a re-export requires any special handling.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Wishlist
Development

No branches or pull requests

1 participant