Skip to content

Conversation

@kbalog
Copy link
Contributor

@kbalog kbalog commented Sep 30, 2025

Fixes #267

@kbalog kbalog requested a review from NoB0 September 30, 2025 10:32
Copy link
Collaborator

@NoB0 NoB0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The following files can also be removed:

  • test_diet_classifier_rasa.py
  • test_annotation_converter_dialoguekit_to_rasa.py

Also the __init__.py files in the following packages should be updated to remove Rasa related component:

  • nlu package should remove import of IntentClassifierRasa
  • utilspackages should remove AnnotationConverterRasa
  • sample_agents should remove RasaParrotAgent

@@ -1,4 +1,4 @@
"""Convert annotations to be compatible with the rasa models."""
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe that this file can be removed, along with the associated test file.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was considering this, yes. Done.

@kbalog kbalog requested a review from NoB0 September 30, 2025 12:59
Copy link
Collaborator

@NoB0 NoB0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Please add requests to requirements.txt.

@kbalog kbalog merged commit 304173d into main Sep 30, 2025
2 of 4 checks passed
@kbalog kbalog deleted the feature/267-Remove-RASA-components branch September 30, 2025 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove RASA components

3 participants