Skip to content
This repository has been archived by the owner on Nov 29, 2023. It is now read-only.

samples.analyze.video_detect_faces_beta_test: test_detect_faces failed #56

Closed
flaky-bot bot opened this issue Oct 6, 2020 · 3 comments · Fixed by #58
Closed

samples.analyze.video_detect_faces_beta_test: test_detect_faces failed #56

flaky-bot bot opened this issue Oct 6, 2020 · 3 comments · Fixed by #58
Assignees
Labels
api: videointelligence Issues related to the googleapis/python-videointelligence API. priority: p2 Moderately-important priority. Fix may not be included in next release. samples Issues that are directly related to samples. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. type: process A process-related concern. May include testing, release, or the like.

Comments

@flaky-bot
Copy link

flaky-bot bot commented Oct 6, 2020

This test failed!

To configure my behavior, see the Build Cop Bot documentation.

If I'm commenting on this issue too often, add the buildcop: quiet label and
I will stop commenting.


commit: f743c0a
buildURL: Build Status, Sponge
status: failed

Test output
capsys = <_pytest.capture.CaptureFixture object at 0x7fa6fc5620f0>
@pytest.mark.flaky(max_runs=3, min_passes=1)
def test_detect_faces(capsys):
    local_file_path = os.path.join(RESOURCES, "googlework_short.mp4")

    video_detect_faces_beta.detect_faces(local_file_path=local_file_path)

    out, _ = capsys.readouterr()

    assert "Face detected:" in out
  assert "Attributes:" in out

E AssertionError: assert 'Attributes:' in '\nProcessing video for face detection annotations.\n\nFinished processing.\n\nFace detected:\nFace detected:\nFace de...ted:\nFace detected:\nFace detected:\nFace detected:\nFace detected:\nFace detected:\nFace detected:\nFace detected:\n'

video_detect_faces_beta_test.py:33: AssertionError

@flaky-bot flaky-bot bot added buildcop: issue priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. labels Oct 6, 2020
@product-auto-label product-auto-label bot added api: videointelligence Issues related to the googleapis/python-videointelligence API. samples Issues that are directly related to samples. labels Oct 6, 2020
danoscarmike added a commit that referenced this issue Oct 8, 2020
danoscarmike added a commit that referenced this issue Oct 9, 2020
@flaky-bot
Copy link
Author

flaky-bot bot commented Oct 10, 2020

Looks like this issue is flaky. 😟

I'm going to leave this open and stop commenting.

A human should fix and close this.


commit: ee1ada6
buildURL: Build Status, Sponge
status: failed

Test output
capsys = <_pytest.capture.CaptureFixture object at 0x7fc1b35a68d0>
@pytest.mark.flaky(max_runs=3, min_passes=1)
def test_detect_faces(capsys):
    local_file_path = os.path.join(RESOURCES, "googlework_short.mp4")

    video_detect_faces_beta.detect_faces(local_file_path=local_file_path)

    out, _ = capsys.readouterr()

    assert "Face detected:" in out
  assert "Attributes:" in out

E AssertionError: assert 'Attributes:' in '\nProcessing video for face detection annotations.\n\nFinished processing.\n\nFace detected:\nFace detected:\nFace de...ted:\nFace detected:\nFace detected:\nFace detected:\nFace detected:\nFace detected:\nFace detected:\nFace detected:\n'

video_detect_faces_beta_test.py:33: AssertionError

@flaky-bot flaky-bot bot reopened this Oct 10, 2020
@yoshi-automation yoshi-automation added the 🚨 This issue needs some love. label Oct 13, 2020
@munkhuushmgl munkhuushmgl self-assigned this Oct 13, 2020
@flaky-bot
Copy link
Author

flaky-bot bot commented Oct 14, 2020

Oops! Looks like this issue is still flaky. It failed again. 😬

I reopened the issue, but a human will need to close it again.


commit: 07806d3
buildURL: Build Status, Sponge
status: failed

Test output
capsys = <_pytest.capture.CaptureFixture object at 0x7f35c0367e48>
@pytest.mark.flaky(max_runs=3, min_passes=1)
def test_detect_faces(capsys):
    local_file_path = os.path.join(RESOURCES, "googlework_short.mp4")

    video_detect_faces_beta.detect_faces(local_file_path=local_file_path)

    out, _ = capsys.readouterr()

    assert "Face detected:" in out
  assert "Attributes:" in out

E AssertionError: assert 'Attributes:' in '\nProcessing video for face detection annotations.\n\nFinished processing.\n\nFace detected:\nFace detected:\nFace de...ted:\nFace detected:\nFace detected:\nFace detected:\nFace detected:\nFace detected:\nFace detected:\nFace detected:\n'

video_detect_faces_beta_test.py:33: AssertionError

@flaky-bot flaky-bot bot reopened this Oct 14, 2020
@munkhuushmgl munkhuushmgl added priority: p2 Moderately-important priority. Fix may not be included in next release. type: process A process-related concern. May include testing, release, or the like. and removed 🚨 This issue needs some love. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. labels Oct 14, 2020
@yoshi-automation yoshi-automation added 🚨 This issue needs some love. and removed 🚨 This issue needs some love. labels Jan 12, 2021
@munkhuushmgl
Copy link
Contributor

Should be fixed by now. do not know why it kept referring to old version of samples

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api: videointelligence Issues related to the googleapis/python-videointelligence API. priority: p2 Moderately-important priority. Fix may not be included in next release. samples Issues that are directly related to samples. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. type: process A process-related concern. May include testing, release, or the like.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants