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

Removed a comment blocking dataset import functionality #19

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

stellasphere
Copy link

Description

Please include a summary of the change and which issue is fixed or implemented. Please also include relevant motivation and context (e.g. links, docs, tickets etc.).

List any dependencies that are required for this change.

There was a comment blocking out the line where all the promises for uploading images would occur.

Resolving an issue raised by a forum user: https://discuss.roboflow.com/t/upload-dataset-from-gcp-vm-gcp-cloud-storage/2632

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

There was a comment blocking out the line where all the promises for uploading images would occur.
@hansent
Copy link
Contributor

hansent commented Jul 12, 2023

@stellasphere Wow, not sure how that would have ever worked like that with that commented out.

Did you test this at all to make sure the import command works with the call to Promise.all not commented out?

@CLAassistant
Copy link

CLAassistant commented Aug 11, 2023

CLA assistant check
All committers have signed the CLA.

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