[BUG] hub.copy does not support pathlib.Path arguments

🐛🐛 Bug Report

hub.copy does not seem to play well with pathlib.Path. Since pathlib.Path seems to be the up and coming new standard for Python3 paths, I think it would greatly reduce friction and increase compatibility if it was supported.

⚗️ Current Behavior

Currently, the hub.copy method’s signature looks like:

@staticmethod
    def copy(
        src: Union[str, Dataset],
        dest: str,
        overwrite: bool = False,
        src_creds=None,
        src_token=None,
        dest_creds=None,
        dest_token=None,
        num_workers: int = 0,
        scheduler="threaded",
        progressbar=True,
    ):

For src and dest we should include pathlib.Path as a potential type. One can easily convert a pathlib.Path to a string path if necessary by using

path_to_string = lambda x: os.fspath(x.resolve())

⚙️ Environment

• Python version(s): python 3.8.12 ha38a3c6_3_cpython conda-forg
• OS: [e.g. Ubuntu 20.04]
• IDE: [PyCharm]
• Packages: [ Pytorch-latest]

[farizrahman4u]

@dhiganthrao we have a PR up: https://github.com/activeloopai/Hub/pull/1651

[tatevikh]

Hi @AntreasAntoniou thanks for raising the issue. Will let you know as soon as this is fixed.