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

Use bytearray in cloud server #2495

Merged
merged 2 commits into from
Dec 14, 2022

Conversation

suzhoum
Copy link
Contributor

@suzhoum suzhoum commented Nov 29, 2022

Issue #, if available:

Description of changes:

This PR adds bytearray support in cloud module multimodal inference script.

Instead of saving encoded image bytearrays to disk and update the test data with image_path, we can now directly make calls to MultiModalPredictor with decoded bytearrays.

We have also added S_IMAGE_BYTEARRAY to feature generator.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@github-actions
Copy link

Job PR-2495-9d45f53 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-2495/9d45f53/index.html

@suzhoum suzhoum force-pushed the use_bytearray_in_cloud_server branch from 9d45f53 to 7b4d786 Compare December 1, 2022 23:42
@github-actions
Copy link

github-actions bot commented Dec 2, 2022

Job PR-2495-7b4d786 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-2495/7b4d786/index.html

@suzhoum suzhoum force-pushed the use_bytearray_in_cloud_server branch 11 times, most recently from 7feb3a2 to 78a44a0 Compare December 13, 2022 20:22
@suzhoum suzhoum force-pushed the use_bytearray_in_cloud_server branch from 78a44a0 to 30792f1 Compare December 13, 2022 23:14
@github-actions
Copy link

Job PR-2495-30792f1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-2495/30792f1/index.html

@suzhoum suzhoum force-pushed the use_bytearray_in_cloud_server branch from 30792f1 to a08f258 Compare December 14, 2022 00:49
@suzhoum suzhoum marked this pull request as ready for review December 14, 2022 00:51
@github-actions
Copy link

Job PR-2495-a08f258 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-2495/a08f258/index.html

Copy link
Contributor

@Innixma Innixma left a comment

Choose a reason for hiding this comment

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

LGTM on the autogluon.features side, thanks! Weisu, please review the cloud side.

Copy link
Collaborator

@yinweisu yinweisu left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for the work!

@suzhoum suzhoum merged commit 6bcbf0d into autogluon:master Dec 14, 2022
yinweisu pushed a commit to autogluon/autogluon-cloud that referenced this pull request Dec 21, 2022
@suzhoum suzhoum added this to the 0.7 Release milestone Feb 16, 2023
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.

None yet

3 participants