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

Ensure that dependencies are added to the operatorList for locally cached images (issue 18259) #18269

Merged
merged 1 commit into from
Jun 18, 2024

Conversation

Snuffleupagus
Copy link
Collaborator

No description provided.

@Snuffleupagus Snuffleupagus linked an issue Jun 18, 2024 that may be closed by this pull request
@Snuffleupagus Snuffleupagus changed the title Ensure that dependencies are added to the operatorList for locally cached images (issue ´) Ensure that dependencies are added to the operatorList for locally cached images (issue 18259) Jun 18, 2024
@Snuffleupagus
Copy link
Collaborator Author

/botio test

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Linux m4)


Received

Command cmd_test from @Snuffleupagus received. Current queue size: 0

Live output at: http://54.241.84.105:8877/6e0ea539a1a9e89/output.txt

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Windows)


Received

Command cmd_test from @Snuffleupagus received. Current queue size: 0

Live output at: http://54.193.163.58:8877/c7c5f7714d511d0/output.txt

Copy link
Contributor

@calixteman calixteman left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you.

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Linux m4)


Failed

Full output at http://54.241.84.105:8877/6e0ea539a1a9e89/output.txt

Total script time: 28.63 mins

  • Unit tests: Passed
  • Integration Tests: Passed
  • Regression tests: FAILED
  different ref/snapshot: 13
  different first/second rendering: 1

Image differences available at: http://54.241.84.105:8877/6e0ea539a1a9e89/reftest-analyzer.html#web=eq.log

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Windows)


Failed

Full output at http://54.193.163.58:8877/c7c5f7714d511d0/output.txt

Total script time: 44.11 mins

  • Unit tests: Passed
  • Integration Tests: Passed
  • Regression tests: FAILED
  different ref/snapshot: 3

Image differences available at: http://54.193.163.58:8877/c7c5f7714d511d0/reftest-analyzer.html#web=eq.log

@Snuffleupagus
Copy link
Collaborator Author

Missed one case...

/botio test

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Linux m4)


Received

Command cmd_test from @Snuffleupagus received. Current queue size: 0

Live output at: http://54.241.84.105:8877/4f8a8bd7a5fea7f/output.txt

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Windows)


Received

Command cmd_test from @Snuffleupagus received. Current queue size: 0

Live output at: http://54.193.163.58:8877/205590cf8f15436/output.txt

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Linux m4)


Failed

Full output at http://54.241.84.105:8877/4f8a8bd7a5fea7f/output.txt

Total script time: 28.68 mins

  • Unit tests: Passed
  • Integration Tests: Passed
  • Regression tests: FAILED
  different ref/snapshot: 17
  different first/second rendering: 2

Image differences available at: http://54.241.84.105:8877/4f8a8bd7a5fea7f/reftest-analyzer.html#web=eq.log

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Windows)


Failed

Full output at http://54.193.163.58:8877/205590cf8f15436/output.txt

Total script time: 42.02 mins

  • Unit tests: Passed
  • Integration Tests: FAILED
  • Regression tests: FAILED
  different ref/snapshot: 9

Image differences available at: http://54.193.163.58:8877/205590cf8f15436/reftest-analyzer.html#web=eq.log

@Snuffleupagus
Copy link
Collaborator Author

Snuffleupagus commented Jun 18, 2024

Locally I get less "severe" movement when testing on Windows, perhaps this is worse with devicePixelRatio === 1 since I believe the bots use that?
Perhaps the QueueOptimizer logic is also affecting things, now that we're correctly handling cached imageMask-data as well?

In any case, given that this patch ought to be correct are we OK with the ref-test movement in the latest round of testing?

@timvandermeij
Copy link
Contributor

I have checked locally that this indeed solves the issue. I have also checked the last round of reference tests and didn't find anything that was either noticeable with the naked eye or that was unexpected movement, and given that I agree that this patch should be correct I think we can do this. Thanks!

@timvandermeij timvandermeij merged commit 4c82438 into mozilla:master Jun 18, 2024
9 checks passed
@timvandermeij
Copy link
Contributor

/botio makeref

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Windows)


Received

Command cmd_makeref from @timvandermeij received. Current queue size: 0

Live output at: http://54.193.163.58:8877/018642573633292/output.txt

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Linux m4)


Received

Command cmd_makeref from @timvandermeij received. Current queue size: 0

Live output at: http://54.241.84.105:8877/2645d88b795f3f7/output.txt

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Linux m4)


Success

Full output at http://54.241.84.105:8877/2645d88b795f3f7/output.txt

Total script time: 19.88 mins

  • Make references: Passed
  • Check references: Passed

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Windows)


Success

Full output at http://54.193.163.58:8877/018642573633292/output.txt

Total script time: 25.13 mins

  • Make references: Passed
  • Check references: Passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix reference test "pr12564"
4 participants