Use the new state name when setting is_hydrated to false
#3738
+1
−1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Nice catch, I think I missed this one because of the hidden templates folder. |
benedikt-bartscher
added a commit
to benedikt-bartscher/reflex
that referenced
this pull request
Aug 4, 2024
|
@benedikt-bartscher instead of templating state.js, i think we should just move the "main_state_name" to context.js.jinja2 and import it from state.js The main reason being that the complexity in state.js is already quite high and moving it to a jinja2 file means we lose intellisense and lenting capability when editing that file. |
picklelo
approved these changes
Aug 5, 2024
benedikt-bartscher
added a commit
to benedikt-bartscher/reflex
that referenced
this pull request
Aug 5, 2024
This reverts commit 4d2a72c.
cutoffthetop
referenced
this pull request
in robert-koch-institut/mex-editor
Aug 8, 2024
This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [reflex](https://reflex.dev) ([source](https://togithub.com/reflex-dev/reflex)) | project.dependencies | patch | `==0.5.8` -> `==0.5.9` | --- ### Release Notes <details> <summary>reflex-dev/reflex (reflex)</summary> ### [`v0.5.9`](https://togithub.com/reflex-dev/reflex/releases/tag/v0.5.9) [Compare Source](https://togithub.com/reflex-dev/reflex/compare/v0.5.8...v0.5.9) #### Bug Fixes - fix silly bug when style is set directly to breakpoints by [@​adhami3310](https://togithub.com/adhami3310) in [https://github.com/reflex-dev/reflex/pull/3719](https://togithub.com/reflex-dev/reflex/pull/3719) - Fix event actions like `stop_propagation` for recharts event triggers - \[REF-3203] Find a DOM event-like object in addEvents by [@​masenf](https://togithub.com/masenf) in [https://github.com/reflex-dev/reflex/pull/3706](https://togithub.com/reflex-dev/reflex/pull/3706) - fix initial_value for computed_var by [@​benedikt-bartscher](https://togithub.com/benedikt-bartscher) in [https://github.com/reflex-dev/reflex/pull/3726](https://togithub.com/reflex-dev/reflex/pull/3726) - add test for initial state dict by [@​benedikt-bartscher](https://togithub.com/benedikt-bartscher) in [https://github.com/reflex-dev/reflex/pull/3727](https://togithub.com/reflex-dev/reflex/pull/3727) - Improve Client-Side Routing on Page Not Found by [@​Manas1820](https://togithub.com/Manas1820) in [https://github.com/reflex-dev/reflex/pull/3723](https://togithub.com/reflex-dev/reflex/pull/3723) - Use the new state name when setting `is_hydrated` to false by [@​masenf](https://togithub.com/masenf) in [https://github.com/reflex-dev/reflex/pull/3738](https://togithub.com/reflex-dev/reflex/pull/3738) - Use `._is_mutable()` to account for parent state proxy by [@​masenf](https://togithub.com/masenf) in [https://github.com/reflex-dev/reflex/pull/3739](https://togithub.com/reflex-dev/reflex/pull/3739) #### Doc Updates - Add comments to html by [@​Alek99](https://togithub.com/Alek99) in [https://github.com/reflex-dev/reflex/pull/3731](https://togithub.com/reflex-dev/reflex/pull/3731) #### Var Refactor This release includes some new experimental features being developed incrementally in reflex.\_x.vars. - \[REF-3328] Implement **getitem** for ArrayVar by [@​adhami3310](https://togithub.com/adhami3310) in [https://github.com/reflex-dev/reflex/pull/3705](https://togithub.com/reflex-dev/reflex/pull/3705) - add type hinting to existing types by [@​adhami3310](https://togithub.com/adhami3310) in [https://github.com/reflex-dev/reflex/pull/3729](https://togithub.com/reflex-dev/reflex/pull/3729) **Full Changelog**: reflex-dev/reflex@v0.5.8...v0.5.9 </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://togithub.com/renovatebot/renovate). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC4xOC4xNyIsInVwZGF0ZWRJblZlciI6IjM4LjIzLjAiLCJ0YXJnZXRCcmFuY2giOiJtYWluIiwibGFiZWxzIjpbXX0=-->
benedikt-bartscher
pushed a commit
to benedikt-bartscher/reflex
that referenced
this pull request
Aug 11, 2024
cutoffthetop
referenced
this pull request
in robert-koch-institut/mex-drop
Aug 20, 2024
This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [reflex](https://reflex.dev) ([source](https://togithub.com/reflex-dev/reflex)) | project.dependencies | patch | `==0.5.8` -> `==0.5.9` | --- ### Release Notes <details> <summary>reflex-dev/reflex (reflex)</summary> ### [`v0.5.9`](https://togithub.com/reflex-dev/reflex/releases/tag/v0.5.9) [Compare Source](https://togithub.com/reflex-dev/reflex/compare/v0.5.8...v0.5.9) #### Bug Fixes - fix silly bug when style is set directly to breakpoints by [@​adhami3310](https://togithub.com/adhami3310) in [https://github.com/reflex-dev/reflex/pull/3719](https://togithub.com/reflex-dev/reflex/pull/3719) - Fix event actions like `stop_propagation` for recharts event triggers - \[REF-3203] Find a DOM event-like object in addEvents by [@​masenf](https://togithub.com/masenf) in [https://github.com/reflex-dev/reflex/pull/3706](https://togithub.com/reflex-dev/reflex/pull/3706) - fix initial_value for computed_var by [@​benedikt-bartscher](https://togithub.com/benedikt-bartscher) in [https://github.com/reflex-dev/reflex/pull/3726](https://togithub.com/reflex-dev/reflex/pull/3726) - add test for initial state dict by [@​benedikt-bartscher](https://togithub.com/benedikt-bartscher) in [https://github.com/reflex-dev/reflex/pull/3727](https://togithub.com/reflex-dev/reflex/pull/3727) - Improve Client-Side Routing on Page Not Found by [@​Manas1820](https://togithub.com/Manas1820) in [https://github.com/reflex-dev/reflex/pull/3723](https://togithub.com/reflex-dev/reflex/pull/3723) - Use the new state name when setting `is_hydrated` to false by [@​masenf](https://togithub.com/masenf) in [https://github.com/reflex-dev/reflex/pull/3738](https://togithub.com/reflex-dev/reflex/pull/3738) - Use `._is_mutable()` to account for parent state proxy by [@​masenf](https://togithub.com/masenf) in [https://github.com/reflex-dev/reflex/pull/3739](https://togithub.com/reflex-dev/reflex/pull/3739) #### Doc Updates - Add comments to html by [@​Alek99](https://togithub.com/Alek99) in [https://github.com/reflex-dev/reflex/pull/3731](https://togithub.com/reflex-dev/reflex/pull/3731) #### Var Refactor This release includes some new experimental features being developed incrementally in reflex.\_x.vars. - \[REF-3328] Implement **getitem** for ArrayVar by [@​adhami3310](https://togithub.com/adhami3310) in [https://github.com/reflex-dev/reflex/pull/3705](https://togithub.com/reflex-dev/reflex/pull/3705) - add type hinting to existing types by [@​adhami3310](https://togithub.com/adhami3310) in [https://github.com/reflex-dev/reflex/pull/3729](https://togithub.com/reflex-dev/reflex/pull/3729) **Full Changelog**: reflex-dev/reflex@v0.5.8...v0.5.9 </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://togithub.com/renovatebot/renovate). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC40NC4wIiwidXBkYXRlZEluVmVyIjoiMzguNDQuMCIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOltdfQ==-->
masenf
added a commit
that referenced
this pull request
Aug 22, 2024
* Add comments to html metadata component (#3731) * fix: add verification for path /404 (#3723) Co-authored-by: coolstorm <[email protected]> * Use the new state name when setting `is_hydrated` to false (#3738) * Use `._is_mutable()` to account for parent state proxy (#3739) When a parent state proxy is set, also allow child StateProxy._self_mutable to override the parent's `_is_mutable()`. * bump to 0.5.9 (#3746) * add message when installing requirements.txt is needed for chosen template during init (#3750) * #3752 bugfix add domain for XAxis (#3764) * fix appharness app_source typing (#3777) * fix import clash between connectionToaster and hooks.useState (#3749) * use different registry when in china, fixes #3700 (#3702) * do not reload compilation if using local app in AppHarness (#3790) * do not reload if using local app * Update reflex/testing.py Co-authored-by: Masen Furer <[email protected]> --------- Co-authored-by: Masen Furer <[email protected]> * Bump memory on relevant actions (#3781) Co-authored-by: Alek Petuskey <[email protected]> * [REF-3334] Validate Toast Props (#3793) * [REF-3536][REF-3537][REF-3541] Move chakra components into its repo(reflex-chakra) (#3798) * fix get_uuid_string_var (#3795) * minor State cleanup (#3768) * Fix code wrap in markdown (#3755) --------- Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Manas Gupta <[email protected]> Co-authored-by: coolstorm <[email protected]> Co-authored-by: Thomas Brandého <[email protected]> Co-authored-by: Shubhankar Dimri <[email protected]> Co-authored-by: benedikt-bartscher <[email protected]> Co-authored-by: Khaleel Al-Adhami <[email protected]> Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Elijah Ahianyo <[email protected]>
masenf
added a commit
that referenced
this pull request
Aug 26, 2024
* fully migrate vars into new system * i hate rufffff (no i don't) * fix silly pright issues (except colormode and state) * remove all instances of Var.create * create immutable callable var and get rid of more base vars * implement hash for all functions * get reflex-web to compile * get it to compile reflex-web successfully * fix tests * fix pyi * use override from typing_extension * put plotly inside of a catch * dicts are unusable sadly * fix silly mistake * overload equals to special case immutable var * improve test_cond * solve more CI issues, down to 94 failures * down to 20 errors * down to 13 errors * pass all testcases * fix pyright issues * reorder things * use get origin more * use fixed_type logic * various optimizations * go back to passing test cases * use less boilerplate * remove unnecessary print message * remove weird comment * add test for html issue * add type ignore * fix another silly issue * override get all var data for var operations call * make integration tests pass * fix immutable call var * better logic for finding parent class * use even better logic for finding state wrt computedvar * only choose the ones that are defined in the same module * small dict to large dict * [REF-3591] Remove chakra-related files from immutable vars PR (#3821) * Add comments to html metadata component (#3731) * fix: add verification for path /404 (#3723) Co-authored-by: coolstorm <[email protected]> * Use the new state name when setting `is_hydrated` to false (#3738) * Use `._is_mutable()` to account for parent state proxy (#3739) When a parent state proxy is set, also allow child StateProxy._self_mutable to override the parent's `_is_mutable()`. * bump to 0.5.9 (#3746) * add message when installing requirements.txt is needed for chosen template during init (#3750) * #3752 bugfix add domain for XAxis (#3764) * fix appharness app_source typing (#3777) * fix import clash between connectionToaster and hooks.useState (#3749) * use different registry when in china, fixes #3700 (#3702) * do not reload compilation if using local app in AppHarness (#3790) * do not reload if using local app * Update reflex/testing.py Co-authored-by: Masen Furer <[email protected]> --------- Co-authored-by: Masen Furer <[email protected]> * Bump memory on relevant actions (#3781) Co-authored-by: Alek Petuskey <[email protected]> * [REF-3334] Validate Toast Props (#3793) * [REF-3536][REF-3537][REF-3541] Move chakra components into its repo(reflex-chakra) (#3798) * fix get_uuid_string_var (#3795) * minor State cleanup (#3768) * Fix code wrap in markdown (#3755) --------- Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Manas Gupta <[email protected]> Co-authored-by: coolstorm <[email protected]> Co-authored-by: Thomas Brandého <[email protected]> Co-authored-by: Shubhankar Dimri <[email protected]> Co-authored-by: benedikt-bartscher <[email protected]> Co-authored-by: Khaleel Al-Adhami <[email protected]> Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Elijah Ahianyo <[email protected]> * pyproject.toml: bump to 0.6.0a1 * pyproject.toml: depend on reflex-chakra>=0.6.0a New Var system support in reflex-chakra 0.6.0a1 * poetry.lock: relock dependencies * integration: bump listening timeout to 1200 seconds * integration: bump listening timeout to 1800 seconds * Use cached_var_no_lock to avoid ImmutableVar deadlocks (#3835) * Use cached_var_no_lock to avoid ImmutableVar deadlocks ImmutableVar subclasses will always return the same value for a _var_name or _get_all_var_data so there is no need to use a per-class lock to protect a cached attribute on an instance, and doing so actually is observed to cause deadlocks when a particular _cached_var_name creates new LiteralVar instances and attempts to serialize them. * remove unused module global --------- Co-authored-by: Masen Furer <[email protected]> Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Manas Gupta <[email protected]> Co-authored-by: coolstorm <[email protected]> Co-authored-by: Thomas Brandého <[email protected]> Co-authored-by: Shubhankar Dimri <[email protected]> Co-authored-by: benedikt-bartscher <[email protected]> Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Elijah Ahianyo <[email protected]>
adhami3310
added a commit
that referenced
this pull request
Aug 27, 2024
* fully migrate vars into new system * i hate rufffff (no i don't) * fix silly pright issues (except colormode and state) * remove all instances of Var.create * create immutable callable var and get rid of more base vars * implement hash for all functions * get reflex-web to compile * get it to compile reflex-web successfully * fix tests * fix pyi * use override from typing_extension * put plotly inside of a catch * dicts are unusable sadly * fix silly mistake * overload equals to special case immutable var * improve test_cond * solve more CI issues, down to 94 failures * down to 20 errors * down to 13 errors * pass all testcases * fix pyright issues * reorder things * use get origin more * use fixed_type logic * various optimizations * go back to passing test cases * use less boilerplate * remove unnecessary print message * remove weird comment * add test for html issue * add type ignore * fix another silly issue * override get all var data for var operations call * make integration tests pass * fix immutable call var * better logic for finding parent class * use even better logic for finding state wrt computedvar * only choose the ones that are defined in the same module * small dict to large dict * [REF-3591] Remove chakra-related files from immutable vars PR (#3821) * Add comments to html metadata component (#3731) * fix: add verification for path /404 (#3723) Co-authored-by: coolstorm <[email protected]> * Use the new state name when setting `is_hydrated` to false (#3738) * Use `._is_mutable()` to account for parent state proxy (#3739) When a parent state proxy is set, also allow child StateProxy._self_mutable to override the parent's `_is_mutable()`. * bump to 0.5.9 (#3746) * add message when installing requirements.txt is needed for chosen template during init (#3750) * #3752 bugfix add domain for XAxis (#3764) * fix appharness app_source typing (#3777) * fix import clash between connectionToaster and hooks.useState (#3749) * use different registry when in china, fixes #3700 (#3702) * do not reload compilation if using local app in AppHarness (#3790) * do not reload if using local app * Update reflex/testing.py Co-authored-by: Masen Furer <[email protected]> --------- Co-authored-by: Masen Furer <[email protected]> * Bump memory on relevant actions (#3781) Co-authored-by: Alek Petuskey <[email protected]> * [REF-3334] Validate Toast Props (#3793) * [REF-3536][REF-3537][REF-3541] Move chakra components into its repo(reflex-chakra) (#3798) * fix get_uuid_string_var (#3795) * minor State cleanup (#3768) * Fix code wrap in markdown (#3755) --------- Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Manas Gupta <[email protected]> Co-authored-by: coolstorm <[email protected]> Co-authored-by: Thomas Brandého <[email protected]> Co-authored-by: Shubhankar Dimri <[email protected]> Co-authored-by: benedikt-bartscher <[email protected]> Co-authored-by: Khaleel Al-Adhami <[email protected]> Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Elijah Ahianyo <[email protected]> * pyproject.toml: bump to 0.6.0a1 * pyproject.toml: depend on reflex-chakra>=0.6.0a New Var system support in reflex-chakra 0.6.0a1 * poetry.lock: relock dependencies * integration: bump listening timeout to 1200 seconds * integration: bump listening timeout to 1800 seconds * Use cached_var_no_lock to avoid ImmutableVar deadlocks (#3835) * Use cached_var_no_lock to avoid ImmutableVar deadlocks ImmutableVar subclasses will always return the same value for a _var_name or _get_all_var_data so there is no need to use a per-class lock to protect a cached attribute on an instance, and doing so actually is observed to cause deadlocks when a particular _cached_var_name creates new LiteralVar instances and attempts to serialize them. * remove unused module global --------- Co-authored-by: Masen Furer <[email protected]> Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Manas Gupta <[email protected]> Co-authored-by: coolstorm <[email protected]> Co-authored-by: Thomas Brandého <[email protected]> Co-authored-by: Shubhankar Dimri <[email protected]> Co-authored-by: benedikt-bartscher <[email protected]> Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Elijah Ahianyo <[email protected]>
adhami3310
added a commit
that referenced
this pull request
Aug 27, 2024
* fully migrate vars into new system * i hate rufffff (no i don't) * fix silly pright issues (except colormode and state) * remove all instances of Var.create * create immutable callable var and get rid of more base vars * implement hash for all functions * get reflex-web to compile * get it to compile reflex-web successfully * fix tests * fix pyi * use override from typing_extension * put plotly inside of a catch * dicts are unusable sadly * fix silly mistake * overload equals to special case immutable var * improve test_cond * solve more CI issues, down to 94 failures * down to 20 errors * down to 13 errors * pass all testcases * fix pyright issues * reorder things * use get origin more * use fixed_type logic * various optimizations * go back to passing test cases * use less boilerplate * remove unnecessary print message * remove weird comment * add test for html issue * add type ignore * fix another silly issue * override get all var data for var operations call * make integration tests pass * fix immutable call var * better logic for finding parent class * use even better logic for finding state wrt computedvar * only choose the ones that are defined in the same module * small dict to large dict * [REF-3591] Remove chakra-related files from immutable vars PR (#3821) * Add comments to html metadata component (#3731) * fix: add verification for path /404 (#3723) Co-authored-by: coolstorm <[email protected]> * Use the new state name when setting `is_hydrated` to false (#3738) * Use `._is_mutable()` to account for parent state proxy (#3739) When a parent state proxy is set, also allow child StateProxy._self_mutable to override the parent's `_is_mutable()`. * bump to 0.5.9 (#3746) * add message when installing requirements.txt is needed for chosen template during init (#3750) * #3752 bugfix add domain for XAxis (#3764) * fix appharness app_source typing (#3777) * fix import clash between connectionToaster and hooks.useState (#3749) * use different registry when in china, fixes #3700 (#3702) * do not reload compilation if using local app in AppHarness (#3790) * do not reload if using local app * Update reflex/testing.py Co-authored-by: Masen Furer <[email protected]> --------- Co-authored-by: Masen Furer <[email protected]> * Bump memory on relevant actions (#3781) Co-authored-by: Alek Petuskey <[email protected]> * [REF-3334] Validate Toast Props (#3793) * [REF-3536][REF-3537][REF-3541] Move chakra components into its repo(reflex-chakra) (#3798) * fix get_uuid_string_var (#3795) * minor State cleanup (#3768) * Fix code wrap in markdown (#3755) --------- Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Manas Gupta <[email protected]> Co-authored-by: coolstorm <[email protected]> Co-authored-by: Thomas Brandého <[email protected]> Co-authored-by: Shubhankar Dimri <[email protected]> Co-authored-by: benedikt-bartscher <[email protected]> Co-authored-by: Khaleel Al-Adhami <[email protected]> Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Elijah Ahianyo <[email protected]> * pyproject.toml: bump to 0.6.0a1 * pyproject.toml: depend on reflex-chakra>=0.6.0a New Var system support in reflex-chakra 0.6.0a1 * poetry.lock: relock dependencies * integration: bump listening timeout to 1200 seconds * integration: bump listening timeout to 1800 seconds * Use cached_var_no_lock to avoid ImmutableVar deadlocks (#3835) * Use cached_var_no_lock to avoid ImmutableVar deadlocks ImmutableVar subclasses will always return the same value for a _var_name or _get_all_var_data so there is no need to use a per-class lock to protect a cached attribute on an instance, and doing so actually is observed to cause deadlocks when a particular _cached_var_name creates new LiteralVar instances and attempts to serialize them. * remove unused module global --------- Co-authored-by: Masen Furer <[email protected]> Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Manas Gupta <[email protected]> Co-authored-by: coolstorm <[email protected]> Co-authored-by: Thomas Brandého <[email protected]> Co-authored-by: Shubhankar Dimri <[email protected]> Co-authored-by: benedikt-bartscher <[email protected]> Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Elijah Ahianyo <[email protected]>
adhami3310
added a commit
that referenced
this pull request
Aug 27, 2024
* fully migrate vars into new system * i hate rufffff (no i don't) * fix silly pright issues (except colormode and state) * remove all instances of Var.create * create immutable callable var and get rid of more base vars * implement hash for all functions * get reflex-web to compile * get it to compile reflex-web successfully * fix tests * fix pyi * use override from typing_extension * put plotly inside of a catch * dicts are unusable sadly * fix silly mistake * overload equals to special case immutable var * improve test_cond * solve more CI issues, down to 94 failures * down to 20 errors * down to 13 errors * pass all testcases * fix pyright issues * reorder things * use get origin more * use fixed_type logic * various optimizations * go back to passing test cases * use less boilerplate * remove unnecessary print message * remove weird comment * add test for html issue * add type ignore * fix another silly issue * override get all var data for var operations call * make integration tests pass * fix immutable call var * better logic for finding parent class * use even better logic for finding state wrt computedvar * only choose the ones that are defined in the same module * small dict to large dict * [REF-3591] Remove chakra-related files from immutable vars PR (#3821) * Add comments to html metadata component (#3731) * fix: add verification for path /404 (#3723) Co-authored-by: coolstorm <[email protected]> * Use the new state name when setting `is_hydrated` to false (#3738) * Use `._is_mutable()` to account for parent state proxy (#3739) When a parent state proxy is set, also allow child StateProxy._self_mutable to override the parent's `_is_mutable()`. * bump to 0.5.9 (#3746) * add message when installing requirements.txt is needed for chosen template during init (#3750) * #3752 bugfix add domain for XAxis (#3764) * fix appharness app_source typing (#3777) * fix import clash between connectionToaster and hooks.useState (#3749) * use different registry when in china, fixes #3700 (#3702) * do not reload compilation if using local app in AppHarness (#3790) * do not reload if using local app * Update reflex/testing.py Co-authored-by: Masen Furer <[email protected]> --------- Co-authored-by: Masen Furer <[email protected]> * Bump memory on relevant actions (#3781) Co-authored-by: Alek Petuskey <[email protected]> * [REF-3334] Validate Toast Props (#3793) * [REF-3536][REF-3537][REF-3541] Move chakra components into its repo(reflex-chakra) (#3798) * fix get_uuid_string_var (#3795) * minor State cleanup (#3768) * Fix code wrap in markdown (#3755) --------- Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Manas Gupta <[email protected]> Co-authored-by: coolstorm <[email protected]> Co-authored-by: Thomas Brandého <[email protected]> Co-authored-by: Shubhankar Dimri <[email protected]> Co-authored-by: benedikt-bartscher <[email protected]> Co-authored-by: Khaleel Al-Adhami <[email protected]> Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Elijah Ahianyo <[email protected]> * pyproject.toml: bump to 0.6.0a1 * pyproject.toml: depend on reflex-chakra>=0.6.0a New Var system support in reflex-chakra 0.6.0a1 * poetry.lock: relock dependencies * integration: bump listening timeout to 1200 seconds * integration: bump listening timeout to 1800 seconds * Use cached_var_no_lock to avoid ImmutableVar deadlocks (#3835) * Use cached_var_no_lock to avoid ImmutableVar deadlocks ImmutableVar subclasses will always return the same value for a _var_name or _get_all_var_data so there is no need to use a per-class lock to protect a cached attribute on an instance, and doing so actually is observed to cause deadlocks when a particular _cached_var_name creates new LiteralVar instances and attempts to serialize them. * remove unused module global --------- Co-authored-by: Masen Furer <[email protected]> Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Manas Gupta <[email protected]> Co-authored-by: coolstorm <[email protected]> Co-authored-by: Thomas Brandého <[email protected]> Co-authored-by: Shubhankar Dimri <[email protected]> Co-authored-by: benedikt-bartscher <[email protected]> Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Elijah Ahianyo <[email protected]>
adhami3310
added a commit
that referenced
this pull request
Oct 25, 2024
* basic functionality * reorder page evaluation * Fix benchmark tests (#3822) * Upper bound for reflex-chakra dependency (#3824) * Upper bound for reflex-chakra dependency We have to make a breaking change in reflex-chakra, and it will happen in 0.6.0, so ensure that reflex 0.5.10 never installs the breaking version of reflex-chakra. * Relock deps * Add comments to DataList components (#3827) * fix: Adding missing comments to the data list * fix: Ran make pyi * fully migrate vars into new system (#3743) * fully migrate vars into new system * i hate rufffff (no i don't) * fix silly pright issues (except colormode and state) * remove all instances of Var.create * create immutable callable var and get rid of more base vars * implement hash for all functions * get reflex-web to compile * get it to compile reflex-web successfully * fix tests * fix pyi * use override from typing_extension * put plotly inside of a catch * dicts are unusable sadly * fix silly mistake * overload equals to special case immutable var * improve test_cond * solve more CI issues, down to 94 failures * down to 20 errors * down to 13 errors * pass all testcases * fix pyright issues * reorder things * use get origin more * use fixed_type logic * various optimizations * go back to passing test cases * use less boilerplate * remove unnecessary print message * remove weird comment * add test for html issue * add type ignore * fix another silly issue * override get all var data for var operations call * make integration tests pass * fix immutable call var * better logic for finding parent class * use even better logic for finding state wrt computedvar * only choose the ones that are defined in the same module * small dict to large dict * [REF-3591] Remove chakra-related files from immutable vars PR (#3821) * Add comments to html metadata component (#3731) * fix: add verification for path /404 (#3723) Co-authored-by: coolstorm <[email protected]> * Use the new state name when setting `is_hydrated` to false (#3738) * Use `._is_mutable()` to account for parent state proxy (#3739) When a parent state proxy is set, also allow child StateProxy._self_mutable to override the parent's `_is_mutable()`. * bump to 0.5.9 (#3746) * add message when installing requirements.txt is needed for chosen template during init (#3750) * #3752 bugfix add domain for XAxis (#3764) * fix appharness app_source typing (#3777) * fix import clash between connectionToaster and hooks.useState (#3749) * use different registry when in china, fixes #3700 (#3702) * do not reload compilation if using local app in AppHarness (#3790) * do not reload if using local app * Update reflex/testing.py Co-authored-by: Masen Furer <[email protected]> --------- Co-authored-by: Masen Furer <[email protected]> * Bump memory on relevant actions (#3781) Co-authored-by: Alek Petuskey <[email protected]> * [REF-3334] Validate Toast Props (#3793) * [REF-3536][REF-3537][REF-3541] Move chakra components into its repo(reflex-chakra) (#3798) * fix get_uuid_string_var (#3795) * minor State cleanup (#3768) * Fix code wrap in markdown (#3755) --------- Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Manas Gupta <[email protected]> Co-authored-by: coolstorm <[email protected]> Co-authored-by: Thomas Brandého <[email protected]> Co-authored-by: Shubhankar Dimri <[email protected]> Co-authored-by: benedikt-bartscher <[email protected]> Co-authored-by: Khaleel Al-Adhami <[email protected]> Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Elijah Ahianyo <[email protected]> * pyproject.toml: bump to 0.6.0a1 * pyproject.toml: depend on reflex-chakra>=0.6.0a New Var system support in reflex-chakra 0.6.0a1 * poetry.lock: relock dependencies * integration: bump listening timeout to 1200 seconds * integration: bump listening timeout to 1800 seconds * Use cached_var_no_lock to avoid ImmutableVar deadlocks (#3835) * Use cached_var_no_lock to avoid ImmutableVar deadlocks ImmutableVar subclasses will always return the same value for a _var_name or _get_all_var_data so there is no need to use a per-class lock to protect a cached attribute on an instance, and doing so actually is observed to cause deadlocks when a particular _cached_var_name creates new LiteralVar instances and attempts to serialize them. * remove unused module global --------- Co-authored-by: Masen Furer <[email protected]> Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Manas Gupta <[email protected]> Co-authored-by: coolstorm <[email protected]> Co-authored-by: Thomas Brandého <[email protected]> Co-authored-by: Shubhankar Dimri <[email protected]> Co-authored-by: benedikt-bartscher <[email protected]> Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Elijah Ahianyo <[email protected]> * guess_type: if the type is optional, treat it like it's "not None" (#3839) * guess_type: if the type is optional, treat it like it's "not None" When guessing the type for an Optional annotation, the Optional was already being stripped off, but this value was being ignored, except for error messages. So actually use the Optional-stripped value. * Strip Optional when casting Var .to * Fix double-quoting of defaultColorMode (#3840) If the user provided an `appearance` or `color_mode` value to `rx.theme`, then the resulting value ended up being double-double-quoted in the resulting JS output. Instead remove the quotes from the context.js.jinja2 and always pass appearance as a Var. * basic functionality * run pyright * fully migrate vars into new system (#3743) * fully migrate vars into new system * i hate rufffff (no i don't) * fix silly pright issues (except colormode and state) * remove all instances of Var.create * create immutable callable var and get rid of more base vars * implement hash for all functions * get reflex-web to compile * get it to compile reflex-web successfully * fix tests * fix pyi * use override from typing_extension * put plotly inside of a catch * dicts are unusable sadly * fix silly mistake * overload equals to special case immutable var * improve test_cond * solve more CI issues, down to 94 failures * down to 20 errors * down to 13 errors * pass all testcases * fix pyright issues * reorder things * use get origin more * use fixed_type logic * various optimizations * go back to passing test cases * use less boilerplate * remove unnecessary print message * remove weird comment * add test for html issue * add type ignore * fix another silly issue * override get all var data for var operations call * make integration tests pass * fix immutable call var * better logic for finding parent class * use even better logic for finding state wrt computedvar * only choose the ones that are defined in the same module * small dict to large dict * [REF-3591] Remove chakra-related files from immutable vars PR (#3821) * Add comments to html metadata component (#3731) * fix: add verification for path /404 (#3723) Co-authored-by: coolstorm <[email protected]> * Use the new state name when setting `is_hydrated` to false (#3738) * Use `._is_mutable()` to account for parent state proxy (#3739) When a parent state proxy is set, also allow child StateProxy._self_mutable to override the parent's `_is_mutable()`. * bump to 0.5.9 (#3746) * add message when installing requirements.txt is needed for chosen template during init (#3750) * #3752 bugfix add domain for XAxis (#3764) * fix appharness app_source typing (#3777) * fix import clash between connectionToaster and hooks.useState (#3749) * use different registry when in china, fixes #3700 (#3702) * do not reload compilation if using local app in AppHarness (#3790) * do not reload if using local app * Update reflex/testing.py Co-authored-by: Masen Furer <[email protected]> --------- Co-authored-by: Masen Furer <[email protected]> * Bump memory on relevant actions (#3781) Co-authored-by: Alek Petuskey <[email protected]> * [REF-3334] Validate Toast Props (#3793) * [REF-3536][REF-3537][REF-3541] Move chakra components into its repo(reflex-chakra) (#3798) * fix get_uuid_string_var (#3795) * minor State cleanup (#3768) * Fix code wrap in markdown (#3755) --------- Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Manas Gupta <[email protected]> Co-authored-by: coolstorm <[email protected]> Co-authored-by: Thomas Brandého <[email protected]> Co-authored-by: Shubhankar Dimri <[email protected]> Co-authored-by: benedikt-bartscher <[email protected]> Co-authored-by: Khaleel Al-Adhami <[email protected]> Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Elijah Ahianyo <[email protected]> * pyproject.toml: bump to 0.6.0a1 * pyproject.toml: depend on reflex-chakra>=0.6.0a New Var system support in reflex-chakra 0.6.0a1 * poetry.lock: relock dependencies * integration: bump listening timeout to 1200 seconds * integration: bump listening timeout to 1800 seconds * Use cached_var_no_lock to avoid ImmutableVar deadlocks (#3835) * Use cached_var_no_lock to avoid ImmutableVar deadlocks ImmutableVar subclasses will always return the same value for a _var_name or _get_all_var_data so there is no need to use a per-class lock to protect a cached attribute on an instance, and doing so actually is observed to cause deadlocks when a particular _cached_var_name creates new LiteralVar instances and attempts to serialize them. * remove unused module global --------- Co-authored-by: Masen Furer <[email protected]> Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Manas Gupta <[email protected]> Co-authored-by: coolstorm <[email protected]> Co-authored-by: Thomas Brandého <[email protected]> Co-authored-by: Shubhankar Dimri <[email protected]> Co-authored-by: benedikt-bartscher <[email protected]> Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Elijah Ahianyo <[email protected]> * basic functionality * reorder page evaluation * fully migrate vars into new system (#3743) * fully migrate vars into new system * i hate rufffff (no i don't) * fix silly pright issues (except colormode and state) * remove all instances of Var.create * create immutable callable var and get rid of more base vars * implement hash for all functions * get reflex-web to compile * get it to compile reflex-web successfully * fix tests * fix pyi * use override from typing_extension * put plotly inside of a catch * dicts are unusable sadly * fix silly mistake * overload equals to special case immutable var * improve test_cond * solve more CI issues, down to 94 failures * down to 20 errors * down to 13 errors * pass all testcases * fix pyright issues * reorder things * use get origin more * use fixed_type logic * various optimizations * go back to passing test cases * use less boilerplate * remove unnecessary print message * remove weird comment * add test for html issue * add type ignore * fix another silly issue * override get all var data for var operations call * make integration tests pass * fix immutable call var * better logic for finding parent class * use even better logic for finding state wrt computedvar * only choose the ones that are defined in the same module * small dict to large dict * [REF-3591] Remove chakra-related files from immutable vars PR (#3821) * Add comments to html metadata component (#3731) * fix: add verification for path /404 (#3723) Co-authored-by: coolstorm <[email protected]> * Use the new state name when setting `is_hydrated` to false (#3738) * Use `._is_mutable()` to account for parent state proxy (#3739) When a parent state proxy is set, also allow child StateProxy._self_mutable to override the parent's `_is_mutable()`. * bump to 0.5.9 (#3746) * add message when installing requirements.txt is needed for chosen template during init (#3750) * #3752 bugfix add domain for XAxis (#3764) * fix appharness app_source typing (#3777) * fix import clash between connectionToaster and hooks.useState (#3749) * use different registry when in china, fixes #3700 (#3702) * do not reload compilation if using local app in AppHarness (#3790) * do not reload if using local app * Update reflex/testing.py Co-authored-by: Masen Furer <[email protected]> --------- Co-authored-by: Masen Furer <[email protected]> * Bump memory on relevant actions (#3781) Co-authored-by: Alek Petuskey <[email protected]> * [REF-3334] Validate Toast Props (#3793) * [REF-3536][REF-3537][REF-3541] Move chakra components into its repo(reflex-chakra) (#3798) * fix get_uuid_string_var (#3795) * minor State cleanup (#3768) * Fix code wrap in markdown (#3755) --------- Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Manas Gupta <[email protected]> Co-authored-by: coolstorm <[email protected]> Co-authored-by: Thomas Brandého <[email protected]> Co-authored-by: Shubhankar Dimri <[email protected]> Co-authored-by: benedikt-bartscher <[email protected]> Co-authored-by: Khaleel Al-Adhami <[email protected]> Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Elijah Ahianyo <[email protected]> * pyproject.toml: bump to 0.6.0a1 * pyproject.toml: depend on reflex-chakra>=0.6.0a New Var system support in reflex-chakra 0.6.0a1 * poetry.lock: relock dependencies * integration: bump listening timeout to 1200 seconds * integration: bump listening timeout to 1800 seconds * Use cached_var_no_lock to avoid ImmutableVar deadlocks (#3835) * Use cached_var_no_lock to avoid ImmutableVar deadlocks ImmutableVar subclasses will always return the same value for a _var_name or _get_all_var_data so there is no need to use a per-class lock to protect a cached attribute on an instance, and doing so actually is observed to cause deadlocks when a particular _cached_var_name creates new LiteralVar instances and attempts to serialize them. * remove unused module global --------- Co-authored-by: Masen Furer <[email protected]> Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Manas Gupta <[email protected]> Co-authored-by: coolstorm <[email protected]> Co-authored-by: Thomas Brandého <[email protected]> Co-authored-by: Shubhankar Dimri <[email protected]> Co-authored-by: benedikt-bartscher <[email protected]> Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Elijah Ahianyo <[email protected]> * guess_type: if the type is optional, treat it like it's "not None" (#3839) * guess_type: if the type is optional, treat it like it's "not None" When guessing the type for an Optional annotation, the Optional was already being stripped off, but this value was being ignored, except for error messages. So actually use the Optional-stripped value. * Strip Optional when casting Var .to * run format * use original mp * evaluate page regardless * use old typing * dangit pydantic * i have two braincells and they are NOT collaborating * adjust testcases * always add the upload endpoint * retrieve upload but after evaluate component * check against none Co-authored-by: Masen Furer <[email protected]> * make it var * fix page title * don't change style.py * fix counter * remove duplicated logic --------- Co-authored-by: Elijah Ahianyo <[email protected]> Co-authored-by: Masen Furer <[email protected]> Co-authored-by: elvis kahoro <[email protected]> Co-authored-by: Alek Petuskey <[email protected]> Co-authored-by: Manas Gupta <[email protected]> Co-authored-by: coolstorm <[email protected]> Co-authored-by: Thomas Brandého <[email protected]> Co-authored-by: Shubhankar Dimri <[email protected]> Co-authored-by: benedikt-bartscher <[email protected]> Co-authored-by: Alek Petuskey <[email protected]>
benedikt-bartscher
added a commit
to benedikt-bartscher/reflex
that referenced
this pull request
Nov 1, 2024
benedikt-bartscher
added a commit
to benedikt-bartscher/reflex
that referenced
this pull request
Nov 1, 2024
This reverts commit 4d2a72c.
benedikt-bartscher
added a commit
to benedikt-bartscher/reflex
that referenced
this pull request
Nov 1, 2024
benedikt-bartscher
added a commit
to benedikt-bartscher/reflex
that referenced
this pull request
Nov 1, 2024
This reverts commit 4d2a72c.
masenf
pushed a commit
to benedikt-bartscher/reflex
that referenced
this pull request
Nov 5, 2024
masenf
pushed a commit
to benedikt-bartscher/reflex
that referenced
this pull request
Nov 5, 2024
This reverts commit 4d2a72c.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This change ensures that
is_hydratedis reset tofalsewhen navigating between pages to avoid the "flicker" that shows incorrect data during navigation events.