Skip to content
This repository has been archived by the owner on Dec 21, 2023. It is now read-only.

turicreate.load_audio broken with home directory present. #2685

Closed
hoytak opened this issue Nov 19, 2019 · 2 comments · Fixed by #3026
Closed

turicreate.load_audio broken with home directory present. #2685

hoytak opened this issue Nov 19, 2019 · 2 comments · Fixed by #3026

Comments

@hoytak
Copy link
Collaborator

hoytak commented Nov 19, 2019

The following code generates a dataset that has no entries:

In [13]: import turicreate as tc
    ...:
    ...: data = tc.load_audio("~/data/ESC-50/audio/")

However, the same code where the ~ is replaced by the absolute path name works fine.

@hoytak hoytak assigned hoytak and unassigned hoytak Nov 19, 2019
@TobyRoseman
Copy link
Collaborator

A ~ in the path also doesn't work with tc.load_images. It does however work for tc.load_sarray and tc.load_sframe.

@TobyRoseman
Copy link
Collaborator

A ~ also works in the path for tc.load_model and tc.load_sgraph. So this just needs to be fixed in tc.load_audio and tc.load_images.

The functions that do this properly all seem to use the _make_internal_url method. It seems like using this will also fix other issues too (e.x. S3 paths).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants