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

Export data from interactive learning broken #6914

Merged
merged 30 commits into from
Oct 6, 2020

Conversation

degiz
Copy link
Contributor

@degiz degiz commented Oct 5, 2020

Proposed changes:

  • Add YAML output support for the interactive learning

Status (please check what you already did):

  • added some tests for the functionality
  • updated the documentation
  • updated the changelog (please check changelog for instructions)
  • reformat files using black (please check Readme for instructions)

aleronupe and others added 14 commits September 23, 2020 00:28
Co-authored-by: Alexandre Miguel <[email protected]>
Co-authored-by: Alexandre Miguel <[email protected]>
@degiz degiz changed the base branch from master to 2.0.x October 5, 2020 18:18
@degiz degiz force-pushed the FGA-GCES-export-data-from-interactive-learning-broken branch from 2e9ec6d to ea82c17 Compare October 6, 2020 08:51
@degiz degiz added this to the 2.0 Rasa Open Source milestone Oct 6, 2020
@degiz degiz requested a review from alwx October 6, 2020 08:57
@tmbo
Copy link
Member

tmbo commented Oct 6, 2020

@degiz I think the ci is failing on code quality

@degiz degiz force-pushed the FGA-GCES-export-data-from-interactive-learning-broken branch from b2c3375 to 486e208 Compare October 6, 2020 12:35
@degiz degiz force-pushed the FGA-GCES-export-data-from-interactive-learning-broken branch from 486e208 to cee30cb Compare October 6, 2020 12:58
@rasabot rasabot merged commit d779e6a into 2.0.x Oct 6, 2020
@rasabot rasabot deleted the FGA-GCES-export-data-from-interactive-learning-broken branch October 6, 2020 20:01
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.

6 participants