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.

Sign up for GitHub

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

Jump to bottom

vision: address flakes due to collisions. #1458

Merged
merged 5 commits into from
Jun 11, 2019
Merged

vision: address flakes due to collisions. #1458

merged 5 commits into from
Jun 11, 2019

Commits on Jun 11, 2019

  1. vision: address flakes due to collisions. 

    Multiple tests in the vision suite write output to GCS.  Sadly, all wrote to the
same output prefix in a nondeterministic order, which can cause issues.  This change does two things:

1) ensures each test run uses a unique prefix
2) makes the prefix clearer to disambiguate output from the cloud_client and the beta samples.

I suspect the beta samples should be cleaned up, but it appears there's still linkages into cloud docs which is a more involved change.  This should get the
tests green once more.
    @shollyman
    shollyman committed Jun 11, 2019
    Configuration menu
    Copy the full SHA
    8da1edf View commit details
    Browse the repository at this point in the history

  2. Merge branch 'master' into visionfixes

    @shollyman
    shollyman committed Jun 11, 2019
    Configuration menu
    Copy the full SHA
    a87653f View commit details
    Browse the repository at this point in the history

  3. beta: convert batch future to actually wait for output properly.

    @shollyman
    shollyman committed Jun 11, 2019
    Configuration menu
    Copy the full SHA
    98223de View commit details
    Browse the repository at this point in the history

  4. explicit imports

    @shollyman
    shollyman committed Jun 11, 2019
    Configuration menu
    Copy the full SHA
    01ed157 View commit details
    Browse the repository at this point in the history

  5. additional explicit import

    @shollyman
    shollyman committed Jun 11, 2019
    Configuration menu
    Copy the full SHA
    d6c945c View commit details
    Browse the repository at this point in the history