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
allow components to add fingerprints of external data #12778
Conversation
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.
lgtm
@@ -76,3 +83,44 @@ def test_fingerprint_changes_due_to_changed_source(monkeypatch: MonkeyPatch): | |||
assert key1 != key2 | |||
|
|||
get_source_mock.assert_called_once_with(TEDPolicy) | |||
|
|||
|
|||
def test_fingerprint_changes_when_external_file_changes(): |
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.
OPT: can you split this into two tests: one testing that the fingerprint is stable and another to check it is changing?
🚀 A preview of the docs have been deployed at the following URL: https://12778--rasahq-docs-rasa-v2.netlify.app/docs/rasa |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information The version of Java (11.0.20) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17. |
* allow components to add fingerprints of external data
* allow components to add fingerprints of external data
Proposed changes:
GraphComponent
that allows users to add additional fingerprint data to their component, such as external files, knowledge bases etc. that are not directly in the rasa graphStatus (please check what you already did):
black
(please check Readme for instructions)