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

Add verification when summarize_corpus returns null. Fix #1531. #1570

Merged
merged 5 commits into from
Sep 18, 2017
Merged

Add verification when summarize_corpus returns null. Fix #1531. #1570

merged 5 commits into from
Sep 18, 2017

Commits on Sep 5, 2017

  1. Avoid "NoneType is not iterable..." error for few documents in corpus.

    @fbarrios
    fbarrios committed Sep 5, 2017
    Configuration menu
    Copy the full SHA
    d200a5d View commit details
    Browse the repository at this point in the history

  2. Fix comment.

    @fbarrios
    fbarrios committed Sep 5, 2017
    Configuration menu
    Copy the full SHA
    3b4982e View commit details
    Browse the repository at this point in the history

  3. Adding relevant test.

    @fbarrios
    fbarrios committed Sep 5, 2017
    Configuration menu
    Copy the full SHA
    78147bc View commit details
    Browse the repository at this point in the history

Commits on Sep 9, 2017

  1. Fixed return types on summarization border cases: 

    - Returns empty list on border case of summarize_corpus.
- Returns empty string or empty list on border case of summarize.
- Fixed test accordingly.
- Removed some test code repetition.
    @fbarrios
    fbarrios committed Sep 9, 2017
    Configuration menu
    Copy the full SHA
    e3a1d88 View commit details
    Browse the repository at this point in the history

Commits on Sep 14, 2017

  1. Replace is to ==

    @menshikh-iv
    menshikh-iv committed Sep 14, 2017
    Configuration menu
    Copy the full SHA
    91c8142 View commit details
    Browse the repository at this point in the history