AI page and app modularization #46
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
# Conflicts: # app.py # html_layout.py
- all tooltip icons are pushed towards right hand side column of the form - leading to layout reorganisation of the radio rows and server input
…tooltip element (including the icon), not relatively to the browser window
- improved tooltip icon location for large screen - additional and improved textual content
…s should be indicated beforehand when the app is instantiated. Does not work because dash version is lower than 2.14 (2.9.2 currently). A possible workaround is to instantiate the arguments in the html.py file directly.
…in a more suitable way. URL querry strings are now properly caught by the layout but not handled yet
The layout function from Dash pages automatically collects query strings from url, one juste has to parse them. Does not require any lirary update. Now url query string are not handled in callbacks anymore but when initiating the html layout.
…ult values: checking if possible to remove all Input(url_content, search) from callbacks documenting the solutions see callback set_server_value and set_serverContinents_value refacto: renaming an few additional comments
…ult values: checking if possible to remove all Input(url_content, search) from callbacks documenting the solutions see callback set_server_value and set_serverContinents_value refacto: renaming an few additional comments
- both functionalities work - TODO: flush input content properly and check cicrcular callback behaviour - TODO: add sanity checks when receiving inputs from csv
…s to let the user fill in the same csv again after modifying the form
|
In hindsight, I think we should include the methodology boxes ( |
|
I'm not sure about the role of the |
…here + frontend adjustment
…g different than google cloud
…the csv exported from the ai page
Closed
Llannelongue
approved these changes
Feb 4, 2025
This was referenced Feb 4, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The creation of the AI page motivated the app modularization.
Main features:
dash_extensions.enrich.DashBlueprint: