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

Bugfix/utils get and tokenize dataset #4

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Bugfix/utils get and tokenize dataset #4

wants to merge 4 commits into from

Commits on Sep 6, 2019

  1. Fixed issue: invalid index while using dataset_map['labels']

    @sahilsharma05
    sahilsharma05 committed Sep 6, 2019
    Configuration menu
    Copy the full SHA
    3ce6b2b View commit details
    Browse the repository at this point in the history

  2. Fixed issue: 

    - replaced 'test' with 'valid' key in global variables DATASETS_URL and DATASETS_LABELS_URL. The training logic is using valid files.
- for IMDB dataset, test.txt and test.labels.txt doesn't exist. replaced with valid.txt and valid.labels.txt (exist on s3 server).
    @sahilsharma05
    sahilsharma05 committed Sep 6, 2019
    Configuration menu
    Copy the full SHA
    cb56df7 View commit details
    Browse the repository at this point in the history

  3. delete extra added code

    @sahilsharma05
    sahilsharma05 committed Sep 6, 2019
    Configuration menu
    Copy the full SHA
    ac9f44b View commit details
    Browse the repository at this point in the history

  4. added unit test case for get_and_tokenize_dataset method

    @sahilsharma05
    sahilsharma05 committed Sep 6, 2019
    Configuration menu
    Copy the full SHA
    ec5e9bb View commit details
    Browse the repository at this point in the history