Bryan Van de Ven

Results 745 comments of Bryan Van de Ven

@mattpap I self-assigned the issue to indicate that I had already begun working on a PR.

> changes in this PR do not overlap with your proposed changes in the issue They do, as my plans included replacing regexes in the property to delegate validation to...

@mattpap after digging up my old branch I actually don't like the approach in it at all anymore. There are so may cases I do think we need something that...

@annesophie-cls First off, please try things out with latest version 3.5 and report back, in case the issue has been fixed since 3.2.2. In any case there will be no...

@annesophie-cls looking at the actual codepaths more closely, it does not appear that it is currently possible to combined `{safe}` and `{custom}` at the same time. TLDR; `{safe}` is itself...

That was my first thought as well, but it's also a little weird. `class_default` and `instance_default` return the themed default, so this *shouldn't* be a no-op: https://github.com/bokeh/bokeh/blob/0326ae1c0a52213bbe62a4624f3621af3e03affe/src/bokeh/core/property/descriptors.py#L308-L335 So maybe there...

> We need to mark such properties as "dirty" @mattpap FYI there has also been some recent talk around supporting light/dark modes via themes, so we will want to make...

@Murat-U-Saglam we cannot evaluate anything without a ***complete*** minimal reproducing example. We need code that we can take and run directly, *without any modifications*, that reproduces the issue you want...

@Murat-U-Saglam This appears to be a bug related to deserializing the CSS style `rgb(...)` color values. If save the JSON string to a file then manually edit those to be...

For reference, even just this code fails to validate on either Bokeh 2.4.3 or latest `branch-3.6` ```python In [4]: from bokeh.models import LinearColorMapper ...: color = LinearColorMapper( ...: low=0, high=4,...