Ryan Henning

I agree. In fact, caching will likely be slower than simply re-parsing the UA on each request. A warmed-up parser takes microseconds to parse the UA. See below: No reason...

This is **EXACTLY** the PR I was planning to send in when I wrote issue #21608. Nice work! Well, one difference actually: I would do the same change for `PointLightHelper`,...

PS: This is a big problem being addressed here. It's easy to create a light helper that shows you the wrong thing, in which case it does the **opposite** of...

@mrdoob Do you think we get this in the r128 release?

@mrdoob Anything we can do to help you feel comfortable with this? Excited to get it in r136. 😉

Maybe the issue here (described above) is the underlying cause of this other issue: https://github.com/waveform80/picamera/issues/242 ??

A workaround is to do `default=lambda: 0` on your ME fields, instead of just the simple `default=0`.

For those following this issue, it might be interesting for you to know there is a _new_ channel layer implementation which is a lightweight layer on top of [Redis Pub/Sub](https://redis.io/topics/pubsub)....

Both implementations allow you to override `serialize()` and `deserialize()`. See [this answer](https://github.com/django/channels_redis/issues/254#issuecomment-952270903) for an example. So, you can call msgpack with other arguments (or even not use msgpack at all)....

Yeah the original impl does more stuff in those methods. On the other hand, the pubsub impl has [simpler methods](https://github.com/django/channels_redis/blob/fe27f1a83a87bf61a9245188c8825ddfa30119fa/channels_redis/pubsub.py#L57-L68) to override. In either case, this is how you can...