-
Notifications
You must be signed in to change notification settings - Fork 902
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
Config Ref man #1851
base: develop
Are you sure you want to change the base?
Config Ref man #1851
Conversation
Giang's feedback Co-authored-by: Đỗ Trường Giang <[email protected]>
☂️ Python Coverage
Overall Coverage
New FilesNo new covered files... Modified Files
|
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.
Are you really generating files for the doc only at runtime for everybody?
* Stack broadcasts so that not one is lost resolves #1608 * fix test and typo * test --------- Co-authored-by: Fred Lefévère-Laoide <[email protected]>
* Toggle examples * lint * example builder + markdown fix crash in datatype * rename folder * files rename --------- Co-authored-by: Fred Lefévère-Laoide <[email protected]>
* feat(taipy-common): add common package and move config, _cli and logger to it * feat(taipy-common): refactor imports * feat(taipy-common): refactor release action * feat(taipy-common): fix templates tests * fix(taipy-common): fix config.pyi path Update config.pyi * chore(taipy-common): fix missing imports on pyi_header * chore(taipy-common): remove cli testing section on partial tests * Update config.pyi * chore(taipy-common): fix config import on migrate_cli * fix: missing common when importing taipy config * chore(taipy-common): bump dev version * chore(taipy-common): fix package name * chore(taipy-common): fix import format --------- Co-authored-by: joaoandre-avaiga <[email protected]> Co-authored-by: trgiangdo <[email protected]>
* manage broadcast resolves #1608 resolves #1891 * should not be --------- Co-authored-by: Fred Lefévère-Laoide <[email protected]>
Made some small change will enhance the quilty of reading
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.
👌 minor comments and questions
@@ -48,14 +48,14 @@ def _update(self, config_as_dict: t.Dict[str, t.Any], default_section=None): | |||
|
|||
@staticmethod | |||
def _configure(**properties) -> "_GuiSection": | |||
"""NOT DOCUMENTED | |||
Configure the Graphical User Interface. | |||
"""Configure the Graphical User Interface. |
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.
Why document that? How requested it?
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.
This is the method dynamically injected into the config. The usage is:
from taipy import Config
Config.configure_gui(...)
By the way, a user asked how to do it in 3.1 since it was not documented.
If you prefer not to promote this way of configuring the GUI, I can remove it without hesitation.
Waiting for Avaiga/taipy-doc#1146 to be merged