Skip to content

Tweak strategies so we don't reject so many examples.#496

Open
dcherian wants to merge 2 commits intomainfrom
tweak-strategies
Open

Tweak strategies so we don't reject so many examples.#496
dcherian wants to merge 2 commits intomainfrom
tweak-strategies

Conversation

@dcherian
Copy link
Collaborator

  ┌─────────────────────────────────────┬────────┬────────┐
  │               Metric                │ Before │ After  │
  ├─────────────────────────────────────┼────────┼────────┤
  │ array.dtype.kind='f'                │ 1.10%  │ 8.49%  │
  ├─────────────────────────────────────┼────────┼────────┤
  │ array.dtype.kind='i'                │ 3.02%  │ 5.26%  │
  ├─────────────────────────────────────┼────────┼────────┤
  │ array.dtype.kind='u'                │ 2.06%  │ 10.36% │
  ├─────────────────────────────────────┼────────┼────────┤
  │ Invalid (line 101, not_overflowing) │ 27.57% │ 21.73% │
  ├─────────────────────────────────────┼────────┼────────┤
  │ Total invalid examples              │ 429    │ 289    │
  ├─────────────────────────────────────┼────────┼────────┤
  │ all-NaN groups                      │ 1.10%  │ 7.64%  │
  └─────────────────────────────────────┴────────┴────────┘

dcherian and others added 2 commits February 25, 2026 22:36
```
  ┌─────────────────────────────────────┬────────┬────────┐
  │               Metric                │ Before │ After  │
  ├─────────────────────────────────────┼────────┼────────┤
  │ array.dtype.kind='f'                │ 1.10%  │ 8.49%  │
  ├─────────────────────────────────────┼────────┼────────┤
  │ array.dtype.kind='i'                │ 3.02%  │ 5.26%  │
  ├─────────────────────────────────────┼────────┼────────┤
  │ array.dtype.kind='u'                │ 2.06%  │ 10.36% │
  ├─────────────────────────────────────┼────────┼────────┤
  │ Invalid (line 101, not_overflowing) │ 27.57% │ 21.73% │
  ├─────────────────────────────────────┼────────┼────────┤
  │ Total invalid examples              │ 429    │ 289    │
  ├─────────────────────────────────────┼────────┼────────┤
  │ all-NaN groups                      │ 1.10%  │ 7.64%  │
  └─────────────────────────────────────┴────────┴────────┘
  ```
- Add float32-aware tolerance (rtol=1e-5) in test_groupby_reduce_numpy_vs_other
  since chunked vs unchunked var/std on float32 with large values legitimately
  differs beyond the default rtol=1e-7.
- Relax test_scans_against_numpy tolerance from rtol=1e-13 to rtol=1e-12 to
  handle cumsum accumulation order differences with large float64 values.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

1 participant