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

Update smart_open pin to include v7.x #84

Merged
merged 3 commits into from
Mar 22, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion docs/tutorial/directory-and-assets.md
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ a **Git repo** instead.
### Downloading from a URL or cloud storage

Under the hood, Weasel uses the
[`smart-open`](https://github.com/RaRe-Technologies/smart_open) library so you
[`smart_open`](https://github.com/RaRe-Technologies/smart_open) library so you
can use any protocol it supports. Note that you may need to install extra
dependencies to use certain protocols.

Expand Down
2 changes: 1 addition & 1 deletion requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ wasabi>=0.9.1,<1.2.0
srsly>=2.4.3,<3.0.0
typer>=0.3.0,<0.10.0
cloudpathlib>=0.7.0,<1.0.0
smart-open>=5.2.1,<7.0.0
smart-open>=5.2.1,<8.0.0
# Third party dependencies
requests>=2.13.0,<3.0.0
pydantic>=1.7.4,!=1.8,!=1.8.1,<3.0.0
Expand Down
2 changes: 1 addition & 1 deletion setup.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ install_requires =
srsly>=2.4.3,<3.0.0
typer>=0.3.0,<0.10.0
cloudpathlib>=0.7.0,<1.0.0
smart-open>=5.2.1,<7.0.0
smart-open>=5.2.1,<8.0.0
requests>=2.13.0,<3.0.0
pydantic>=1.7.4,!=1.8,!=1.8.1,<3.0.0

Expand Down
2 changes: 1 addition & 1 deletion weasel/cli/pull.py
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ def project_pull_cli(
):
"""Retrieve available precomputed outputs from a remote storage.
You can alias remotes in your project.yml by mapping them to storage paths.
A storage can be anything that the smart-open library can upload to, e.g.
A storage can be anything that the smart_open library can upload to, e.g.
AWS, Google Cloud Storage, SSH, local directories etc.

DOCS: https://github.com/explosion/weasel/tree/main/docs/cli.md#arrow_down-push
Expand Down
4 changes: 2 additions & 2 deletions weasel/cli/push.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ def project_push_cli(
):
"""Persist outputs to a remote storage. You can alias remotes in your
project.yml by mapping them to storage paths. A storage can be anything that
the smart-open library can upload to, e.g. AWS, Google Cloud Storage, SSH,
the smart_open library can upload to, e.g. AWS, Google Cloud Storage, SSH,
local directories etc.

DOCS: https://github.com/explosion/weasel/tree/main/docs/cli.md#arrow_up-push
Expand All @@ -30,7 +30,7 @@ def project_push_cli(

def project_push(project_dir: Path, remote: str):
"""Persist outputs to a remote storage. You can alias remotes in your project.yml
by mapping them to storage paths. A storage can be anything that the smart-open
by mapping them to storage paths. A storage can be anything that the smart_open
library can upload to, e.g. gcs, aws, ssh, local directories etc
"""
config = load_project_config(project_dir)
Expand Down
2 changes: 1 addition & 1 deletion weasel/cli/remote_storage.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
class RemoteStorage:
"""Push and pull outputs to and from a remote file storage.

Remotes can be anything that `smart-open` can support: AWS, GCS, file system,
Remotes can be anything that `smart_open` can support: AWS, GCS, file system,
ssh, etc.
"""

Expand Down
Loading