search

uma-pi1 / kge

LibKGE - A knowledge graph embedding library for reproducible research
MIT License
776 stars 124 forks source link

Missing dataset.yaml files #217

Closed Filco306 closed 3 years ago

Filco306 commented 3 years ago

Hello!

I ran your quickstart instructions, and my I seem to be missing the dataset.yaml-file, even for the toy example, so it throws an error when I try the last line in the quickstart example. Would you guys have any idea why?

Thanks!

Cheers

rufex2001 commented 3 years ago

Did you run the download-all.sh script in the data folder?

On Mon, 16 Aug 2021 at 12:30, Filip Cornell @.***> wrote:

Hello!

I ran your quickstart instructions, and my I seem to be missing the dataset.yaml-file, even for the toy example, so it throws an error when I try the last line in the quickstart example. Would you guys have any idea why?

Thanks!

Cheers

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/uma-pi1/kge/issues/217, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABEWXZAFLD7YPBVLEE5YE4LT5DSCXANCNFSM5CHOQMSA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email .

Filco306 commented 3 years ago

Hello! Yes, I immediately found it now, haha. Apparently, python does not work for me; instead I had to change to python3 since python is not a command on my server, so I had to change those lines in the script. It is running now, will report back a bit later!

Either way, it might be good to change that since I guess most people are using python3 and not python, and python3 is always an accurate command, isn't it ?

rufex2001 commented 3 years ago

That depends on system settings. I get python 3 when using calling python.

On Mon, 16 Aug 2021 at 12:33, Filip Cornell @.***> wrote:

Hello! Yes, I immediately found it now, haha. Apparently, python does not work for me; instead I had to change to python3 since python is not a command on my server, so I had to change those lines in the script. It is running now, will report back a bit later!

Either way, it might be good to change that since I guess most people are using python3 and not python, and python3 is always an accurate command, isn't it ?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/uma-pi1/kge/issues/217#issuecomment-899404728, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABEWXZHZRMSU5GX6LJ5S44TT5DSQXANCNFSM5CHOQMSA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email .

Filco306 commented 3 years ago

Yes indeed. I just thought python3 might hold more generally :) either way, thanks for a great repository, I think it really makes KG embedding comparisons way easier!

rufex2001 commented 3 years ago

Thanks! That is the main point! Reproducibility!

On Mon, 16 Aug 2021 at 12:37, Filip Cornell @.***> wrote:

Yes indeed. I just thought python3 might hold more generally :) either way, thanks for a great repository, I think it really makes KG embedding comparisons way easier!

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/uma-pi1/kge/issues/217#issuecomment-899407080, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABEWXZCEYT736VZPPAHUXFLT5DS7NANCNFSM5CHOQMSA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email .