Skip to content
This repository has been archived by the owner on Nov 8, 2022. It is now read-only.

Logger logs done loading mention files before it actually does #25

Closed
aribornstein opened this issue Dec 25, 2018 · 2 comments
Closed
Assignees
Labels
bug Something isn't working Stale

Comments

@aribornstein
Copy link

Target objective:

Steps to objective:

nlp_architect/models/cross_doc_coref/system/cdc_utils.py

def load_mentions_vocab(mentions_files, filter_stop_words=False):
    logger.info('Loading mentions files...')
    mentions = []
    logger.info('Done loading mentions files, starting local dump creation...')
    for _file in mentions_files:
        mentions.extend(MentionData.read_mentions_json_to_mentions_data_list(_file))

    return extract_vocab(mentions, filter_stop_words)

Pull-Request related:

@peteriz peteriz assigned peteriz and AlonEirew and unassigned peteriz Dec 26, 2018
@AlonEirew
Copy link
Contributor

great catch! thanks for sharing it, issue will be fixed soon

@peteriz peteriz added the bug Something isn't working label Dec 26, 2018
@github-actions
Copy link

Issue marked as stale, closing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working Stale
Projects
None yet
Development

No branches or pull requests

3 participants