You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug
The function _get_package_config is called several times under normal operation of the reflex run publish command. The _get_package_config makes an IO call to load the pyproject.toml file. While not a huge issue, this is unnecessary to reload the file multiple times.
Expected behavior
Typically, you only want to load a configuration file once and use the results. Or alternatively cache the results for future reference.
Screenshots
n/a
Specifics (please complete the following information):
Python Version: 3.12.2
Reflex Version: 0.5.0.post1
OS: macOS Sonoma 14.4.1
Browser (Optional): n/a
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered:
This change adds the functools cache decorator to avoid repeated
calls to _get_package_config, which results in IO hits to read
a file.
Closesreflex-dev#3341
Signed-off-by: Eric Brown <eric_wade_brown@yahoo.com>
This change adds the functools cache decorator to avoid repeated
calls to _get_package_config, which results in IO hits to read
a file.
Closesreflex-dev#3341
Signed-off-by: Eric Brown <eric_wade_brown@yahoo.com>
Describe the bug
The function
_get_package_config
is called several times under normal operation of thereflex run publish
command. The_get_package_config
makes an IO call to load the pyproject.toml file. While not a huge issue, this is unnecessary to reload the file multiple times.To Reproduce
Steps to reproduce the behavior:
_get_package_config
run reflex component publish -u saf -p asdf
Expected behavior
Typically, you only want to load a configuration file once and use the results. Or alternatively cache the results for future reference.
Screenshots
n/a
Specifics (please complete the following information):
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: