-
Notifications
You must be signed in to change notification settings - Fork 370
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
Configuration for topk and sort order #206
Changes from all commits
c1944a2
5c8b284
49daeec
801b469
a8ab02e
7a203df
74f2c7e
298a87e
efd8fd0
a495180
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -19,6 +19,7 @@ lux.vislib.altair.AltairChart.AltairChart | |
~AltairChart.apply_default_config | ||
~AltairChart.encode_color | ||
~AltairChart.initialize_chart | ||
~AltairChart.sanitize_dataframe | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What are the There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The |
||
|
||
|
||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a way to call
expire_recs()
automatically when a config property is set, like have setter functions and includeexpire_recs()
at the beginning?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point! The trouble is that we don't currently have a list of dataframes applicable in the session so we don't have a way of expiring these. I've opened an issue #209 for this to address in the future. Thanks for raising this!