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.

Sign up for GitHub

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

Jump to bottom

Update requirements #1663

Closed
wants to merge 71 commits into from
Closed

Update requirements #1663

wants to merge 71 commits into from

Conversation

EPedrotti
Copy link
Contributor

Proposed changes:

  • update the requirements to match core and sdk
  • pin keras dependencies

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

tmbo and others added 30 commits January 23, 2019 14:50
@tmbo
removed future imports and six
fd9b7bc
@tmbo
removed py2 from travis build
a59fe5d
@tmbo
added py3 note
9181d33
@tmbo
removed py2 building of wheel
ba9f40c
@tmbo
fixed docker container
cd017f9
@tmbo
added to changelog
1d3a46e
@tmbo
increased version number
e96cb92
@tmbo
fixed any type annotations
fa4a074
@tmbo
fixed error
b647f32
@tmbo
fixed pep8
bba7884
@tmbo
fixed more comment style issues
72bcad3
@tmbo
fixed type imports
d63519f
@tmbo
fixed some more pep8 errors
a1f2d6c
@ricwo @tmbo
Update rasa_nlu/classifiers/__init__.py
f9bef31 
Co-Authored-By: tmbo <tom@rasa.com>
@ricwo @tmbo
Update rasa_nlu/classifiers/embedding_intent_classifier.py
f5f857c 
Co-Authored-By: tmbo <tom@rasa.com>
@ricwo @tmbo
Update rasa_nlu/classifiers/sklearn_intent_classifier.py
c16565a 
Co-Authored-By: tmbo <tom@rasa.com>
@ricwo @tmbo
Update rasa_nlu/classifiers/mitie_intent_classifier.py
7b21ed3 
Co-Authored-By: tmbo <tom@rasa.com>
@ricwo @tmbo
Update rasa_nlu/convert.py
579b0cc 
Co-Authored-By: tmbo <tom@rasa.com>
@ricwo @tmbo
Update rasa_nlu/data_router.py
ee4af9a 
Co-Authored-By: tmbo <tom@rasa.com>
@ricwo @tmbo
Update tests/base/test_synonyms.py
f445801 
Co-Authored-By: tmbo <tom@rasa.com>
@ricwo @tmbo
Update tests/example_component.py
9a67c37 
Co-Authored-By: tmbo <tom@rasa.com>
@ricwo @tmbo
Update tests/training/test_train.py
e5ac301 
Co-Authored-By: tmbo <tom@rasa.com>
@ricwo @tmbo
Update tests/utilities.py
614300e 
Co-Authored-By: tmbo <tom@rasa.com>
@ricwo @tmbo
Update tests/conftest.py
6f3117d 
Co-Authored-By: tmbo <tom@rasa.com>
@ricwo @tmbo
Update rasa_nlu/emulators/luis.py
37c1ec6 
Co-Authored-By: tmbo <tom@rasa.com>
@ricwo @tmbo
Update rasa_nlu/emulators/wit.py
0003f81 
Co-Authored-By: tmbo <tom@rasa.com>
@ricwo @tmbo
Update rasa_nlu/evaluate.py
be6b27c 
Co-Authored-By: tmbo <tom@rasa.com>
@ricwo @tmbo
Update rasa_nlu/extractors/crf_entity_extractor.py
a63f49a 
Co-Authored-By: tmbo <tom@rasa.com>
@ricwo @tmbo
Update rasa_nlu/extractors/__init__.py
3cf7b03 
Co-Authored-By: tmbo <tom@rasa.com>
@ricwo @tmbo
Update rasa_nlu/extractors/duckling_http_extractor.py
8a7f51c 
Co-Authored-By: tmbo <tom@rasa.com>
ricwo and others added 28 commits January 23, 2019 18:33
@ricwo @tmbo
Update rasa_nlu/tokenizers/jieba_tokenizer.py
4322642 
Co-Authored-By: tmbo <tom@rasa.com>
@ricwo @tmbo
Update rasa_nlu/tokenizers/spacy_tokenizer.py
0fb895e 
Co-Authored-By: tmbo <tom@rasa.com>
@ricwo @tmbo
Update rasa_nlu/tokenizers/whitespace_tokenizer.py
28738c6 
Co-Authored-By: tmbo <tom@rasa.com>
@ricwo @tmbo
Apply suggestions from code review
2c6a489 
Co-Authored-By: tmbo <tom@rasa.com>
@tmbo
Merge branch 'master' into py3-only
c581b58
@tmbo
Merge pull request #1641 from RasaHQ/py3-only
c525081 
remove python 2 support
@tmbo
merged master
239514c
@tmbo
fixed miti docker file
5d648f1
@tmbo
lots of type changes
4c6b1e3
@tmbo
fixed pep8 issues
34b6170
@tmbo
fixed more pep8 issues
8c40e45
@ricwo @tmbo
Update rasa_nlu/config.py
c99e936 
Co-Authored-By: tmbo <tom@rasa.com>
@ricwo @tmbo
Apply suggestions from code review
636d9dd 
Co-Authored-By: tmbo <tom@rasa.com>
@tmbo
fixed pep8 errors
6cf9ce3
@tmbo
Merge pull request #1642 from RasaHQ/type-annotations
62bc21f 
Type annotations
@tmbo
removed fossa token and added as a setting to travis
c61bc1e
@dthulke
fixes type hints if tensorflow is not imported
06fcf75
@Tpt
Uses Spacy 2 syntax to disable dependency parser
4b3a356 
Closes #1649
@tmbo
trying to fix fossa checks for contributor pushes
64d8276
@akelad
Merge branch 'master' into patch-1
2d5e7b1
@dthulke
Merge branch 'master' into fix-typing
41b0431
@Tpt
update CHANGELOG
ca25341
@wochinge
Merge pull request #1650 from Tpt/patch-1
22af37f 
Use Spacy 2 syntax to disable dependency parser
@akelad
Merge branch 'master' into fix-typing
78d4ca6
@akelad
Merge pull request #1648 from dthulke/fix-typing
7672798 
Fixes type hints if tensorflow is not imported
@EPedrotti
update requirements to match core and sdk
08d2292
@EPedrotti
pin keras dependecies
6ca2275
@EPedrotti
update CHANGELOG, update setup.py with keras pinned deps, move keras …
a62cf3a 
…deps to tensorflow requirements file
@EPedrotti EPedrotti closed this Jan 30, 2019
@tmbo tmbo deleted the update-requirements branch May 9, 2019 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
7 participants
@EPedrotti @tmbo @ricwo @dthulke @Tpt @akelad @wochinge