feat: add dynamic state model creation and update #3271
Merged
Conversation
This file contains 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
dosubot
bot
added
size:L
Pull Request Validation ReportThis comment is automatically generated by Conventional PR Whitelist Report
Result Pull request does not satisfy any enabled whitelist criteria. Pull request will be validated. Validation Report
Result Pull request satisfies all enabled pull request rules. Last Modified at 09 Aug 24 22:05 UTC |
github-actions
bot
added
enhancement
|
This pull request is automatically being deployed by Amplify Hosting (learn more). |
ogabrielluiz
changed the title
github-actions
bot
added
enhancement
dosubot
bot
added
size:XL
ogabrielluiz
force-pushed
the
feat/addstate
branch
2 times, most recently
from
5016132 to
fb82107
Compare
italojohnny
approved these changes
Aug 12, 2024
…output getter in graph state module
…nt outputs in custom_component class
… instance getter in custom_component class
Refactor the Component class in the custom_component module to change the visibility of the method `_get_output_by_method` to public by renaming it to `get_output_by_method`. This change improves the accessibility and clarity of the method for external use.
…er utilities in state model to ensure proper structure
…st_graph_state_model.py
…ent.py for state model instance retrieval
…unctional state update validation
…ed clarity and usage in state model tests
… improved method integrity in state model handling
…rror handling for output methods in model.py
…in test_state_model.py
…l for consistency in test_state_model.py
…odel.py for improved flexibility
….py to enhance code clarity and usability
…rify usage and validation details
…to improve clarity on functionality and usage scenarios
….py to clarify functionality and usage examples
…e_model.py to clarify functionality and provide examples
…ved structure and correctness verification
ogabrielluiz
force-pushed
the
feat/addstate
branch
from
fb82107 to
61aef47
Compare
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.
This pull request includes several refactorings and feature additions to the Component class in the custom_component module. The visibility of the method
_get_output_by_methodhas been changed to public by renaming it toget_output_by_method, improving accessibility and clarity. Additionally, state model management has been implemented with lazy initialization and dynamic instance getter in the custom_component class. The_reset_all_output_valuesmethod has been added to initialize component outputs. The Graph class has been enhanced with lazy loading for state model creation from the graph, and a new methodcreate_state_model_from_graphhas been added to the graph state_model module. Unit tests and functional tests have been added to validate the state model creation and graph state update.