-
Notifications
You must be signed in to change notification settings - Fork 794
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Create aliases for all *Params
classes?
#2903
Comments
Hi @mattijn, that sounds fine to me. The reason I proposed aliases for some of these like |
Thanks for the clarification @ChristopherDavisUCI, that make sense for #2732. The remaining properties are indeed not available as Property Setter, but it is still useful to have an alias I think. |
In addition to what @ChristopherDavisUCI mentioned, I think it does make sense to include alias for the classes that we expect users to directly interface with (and maybe hiding the |
Maybe we should also consider adding shortcuts for some of the
Which might seem inconsistent with that if we modify say the height we can use
So I wonder if it should also be called just import requests as r
import re
vl_scheme = r.get('https://vega.github.io/schema/vega-lite/v5.json').json()
scheme_str = str(vl_scheme['definitions'])
col = []
for word in re.findall(r"\w+", scheme_str):
if 'Config' in word:
col.append(word)
list(set(col))
|
(*Params)
classes?
I'm linking #2918 because this won't be possible right now without causing collisions in the top level namespace I had thought about this in other PRs though |
(*Params)
classes?*Params
classes?
As was originally posted by @mattijn in #2732 (comment):
There are more candidate aliases I think?
Returns 10 candidates:
I think we can extend the aliases to all of these. I was trying:
But hit an:
The text was updated successfully, but these errors were encountered: