feat: add GitHub Actions CI workflow for automated testing#26
Open
westonplatter wants to merge 2 commits intomarcdemers:mainfrom
Open
feat: add GitHub Actions CI workflow for automated testing#26westonplatter wants to merge 2 commits intomarcdemers:mainfrom
westonplatter wants to merge 2 commits intomarcdemers:mainfrom
Conversation
- Create .github/workflows/ci.yml with Python 3.10-3.13 matrix - Update setup.py with test extras (pytest) and modern Python classifiers - Update numba requirement to >=0.61.0 for Python 3.10+ support - Fix test file paths in test_entrypoints.py to use pathlib for proper resolution when running from project root side note: CI will fail until py_lets_be_rational dependency issue is resolved. The package imports from _testcapi which is unavailable in standard Python. See vollib/py_vollib#27 - fix exists in repo but is not published to PyPI.
knaps
added a commit
to knaps/py_vollib_vectorized
that referenced
this pull request
Mar 27, 2026
…s#19, marcdemers#20, marcdemers#21, marcdemers#26) - Fix model='black' crash: remove extra `b` arg from 5 original greeks, fix wrong arg order in gamma_black and syntax error in rho_black (marcdemers#20) - Fix pandas index-alignment in price_dataframe() with np.asarray() (marcdemers#21) - Fix vectorized_black discounting: apply exp(-r*t) to match py_vollib (marcdemers#19) - Switch all S/F numerical bumps from absolute (dS=0.01) to relative (h=S*1e-4), preventing NaN for small spot prices (marcdemers#18) - Fix test file paths with pathlib so tests run from project root (marcdemers#26) - Add GitHub Actions CI workflow, update Python classifiers to 3.10-3.13, bump numba>=0.61.0, replace distutils with setuptools (marcdemers#26) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.
side note: CI will fail until py_lets_be_rational dependency issue is resolved. The package imports from _testcapi which is unavailable in standard Python. See vollib/py_vollib#27 - fix exists in repo but is not published to PyPI.
Example of the Python 3.10-3.14 tests passing => https://github.com/westonplatter/py_vollib_vectorized/actions/runs/21319719922
Closes #23