Conversation
- Make brew update non-fatal - Add fallback logic for pip installation (try pip3, python3 -m pip, or ensurepip first) - This fixes the 'Install Dependencies' step failure on macOS runners
Add --user flag to all pip install commands on macOS to comply with PEP 668 restrictions in Python 3.12+. This installs sympy to the user directory instead of system-wide, which is allowed and safe.
…kages - Add --break-system-packages flag along with --user to comply with PEP 668 - Make sympy installation non-fatal using set +e/set -e - Since sympy is optional (tests work without it), continue even if installation fails - This fixes the externally-managed-environment error on macOS CI runners
- Simplify installation approach: use --break-system-packages flag - Ensure pip is available and up to date before installing sympy - Remove error handling that made installation optional - This ensures sympy is actually installed for tests to use
- Wrap sympy import in try/except to handle missing sympy gracefully - Add _sympy_available flag to track sympy availability - Automatically disable sympy checking when sympy is not available - Add guards in SympyWrapper methods to handle missing sympy - This allows tests to run even when sympy is not installed
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
1 participant
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.
No description provided.