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
Track artifacts on S3 compatible storage #1053
Conversation
7d864d8
to
13365ef
Compare
Test Results 2 files ± 0 2 suites ±0 1m 56s ⏱️ - 26m 9s Results for commit 56b00c2. ± Comparison against base commit 9a0e977. This pull request removes 267 and adds 482 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
boto3 `endpoint_url` support PR: | ||
* https://github.com/boto/boto3/pull/2746 | ||
""" | ||
endpoint_url = os.getenv("S3_ENDPOINT_URL") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably this should be a part of neptune.new.envs
CHANGELOG.md
Outdated
@@ -11,6 +11,7 @@ | |||
- `get_last_run`, `get_run_url`, `get_project` and `neptune.init` marked as deprecated ([#1011](https://github.com/neptune-ai/neptune-client/pull/1011)) | |||
- Deprecated implicit casting of objects to strings with `log` and `assign` operations ([#1028](https://github.com/neptune-ai/neptune-client/pull/1028)) | |||
- Internally extracted legacy client to `legacy` submodule ([#1039](https://github.com/neptune-ai/neptune-client/pull/1039)) | |||
- Track artifacts on S3 compatible storage ([#1053](https://github.com/neptune-ai/neptune-client/pull/1053)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this should be treated as a feature.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
right
No description provided.