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

[s2s] rougeLSum expects \n between sentences #7410

Merged
merged 19 commits into from Sep 27, 2020

Commits on Sep 24, 2020

  1. Fix eval to compute rouge correctly for rouge_score

    Swetha Mandava committed Sep 24, 2020
    Configuration menu
    Copy the full SHA
    2aa5fc2 View commit details
    Browse the repository at this point in the history
  2. styling

    Swetha Mandava committed Sep 24, 2020
    Configuration menu
    Copy the full SHA
    accbc51 View commit details
    Browse the repository at this point in the history

Commits on Sep 25, 2020

  1. moving sentence tokenization to utils from run_eval

    Swetha Mandava committed Sep 25, 2020
    Configuration menu
    Copy the full SHA
    c7e4959 View commit details
    Browse the repository at this point in the history

Commits on Sep 26, 2020

  1. add cli

    sshleifer committed Sep 26, 2020
    Configuration menu
    Copy the full SHA
    fb48108 View commit details
    Browse the repository at this point in the history
  2. use_old option

    sshleifer committed Sep 26, 2020
    Configuration menu
    Copy the full SHA
    5e88bb1 View commit details
    Browse the repository at this point in the history
  3. many kwargs

    sshleifer committed Sep 26, 2020
    Configuration menu
    Copy the full SHA
    d45951a View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    f5f11d7 View commit details
    Browse the repository at this point in the history
  5. One test failing

    sshleifer committed Sep 26, 2020
    Configuration menu
    Copy the full SHA
    71f2eef View commit details
    Browse the repository at this point in the history
  6. Try fix stas

    sshleifer committed Sep 26, 2020
    Configuration menu
    Copy the full SHA
    946b841 View commit details
    Browse the repository at this point in the history
  7. Fix api

    sshleifer committed Sep 26, 2020
    Configuration menu
    Copy the full SHA
    7146d9c View commit details
    Browse the repository at this point in the history
  8. cleaner

    sshleifer committed Sep 26, 2020
    Configuration menu
    Copy the full SHA
    d6781db View commit details
    Browse the repository at this point in the history

Commits on Sep 27, 2020

  1. cleanup rouge cli

    sshleifer committed Sep 27, 2020
    Configuration menu
    Copy the full SHA
    33d9e12 View commit details
    Browse the repository at this point in the history
  2. style issues

    sshleifer committed Sep 27, 2020
    Configuration menu
    Copy the full SHA
    607689d View commit details
    Browse the repository at this point in the history
  3. Fix style

    sshleifer committed Sep 27, 2020
    Configuration menu
    Copy the full SHA
    df4955b View commit details
    Browse the repository at this point in the history
  4. test for rouge_cli

    sshleifer committed Sep 27, 2020
    Configuration menu
    Copy the full SHA
    cbec102 View commit details
    Browse the repository at this point in the history
  5. Add nltk workaround

    sshleifer committed Sep 27, 2020
    Configuration menu
    Copy the full SHA
    717173a View commit details
    Browse the repository at this point in the history
  6. style

    sshleifer committed Sep 27, 2020
    Configuration menu
    Copy the full SHA
    d5b9287 View commit details
    Browse the repository at this point in the history
  7. Configuration menu
    Copy the full SHA
    77cf1cd View commit details
    Browse the repository at this point in the history
  8. more descriptive name

    sshleifer committed Sep 27, 2020
    Configuration menu
    Copy the full SHA
    ee83da0 View commit details
    Browse the repository at this point in the history