You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In #3071 and #3073 types were added to some to_json() methods (well, #3071 added the method entirely to TopLevelMixin) that specify indent as an int. In these, indent is only passed along to json.dumps, where it should be possible to pass None or a string as well: https://docs.python.org/3/library/json.html#json.dump
With indent typed as int and defaulting to 2 in the altair API, there is no way to get the compact indent=None serialization in a way that typechecks.
I think the types should be relaxed to str | int | None or something similar.
The text was updated successfully, but these errors were encountered:
In #3071 and #3073 types were added to some
to_json()
methods (well, #3071 added the method entirely toTopLevelMixin
) that specifyindent
as anint
. In these,indent
is only passed along tojson.dumps
, where it should be possible to passNone
or a string as well: https://docs.python.org/3/library/json.html#json.dumpWith
indent
typed asint
and defaulting to 2 in the altair API, there is no way to get the compactindent=None
serialization in a way that typechecks.I think the types should be relaxed to
str | int | None
or something similar.The text was updated successfully, but these errors were encountered: