Skip to content
This repository has been archived by the owner on Aug 22, 2019. It is now read-only.

Unfeaturized entities for specified intents #763

Closed
wants to merge 43 commits into from
Closed

Unfeaturized entities for specified intents #763

wants to merge 43 commits into from

Conversation

akelad
Copy link
Contributor

@akelad akelad commented Jul 18, 2018 •
edited

Proposed changes:

  • allow user to specify a list of intents in the domain file for which entities should be ignored. These entities will only be ignored if they do not have a featurized slot of the same name.

Status (please check what you already did):

  • made PR ready for code review
  • added some tests for the functionality
  • updated the documentation
  • updated the changelog

akelad and others added 30 commits July 16, 2018 15:54
@akelad
initial attempt
48dcc4c
@akelad
small fix
81c73a4
@akelad
unfeaturize slots too
f7379cd
@akelad
add check for featurized slots
1c46ea5
@akelad
add if statement back in
2082659
@akelad
update logger warning
59f9caf
@akelad
change var name
882a669
@amn41
fix some links
0faec54
@amn41
remove junk
e2eba15
@akelad
Merge pull request #764 from RasaHQ/docs_theme_gcs
d68b12f 
Docs theme gcs
@akelad
prepare next version
c698c50
@amn41
only build and push docs on master (incl PRs)
d2e5f70
@amn41
trigger build
2f6dd78
@akelad
Merge pull request #766 from RasaHQ/only_build_docs_on_master
ce0defc 
only build and push docs on master (incl PRs)
@sousandrei
Reduced Docker image size: removed pip cache
da51ea7
@ricwo
add RasaChatInput
3943820
@ricwo
add comments and restructure
9d0f8f7
@ricwo
use CollectingOutputChannel
3883656
@ricwo
Merge pull request #776 from RasaHQ/rasa_chat
41c5ea7 
New chat input channel
@akelad
Merge pull request #772 from sousandrei/docker-tweak
5d0b5e4 
Reduced Docker image size: removed pip cache
@amn41
trigger build
15bfd0b
@akelad
fix package in dockerfile
ed9e5c2
@amn41
trigger build
2362085
@akelad
Merge pull request #778 from RasaHQ/fix_docker
5d028e4 
Fix Dockerfile
@amn41
remove typo
d3aeb67
@amn41
Merge branch 'master' of https://github.com/RasaHQ/rasa_core
898f96d
@amn41
trigger build
e33135f
@amn41
trigger build
97edf28
@akelad
fix tests
e22d7f7
@akelad
add docs
af7a2e6
@akelad
Copy link
Contributor Author

akelad commented Jul 23, 2018 •
edited

gonna open a new PR (#791) because merging master has broken this one

@akelad akelad closed this Jul 23, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@akelad @amn41 @sousandrei @ricwo