Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Experimental PR #2

Closed
wants to merge 9 commits into from
Closed

Experimental PR #2

wants to merge 9 commits into from

Conversation

trundev
Copy link
Owner

@trundev trundev commented Dec 11, 2024

Experimental GitHub pull request

This is to test the fixes in GitHub readthedocs Webhook and overall Pull request workflow.

trundev and others added 9 commits December 8, 2024 20:42
Issues detected by flake8:
4     E275 missing whitespace after keyword
1     E721 do not compare types, for exact checks use `is` / `is not`, for instance checks use `isinstance()`
6     F401 'random' imported but unused
5     F841 local variable 'biv2' is assigned to but never used
* scipy.e is deprecated, use numpy.e instead
  see scipy/scipy@6e207a1
* Migrate to the latest github actions
* Replace the unsupported python 3.5 with 3.8
* Use python 3.11 instead of 3.8, and 3.12 instead of 3.9
Remove leftovers from python 2 compatibility

Co-authored-by: Eric Wieser <[email protected]>
Done by avoid the numpy.result_type (not supported by numba). Instead
do the array multiplication at-once, then use the result dtype.
This requires some numpy array-indexing tricks.
The test_function_cache is rewritten to use @njit, which may not make
much sense for particular test.
In various `__repr__()` implementations, replace 'list(ndarray)' with
'ndarray.tolist()', in order to avoid inclusion of type information
(like 'np.int64(1)') by the latest numpy version.

This also fixes test_clifford.TestPrettyRepr and test_transformations
pytest-s, as they rely on hardcoded results from IPython.lib.pretty.

Note: numpy>=2.0 is allowed by numba>=0.60.
Replace the deprecated @generated_jit by @njit
Add 'build.os' config value as suggested in readthedocs#11173.
Use python 3.9, as it is the latest version that works - needs removal
of the 'sphinx' version limit.
@codecov-commenter
Copy link

codecov-commenter commented Dec 11, 2024

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 78.12500% with 7 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
clifford/__init__.py 61.53% 5 Missing ⚠️
clifford/cga.py 0.00% 1 Missing ⚠️
clifford/tools/g3c/rotor_estimation.py 0.00% 1 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Files with missing lines Coverage Δ
clifford/_conformal_layout.py 100.00% <100.00%> (ø)
clifford/_layout.py 96.95% <100.00%> (ø)
clifford/_multivector.py 81.39% <100.00%> (ø)
clifford/_numba_utils.py 71.87% <ø> (ø)
clifford/test/test_clifford.py 99.06% <100.00%> (ø)
clifford/test/test_function_cache.py 100.00% <100.00%> (ø)
clifford/test/test_g3c_tools.py 80.94% <100.00%> (ø)
clifford/cga.py 86.01% <0.00%> (ø)
clifford/tools/g3c/rotor_estimation.py 48.92% <0.00%> (ø)
clifford/__init__.py 82.85% <61.53%> (ø)

@trundev trundev closed this Dec 12, 2024
@trundev trundev deleted the master branch December 12, 2024 09:34
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.

3 participants