search

uhh-lt / sensegram

Making sense embedding out of word embeddings using graph-based word sense induction
http://uhh-lt.github.io/sensegram
212 stars 50 forks source link

Wrong import statement and pcz module #25

Open Querela opened 5 years ago

Querela commented 5 years ago

https://github.com/uhh-lt/sensegram/blob/221c830c1c1adc64ffbdac19b5627722e06aa3c3/train.py#L13

Change to:

import pcz.disamgiguate_sense_clusters

Because:

Traceback (most recent call last):
  File "train.py", line 114, in <module>
    main()
  File "train.py", line 106, in main
    pcz.disambiguate_sense_clusters.run(clusters_with_isas_fpath, clusters_disambiguated_fpath)
AttributeError: module 'pcz' has no attribute 'disambiguate_sense_clusters'

And: https://github.com/uhh-lt/sensegram/blob/221c830c1c1adc64ffbdac19b5627722e06aa3c3/train.py#L106

Change to:

        pcz.disambiguate_sense_clusters.run(clusters_with_isas_fpath, clusters_disambiguated_fpath)

Because:

Traceback (most recent call last):
  File "train.py", line 114, in <module>
    main()
  File "train.py", line 106, in main
    pcz.disamgiguate_sense_clusters.run(clusters_with_isas_fpath, clusters_disambiguated_fpath)
AttributeError: module 'pcz' has no attribute 'disamgiguate_sense_clusters'
alexanderpanchenko commented 5 years ago

Thank you? Could you send a pull request? This way you will be among the contributors to the project.