Simplify usage of Google Drive

[jfthuong]

Is your feature request related to a problem? Please describe. Fastai uses fastbook.setup_book as an equivalent to mount Google Drive. This is weird and not very specific. Moreover, it uses the name that is not the default in the code template of Google Colab to mount (/content/drive vs /content/gdrive).

Describe the solution you'd like We could create a GoogleDrive class with different methods:

• mount: mount Google Drive (with the code to copy, or any other simpler solution if that exists)
• enter to do a "cd" and start working in Google Drive instead of /content which is the default)
• copy to copy a file
• move
• delete
• ls to list files (with potential specification of file type like image, text, csv, archive, zip, pkl, all)

Describe alternatives you've considered An alternative is to juste create a mount_gdrive function that could return the Path of mounted Google Drive.

Additional context N/A

[raynardj]

I'm more up for

create a mount_gdrive function that could return the Path of mounted Google Drive.

As write up entire file system commandlines in python functions are

• already done here, we'll end up wrapping functions on these ones as a format
• enter, copy, move, delete for google drive does not differ from enter, copy, move, delete for other part of the hard drive

[raynardj]

so the function will do the following code?

def mount_gdrive(path='/content/drive'):
    from google.colab import drive
    drive.mount(path)

[vtecftwy]

Good points from @raynardj Also, fastai has added a ls() method to pathlib.Path, and would make the new class ls method kind of redundant.

[raynardj]

Also, fastai has added a ls() method to pathlib.Path, and would make the new class ls method kind of redundant.

Love that feature.

[jfthuong]

so the function will do the following code?

def mount_gdrive(path='/content/drive'):
    from google.colab import drive
    drive.mount(path)

Kind of. We shall also handle the case where we are not in Google Drive, avoid trying to mount twice, and maybe storing where it has been mounted (just few more line of codes :) ).

Also, fastai has added a ls() method to pathlib.Path, and would make the new class ls method kind of redundant.

Actually they added more than just ls:

# fastcore/xtras.py

# Cell
@patch
def readlines(self:Path, hint=-1, encoding='utf8'):
    "Read the content of `self`"
    with self.open(encoding=encoding) as f: return f.readlines(hint)

# Cell
@patch
def read_json(self:Path, encoding=None, errors=None):
    "Same as `read_text` followed by `loads`"
    return loads(self.read_text(encoding=encoding, errors=errors))

# Cell
@patch
def mk_write(self:Path, data, encoding=None, errors=None, mode=511):
    "Make all parent dirs of `self`, and write `data`"
    self.parent.mkdir(exist_ok=True, parents=True, mode=mode)
    self.write_text(data, encoding=encoding, errors=errors)

# Cell
@patch
def relpath(self:Path, start=None):
    "Same as `os.path.relpath`, but returns a `Path`, and resolves symlinks"
    return Path(os.path.relpath(self.resolve(), Path(start).resolve()))

# Cell
@patch
def ls(self:Path, n_max=None, file_type=None, file_exts=None):
    "Contents of path as a list"
    extns=L(file_exts)
    if file_type: extns += L(k for k,v in mimetypes.types_map.items() if v.startswith(file_type+'/'))
    has_extns = len(extns)==0
    res = (o for o in self.iterdir() if has_extns or o.suffix in extns)
    if n_max is not None: res = itertools.islice(res, n_max)
    return L(res)

# Cell
@patch
def __repr__(self:Path):
    b = getattr(Path, 'BASE_PATH', None)
    if b:
        try: self = self.relative_to(b)
        except: pass
    return f"Path({self.as_posix()!r})"

# Cell
@patch
def delete(self:Path):
    "Delete a file, symlink, or directory tree"
    if not self.exists(): return
    if self.is_dir(): shutil.rmtree(self)
    else: self.unlink()