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.

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

DBP15K download url seems to be broken #3980

Closed
AndAgio opened this issue Jan 31, 2022 · 4 comments · Fixed by #4428
Closed

DBP15K download url seems to be broken #3980

AndAgio opened this issue Jan 31, 2022 · 4 comments · Fixed by #4428
Labels

Comments

@AndAgio
Copy link

AndAgio commented Jan 31, 2022

🐛 Describe the bug

The google drive file ID linked with the DBP15K dataset seems to point to a broken URL. Indeed, running a simple
dataset = torch_geometric.datasets.DBP15K(os.getcwd(), 'en_fr') returns a zipfile.BadZipFile: File is not a zip file.
This is due to the fact that the code downloads an empty zip file, which can't be unzipped. Digging a bit more it seems that the google drive URL used for the dataset download is not valid anymore. Is there a workaround?

Environment

  • PyG version: 2.0.1
  • PyTorch version: 1.9.0
  • OS: MacOS 12.0.1
  • Python version: 3.8.12
  • How you installed PyTorch and PyG (conda, pip, source): pip
@rusty1s
Copy link
Member

rusty1s commented Feb 1, 2022

Mh, you are right. What a bummer. It looks like the files are not longer available under this repo: https://github.com/syxu828/Crosslingula-KG-Matching. I created an issue here: syxu828/Crosslingula-KG-Matching#18

@nxznm
Copy link
Contributor

nxznm commented Apr 7, 2022

Hi @AndAgio @rusty1s
Recently, I obtained a copy of the DBP15K.zip from this issue, and I put this copy in my google drive. Here is the url: https://drive.google.com/open?id=1ggYlYf2_kTyi7oF9g07oTNn3VDhjl7so.
I also test this dataset with examples/dbp15k.py in the repo, and it works well.

@rusty1s
Copy link
Member

rusty1s commented Apr 7, 2022

This is awesome! Can you send a PR with the applied changes?

@nxznm
Copy link
Contributor

nxznm commented Apr 7, 2022

Yes, I'm glad to do it :)

@rusty1s rusty1s linked a pull request Apr 7, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants