Ryan Morshead

Results 319 comments of Ryan Morshead
trafficstars

@sydney-runkle and @samuelcolvin, if we think modifying `is-instance` schemas is a reasonable solution to this issue, perhaps we could begin conservatively by defining a public function like ```python def adapt_is_instance_schemas(...

Is there any chance that some of these changes could be enabled via feature flags? I'm particularly interested in https://github.com/python-poetry/poetry/issues/3248 but this could be generally useful for easing the community...

If we could take a similar approach to simplifying the flask/tornado backends that would be good too. If not, doesn't seem necessary. Regardless, probably should be done in a separate...

At some point we need to figure out async effects. I don't think that should be a blocker here. Edit: see https://github.com/reactive-python/reactpy/pull/1090

Relevant code is [here](https://github.com/reactive-python/reactpy/blob/main/src/js/packages/%40reactpy/client/src/components.tsx#L178-L220). What's probably happening is that the element is getting recreated. With that said, despite the fact that `render()` on the custom component is [always called](https://github.com/reactive-python/reactpy/blob/main/src/js/packages/%40reactpy/client/src/components.tsx#L208-L217), subsequent...

On a separate note, we might want to add a method to custom components (perhaps `mount()`) that's called exactly once when the custom component is first rendered to a given...

Related: https://github.com/reactive-python/reactpy/discussions/1098#discussioncomment-6442720

I think I'm going to try deploying on GCP and will prioritize this if I think this will make it possible/easier to do so. A lot of people seem to...

> concatenate function in the newer Python typing libraries might be able to be jerry-rigged to help here? Unfortunately `Concatenate` does not help. The PEP [explicitly rejects concatenating keyword arguments](https://peps.python.org/pep-0612/#concatenating-keyword-parameters).

I think we can have `use_async_effect` with a timeout parameter, but something I realized while working on this is that I don't think we can have a default. Having a...