[AL-2075] Batch list and export #551
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
farkob
force-pushed
the
farkob/batch-features
branch
from
7b06216 to
7dee8ef
Compare
gdj0nes
reviewed
Apr 22, 2022
gdj0nes
reviewed
Apr 22, 2022
| @@ -23,3 +32,45 @@ class Batch(DbObject): | |||
| # Relationships | |||
| project = Relationship.ToOne("Project") | |||
| created_by = Relationship.ToOne("User") | |||
|
|
|||
| def export_data_rows(self, timeout_seconds=120) -> Generator: | |||
There was a problem hiding this comment.
is this the same output structure as dataset?
|
LGTM to me but @msokoloff1 should give final sign off |
gdj0nes
approved these changes
Apr 22, 2022
jtsodapop
approved these changes
Apr 22, 2022
msokoloff1
reviewed
Apr 27, 2022
labelbox/schema/project.py
Outdated
Comment on lines
858
to
863
| cursor_path={ | ||
| 'project': None, | ||
| 'batches': None, | ||
| 'pageInfo': None, | ||
| 'endCursor': None | ||
| }, |
There was a problem hiding this comment.
nit: I think the type hint for PaginatedCollection is wrong. This can probably just be a list of the keys.
There was a problem hiding this comment.
Yeah this looked wrong to me too
There was a problem hiding this comment.
If you can also change the type hint from :
cursor_path: Optional[Dict[str, Any]] = None
to
cursor_path: Optional[List[str]] = None
That would be awesome
jtsodapop
reviewed
Apr 27, 2022
| @@ -1,5 +1,14 @@ | |||
| # Changelog | |||
|
|
|||
| # Version 3.20.0 (2022-04-27) | |||
| ## Added | |||
There was a problem hiding this comment.
Since this is going to be a new version of the SDK, can we also include information on this merged PR?
Updated
NDJsonConverternow supports Video bounding box annotations.- Note: Currently does not support nested classifications.
- Note: Converting an export into Labelbox annotation types, and back to export will result in only keyframe annotations. This is to support correct import format.
cc @msokoloff1 if this wording looks okay
msokoloff1
approved these changes
Apr 27, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds
project.batches()batch.export_data_rows()(depends on intelligence pr #8171)Batch.project()fixbatch.remove_queued_data_rows()