Skip to content

test: disable the explain phase in Hypothesis testing#3985

Closed
TaiSakuma wants to merge 1 commit into
scikit-hep:mainfrom
TaiSakuma:test-hypothesis-default-profile
Closed

test: disable the explain phase in Hypothesis testing#3985
TaiSakuma wants to merge 1 commit into
scikit-hep:mainfrom
TaiSakuma:test-hypothesis-default-profile

Conversation

@TaiSakuma

@TaiSakuma TaiSakuma commented Apr 22, 2026

Copy link
Copy Markdown
Member

The explain phase in Hypothesis has an issue HypothesisWorks/hypothesis#4708.
We encountered errors due to this issue, e.g., #3891 (comment).
This PR turns off the phase.

Register a `default` profile in `tests/properties/conftest.py` that
disables `Phase.explain`. Workaround for
HypothesisWorks/hypothesis#4708; remove once
the upstream issue is resolved.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@TaiSakuma TaiSakuma changed the title test: disable the explain phase in the Hypothesis testing. test: disable the explain phase in Hypothesis testing. Apr 22, 2026
@TaiSakuma TaiSakuma changed the title test: disable the explain phase in Hypothesis testing. test: disable the explain phase in Hypothesis testing Apr 22, 2026
@TaiSakuma TaiSakuma changed the title test: disable the explain phase in Hypothesis testing test: disable the explain phase in Hypothesis testing Apr 22, 2026
@TaiSakuma
TaiSakuma requested a review from ikrommyd April 22, 2026 21:36
@codecov

codecov Bot commented Apr 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.27%. Comparing base (db977ac) to head (d7a3585).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@github-actions

Copy link
Copy Markdown

The documentation preview is ready to be viewed at http://preview.awkward-array.org.s3-website.us-east-1.amazonaws.com/PR3985

@ianna ianna left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@TaiSakuma - Thanks for fixing it. Please merge it if you finished with it. Thanks.

@TaiSakuma

Copy link
Copy Markdown
Member Author

This PR is no longer necessary as #3995 has been merged.

@TaiSakuma TaiSakuma closed this May 7, 2026
@TaiSakuma
TaiSakuma deleted the test-hypothesis-default-profile branch June 5, 2026 23:15
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.

2 participants