Repository navigation
Fix bfabric-cli tracebacks on user errors and SIGILL on CPUs without AVX2 - #646
Merged
Merged
Conversation
added 3 commits
October 29, 2026 13:18
…t AVX2 polars only warns about missing CPU features and then the process dies with 'Illegal instruction'. Turn that one warning into an error when bfabric is imported, so the user sees the 'install polars[rtcompat]' hint.
…ceback Wrap suds faults, unknown query fields and lost connections in BfabricRequestError / BfabricUnavailableError, report ValueError, OSError, YAML and HTTP errors from use_client as a single Error: line, and add the report_user_errors context manager for the same handling elsewhere. An unknown config environment raises a ValueError listing the available ones.
… pytest A newer pydantic makes basedpyright reject TypeAdapter(Annotated alias); add targeted ignores. Wrapping the suds calls in _translate_call_errors moved the baseline-grandfathered reportAny lines, so ignore them in place (basedpyright pruned the now-stale baseline entries). pytest 9.2 removed FixtureManager._arg2fixturedefs, which pytest-bdd 9.0 still uses; cap pytest in the asgi_auth test extra.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
bfabric-clito report user errors (unknown query field, SOAP fault, lost connection, missing or malformed config, unknown config env, HTTP errors) in oneError:line instead of a tracebackreport_user_errorsto apply the same handling outsideuse_clientbfabricon a CPU without AVX2 to raise polars' "installpolars[rtcompat]" hint instead of dying with "Illegal instruction"