Skip to content

[PLT-1489] Removed get stream and corresponding classes #1839

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

Merged
merged 7 commits into from
Sep 26, 2024

Conversation

Gabefire
Copy link
Collaborator

Description

  • get_stream is deprecated
  • Removed get_stream and corresponding classes for just get_buffered_stream

@Gabefire Gabefire requested a review from a team as a code owner September 24, 2024 00:55
@mihhail-m
Copy link
Collaborator

mihhail-m commented Sep 24, 2024

hey @Gabefire just want to double check as when get_stream got deprecated? Because currently this is the way Labelbox application suggests users to download their exports, especially for the big projects, datasets etc. Is the substitution for this deprecation already in-place or being developed?

Screenshot 2024-09-24 at 14 00 08

@Gabefire
Copy link
Collaborator Author

hey @mihhail-m just want to double check as when get_stream got deprecated? Because currently this is the way Labelbox application suggests users to download their exports, especially for the big projects, datasets etc. Is the substitution for this deprecation already in-place or being developed?

Screenshot 2024-09-24 at 14 00 08

Yeah, we need to write a ticket to replace that code block inside the UI. But get_stream is deprecated along with all its file converters in the SDK and our docs. https://docs.labelbox.com/reference/export-overview#streaming-results-and-errors The replaced method is get_buffered_stream we are cutting a new version of the SDK V6 specifically to remove all deprecated methods and get a fresh slate

@Gabefire Gabefire merged commit 0e18a26 into v6 Sep 26, 2024
19 of 26 checks passed
@Gabefire Gabefire deleted the gu/removed_get_stream branch September 26, 2024 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants