Skip to content
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

Graph generated upon comparing multiple NLU pipelines is not rendered correctly #8200

Closed
2 tasks done
ganeshviyer opened this issue Mar 15, 2021 · 7 comments · Fixed by #8859
Closed
2 tasks done

Graph generated upon comparing multiple NLU pipelines is not rendered correctly #8200

ganeshviyer opened this issue Mar 15, 2021 · 7 comments · Fixed by #8859
Assignees
Labels
area:rasa-oss 🎡 Anything related to the open source Rasa framework area:rasa-oss/model-testing Issues focused around testing models (e.g. via `rasa test`) type:bug 🐛 Inconsistencies or issues which will cause an issue or problem for users or implementors.

Comments

@ganeshviyer
Copy link

ganeshviyer commented Mar 15, 2021

Rasa version: 2.3.4

Rasa SDK version (if used & relevant): (not relevant; using an NLU-only pipeline)

Rasa X version (if used & relevant): (not relevant for this issue)

Python version: 3.7.9

Operating system (windows, osx, ...): mac OS Big Sur 11.1

Issue:
See this forum topic for more info - https://forum.rasa.com/t/is-there-a-doc-on-how-to-interpret-pipeline-comparison-results/41165/2

I was looking to find a resource that would help me interpret the results of the pipeline comparison, but it appears that there are multiple graphs plotted over each other (as noted by @samsucik).

In addition, he notes that the graph also has lost information about the scale of its axes. I'm not sure what the expected result should be.

Also tagging @dakshvar22.

Error (including full traceback):
See attached file in the forum - https://forum.rasa.com/uploads/short-url/koZaCVIBB4ozHLZdbBMbnJUt9IK.pdf

Command or request that led to error:

rasa test nlu --nlu data/nlu.yml --config available_config/spacy.yml available_config/mitie.yml

Content of configuration file (config.yml) (if relevant):
spacy.yml

language: "en"  # your two-letter language code

pipeline:
  - name: SpacyNLP
    model: "en_core_web_md"
    case_sensitive: False
  - name: SpacyTokenizer
  - name: SpacyFeaturizer
  - name: RegexFeaturizer
  - name: LexicalSyntacticFeaturizer
  - name: CountVectorsFeaturizer
  - name: CountVectorsFeaturizer
    analyzer: "char_wb"
    min_ngram: 1
    max_ngram: 4
  - name: DIETClassifier
    epochs: 100
  - name: EntitySynonymMapper

mitie.yml

language: "en"  # your two-letter language code

pipeline:
  - name: MitieNLP
    model: "data/total_word_feature_extractor.dat"
  - name: MitieTokenizer
  - name: MitieFeaturizer
  - name: RegexFeaturizer
  - name: LexicalSyntacticFeaturizer
  - name: CountVectorsFeaturizer
  - name: CountVectorsFeaturizer
    analyzer: "char_wb"
    min_ngram: 1
    max_ngram: 4
  - name: DIETClassifier
    epochs: 100
  - name: EntitySynonymMapper

Content of domain file (domain.yml) (if relevant): (not relevant here)

Definition of done

  • Cause of this bug determined
  • Fix, if straightforward. Otherwise, create an issue detailing the cause and potential solutions
@ganeshviyer ganeshviyer added area:rasa-oss 🎡 Anything related to the open source Rasa framework type:bug 🐛 Inconsistencies or issues which will cause an issue or problem for users or implementors. labels Mar 15, 2021
@ganeshviyer
Copy link
Author

In addition, could you provide me with directions on where to find the documentation for how to interpret pipeline comparison results? Many thanks in advance! :-)

@samsucik samsucik added the area:rasa-oss/model-testing Issues focused around testing models (e.g. via `rasa test`) label Mar 15, 2021
@sara-tagger
Copy link
Collaborator

Thanks for the issue, @koernerfelicia will get back to you about it soon!

You may find help in the docs and the forum, too 🤗

@koernerfelicia
Copy link
Contributor

@ganeshviyer looks like this has already been routed to the right Inbox. Thanks for submitting the issue! :)

@joejuzl
Copy link
Contributor

joejuzl commented May 20, 2021

@koernerfelicia I think this should probably be handled by research as it's "owned" by them according to the code boundaries. (rasa.nlu.test). WDYT?

@koernerfelicia
Copy link
Contributor

@joejuzl yeah, that seems fair!

@tttthomasssss
Copy link
Contributor

@jupyterjazz assigned as reviewer.

@dakshvar22
Copy link
Contributor

@ganeshviyer This was fixed as part of #8859 . Will be released in the next micro version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:rasa-oss 🎡 Anything related to the open source Rasa framework area:rasa-oss/model-testing Issues focused around testing models (e.g. via `rasa test`) type:bug 🐛 Inconsistencies or issues which will cause an issue or problem for users or implementors.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants