Skip to content

Conversation

@jtsodapop
Copy link
Contributor

No description provided.

Copy link
Contributor

@kkim-labelbox kkim-labelbox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! One comment

}) for result in reader)
return (Entity.DataRow(
self.client, {
**result, 'customMetadata': result['metadata'],
Copy link
Contributor

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 result['customMetadata'], but can you double check by printing the result?

Copy link
Contributor Author

@jtsodapop jtsodapop Jul 14, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DataRow {'created_at': datetime.datetime(2022, 6, 1, 21, 13, 28, tzinfo=datetime.timezone.utc), 'external_id': '5668a62497e6edcd', 'media_attributes': {}, 'metadata': [DataRowMetadataField(schema_id='cl3w2pwsr000nj6ouhqebbf19', value=datetime.datetime(2021, 10, 13, 21, 1, 44, 34911, tzinfo=datetime.timezone.utc))], 'metadata_fields': [{'schemaId': 'cl3w2pwsr000nj6ouhqebbf19', 'name': 'capture_date', 'kind': 'CustomMetadataDateTime', 'value': '2021-10-13T21:01:44.034911Z'}], 'row_data': 'https://storage.googleapis.com/labelbox-datasets/open-images/data/5668a62497e6edcd.jpg', 'uid': 'cl3w355rxh5doj6ou1ftpffq7', 'updated_at': datetime.datetime(2022, 6, 1, 21, 13, 28, tzinfo=datetime.timezone.utc)}>

This is an example result. i think we said that customMetadata -> metadata which should be the flattened view while metadata_fields is the simplified version

Sorry, just realized this formatting is not great here.

Copy link
Contributor

@kkim-labelbox kkim-labelbox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@jtsodapop jtsodapop merged commit 66afe4e into develop Jul 20, 2022
@jtsodapop jtsodapop deleted the jtso/al-2659 branch July 20, 2022 16:47
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