You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@beam.apache.org by "bvolpato (via GitHub)" <gi...@apache.org> on 2026/08/07 04:09:04 UTC

[PR] [Python] Deflake anomaly transform tests [beam]

bvolpato opened a new pull request, #39669:
URL: https://github.com/apache/beam/pull/39669

   Fixes #37779.
   
   `RunOneDetector` reshuffles input before scoring. Existing transform tests used online Z-score detectors and matched each example against scores calculated in creation order. Runner scheduling can legally change that order independently for each ensemble branch, changing detector state and scores. This caused the keyed failure reported in #37779 and a new unkeyed failure in [Python ML precommit](https://github.com/apache/beam/actions/runs/31140935654/job/92750492308).
   
   Replace transform fixtures with an order-independent single-feature detector. Coverage remains for keyed and unkeyed inputs, feature selection, thresholds, ensemble grouping, and `AnyVote` aggregation. Z-score learning behavior remains covered by `detectors/zscore_test.py`.
   
   ## Reproduction
   
   Forcing one detector branch to consume reversed input reproduces score mismatches before this change. Reversing keyed and unkeyed inputs passes all six anomaly transform cases afterward.
   
   ## Testing
   
   - `./gradlew :sdks:python:test-suites:tox:py313:testPython313 -Pposargs='apache_beam.ml.anomaly.transforms_test'` (12 passed)
   - Reversed-order keyed and unkeyed transform cases (6 passed)
   - YAPF, Ruff, and Pyrefly checks
   
   No `CHANGES.md` entry. Test-only change.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [Python] Deflake anomaly transform tests [beam]

Posted by "bvolpato (via GitHub)" <gi...@apache.org>.
bvolpato commented on PR #39669:
URL: https://github.com/apache/beam/pull/39669#issuecomment-5532788730

   Run Python_ML PreCommit 3.10
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [Python] Deflake anomaly transform tests [beam]

Posted by "claudevdm (via GitHub)" <gi...@apache.org>.
claudevdm merged PR #39669:
URL: https://github.com/apache/beam/pull/39669


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [Python] Deflake anomaly transform tests [beam]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #39669:
URL: https://github.com/apache/beam/pull/39669#issuecomment-5614413134

   Assigning reviewers:
   
   R: @claudevdm for label python.
   
   
   Note: If you would like to opt out of this review, comment `assign to next reviewer`.
   
   Available commands:
   - `stop reviewer notifications` - opt out of the automated review tooling
   - `remind me after tests pass` - tag the comment author after tests pass
   - `waiting on author` - shift the attention set back to the author (any comment or push by the author will return the attention set to the reviewers)
   
   The PR bot will only process comments in the main thread (not review comments).


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [Python] Deflake anomaly transform tests [beam]

Posted by "bvolpato (via GitHub)" <gi...@apache.org>.
bvolpato commented on PR #39669:
URL: https://github.com/apache/beam/pull/39669#issuecomment-5532388436

   Run Python_ML PreCommit 3.13


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [Python] Deflake anomaly transform tests [beam]

Posted by "bvolpato (via GitHub)" <gi...@apache.org>.
bvolpato commented on PR #39669:
URL: https://github.com/apache/beam/pull/39669#issuecomment-5532811035

   Run Python_ML PreCommit 3.10


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org