Alexander Reynolds

Results 21 comments of Alexander Reynolds

> If the example has a pinned-down URL (i.e. points to a commit), I agree network access should not be mandatory. @uranusjr note that this isn't really a workable flow...

I am also having what I think is the same issue. Here's a different minimal repro: ```python import time from uuid import uuid4 from dash import MATCH, Dash, DiskcacheManager, Input,...

> I *think* that I want `isoparse` to always return a `datetime` That's why I figured it wouldn't be processed via `parser.isoparse`, since the period is necessarily a delta, though...

@pganssle great, thanks for the background! That all makes sense.

>Your suggestion to use a fixed max depth would create very wide table Arrow tables, that would be very hard to use in practice. Agreed if you wanted max depth...

> otherwise it'll be closed in 7 days lol you closed the same day as this comment! It's not stale

It seems like the `mask` parameter is _optional_ from the pyarrow docs: https://arrow.apache.org/docs/python/generated/pyarrow.StructArray.html#pyarrow.StructArray.from_arrays and the default is `None`: https://github.com/apache/arrow/blob/ea9b15ff941e7492e171cffee05af85b99306631/python/pyarrow/array.pxi#L4021 so probably can just use `None` instead of `pa.repeat(False, len(messages))` either...

But glancing at some of the implementation with the iterable getters, it's not clear to me if a general `Iterable[M]` is actually OK, or if the library specifically needs the...

hmm I don't think `Iterable` implies you can iterate over it many times - iterators and generators, including infinite and non-re-entrant ones, are also `Iterable`. Agreed though that `Sequence` has...