{"action":"created","issue":{"url":"https://api.github.com/repos/UKPLab/emnlp2017-relation-extraction/issues/11","repository_url":"https://api.github.com/repos/UKPLab/emnlp2017-relation-extraction","labels_url":"https://api.github.com/repos/UKPLab/emnlp2017-relation-extraction/issues/11/labels{/name}","comments_url":"https://api.github.com/repos/UKPLab/emnlp2017-relation-extraction/issues/11/comments","events_url":"https://api.github.com/repos/UKPLab/emnlp2017-relation-extraction/issues/11/events","html_url":"https://github.com/UKPLab/emnlp2017-relation-extraction/issues/11","id":321106546,"number":11,"title":"got an unexpected keyword argument 'embeddings'","user":{"login":"shaburanigud","id":21302590,"avatar_url":"https://avatars2.githubusercontent.com/u/21302590?v=4","gravatar_id":"","url":"https://api.github.com/users/shaburanigud","html_url":"https://github.com/shaburanigud","followers_url":"https://api.github.com/users/shaburanigud/followers","following_url":"https://api.github.com/users/shaburanigud/following{/other_user}","gists_url":"https://api.github.com/users/shaburanigud/gists{/gist_id}","starred_url":"https://api.github.com/users/shaburanigud/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/shaburanigud/subscriptions","organizations_url":"https://api.github.com/users/shaburanigud/orgs","repos_url":"https://api.github.com/users/shaburanigud/repos","events_url":"https://api.github.com/users/shaburanigud/events{/privacy}","received_events_url":"https://api.github.com/users/shaburanigud/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-05-08T09:18:30Z","updated_at":"2018-05-11T10:01:14Z","closed_at":null,"author_association":"NONE","body":"When I try to recreate the training process I seem to get this error\r\n\r\n> python3 model_train.py model_ContextWeighted train-continue ../data/wikipedia-wikidata/enwiki-20160501/semantic-graphs-filtered-training.02_06.json ../data/wikipedia-wikidata/enwiki-20160501/semantic-graphs-filtered-validation.02_06.json\r\n/usr/local/lib/python3.5/dist-packages/h5py/__init__.py:36: FutureWarning: Conversion of the second argument of issubdtype from `float` to `np.floating` is deprecated. In future, it will be treated as `np.float64 == np.dtype(float).type`.\r\n  from ._conv import register_converters as _register_converters\r\nUsing TensorFlow backend.\r\nLoaded embeddings: (400002, 50)\r\nLoaded data size: 372059\r\nLoaded data size: 123824\r\nTraining data size: 372059\r\nValidation data size: 123824\r\nMax sentence length set to: 36\r\nTraceback (most recent call last):\r\n  File \"model_train.py\", line 92, in <module>\r\n    embeddings=embedding_matrix, position2idx=keras_models.position2idx))\r\nTypeError: to_indices_with_extracted_entities() got an unexpected keyword argument 'embeddings'\r\n\r\nin which is caused by this part of the code in model_train.py\r\n\r\n> if \"Context\" in model_name:\r\n        to_one_hot = embeddings.timedistributed_to_one_hot\r\n        graphs_to_indices = keras_models.to_indices_with_extracted_entities\r\n    elif \"CNN\" in model_name:\r\n        graphs_to_indices = keras_models.to_indices_with_relative_positions\r\n\r\n    train_as_indices = list(graphs_to_indices(training_data, word2idx, max_sent_len,                                               embeddings=embedding_matrix, position2idx=keras_models.position2idx))\r\n    print(\"Dataset shapes: {}\".format([d.shape for d in train_as_indices]))\r\n    training_data = None\r\n    n_out = len(keras_models.property2idx)\r\n    print(\"N_out:\", n_out)\r\n\r\n    val_as_indices = list(graphs_to_indices(val_data, word2idx, max_sent_len,                                             embeddings=embedding_matrix, position2idx=keras_models.position2idx))\r\n    val_data = None\r\nWould you mind giving any advice on how to fix this?\r\n\r\nP.S. Line 92 refers to  train_as_indices = list(graphs_to_indices(training_data, word2idx, max_sent_len,                                              embeddings=embedding_matrix, position2idx=keras_models.position2idx))\r\n\r\nP.S.2 Initially when I tried to run the code with \"train-test\" I found that it is an invalid choice. Unless I missed something, is the README currently outdated?"},"comment":{"url":"https://api.github.com/repos/UKPLab/emnlp2017-relation-extraction/issues/comments/388319913","html_url":"https://github.com/UKPLab/emnlp2017-relation-extraction/issues/11#issuecomment-388319913","issue_url":"https://api.github.com/repos/UKPLab/emnlp2017-relation-extraction/issues/11","id":388319913,"user":{"login":"daniilsorokin","id":6469854,"avatar_url":"https://avatars3.githubusercontent.com/u/6469854?v=4","gravatar_id":"","url":"https://api.github.com/users/daniilsorokin","html_url":"https://github.com/daniilsorokin","followers_url":"https://api.github.com/users/daniilsorokin/followers","following_url":"https://api.github.com/users/daniilsorokin/following{/other_user}","gists_url":"https://api.github.com/users/daniilsorokin/gists{/gist_id}","starred_url":"https://api.github.com/users/daniilsorokin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/daniilsorokin/subscriptions","organizations_url":"https://api.github.com/users/daniilsorokin/orgs","repos_url":"https://api.github.com/users/daniilsorokin/repos","events_url":"https://api.github.com/users/daniilsorokin/events{/privacy}","received_events_url":"https://api.github.com/users/daniilsorokin/received_events","type":"User","site_admin":false},"created_at":"2018-05-11T10:01:14Z","updated_at":"2018-05-11T10:01:14Z","author_association":"CONTRIBUTOR","body":"Thanks for the report! Yes, that was a bug, that was caused by a new signature for the data encoding methods. Try the latest version, please.\r\n\r\nREADME was also updated."}}
{"actor":{"display_login":"daniilsorokin"}}