Closed zexuan-zhou closed 2 years ago
@rsmith49 @bhancock8 Please take a look. I need some help on the tests for py36, py37, and spark, and CICD. Thank you
@zexuan-zhou thanks for getting going here! Can you re-create this PR to merge into snorkel-team:update-numpy-version
instead of snorkel-team:master
? That way we can run the checks in CI after merging in these changes before merging into master
@zexuan-zhou thanks for getting going here! Can you re-create this PR to merge into
snorkel-team:update-numpy-version
instead ofsnorkel-team:master
? That way we can run the checks in CI after merging in these changes before merging into master
@rsmith49 Done. Changed the base to snorkel-team:update-numpy-version
Description of proposed changes
Loosen versions for
numpy
,blis
to support M1 MacRelated issue(s)
https://github.com/snorkel-team/snorkel/issues/1695
Fixes # (issue) https://github.com/snorkel-team/snorkel/issues/1695
Test plan
Need help on this
Checklist
Need help on these? Just ask! Need help to test py36 and py37 as they are hard to install on M1 Mac. Need help on
tox -e spark
Need help on CICD Need help on what document to update Need help on whether there should be new test casestox -e complex
and/ortox -e spark
if appropriate. [X]tox -e complex
runs and passes all test [ ]tox -e spark
is currently failing withException: Java gateway process exited before sending its port number
tox -e py38
passes [X]tox -e complex
passes [X]tox -e doctest
passes [X]tox -e check
passes [X]tox -e type
passes (all code changes are regarding this) [ ]tox -e py36 # Run unit tests pytest in Python 3.6
[ ]tox -e py37 # Run unit tests pytest in Python 3.7
[ ]tox -e spark # Run Spark-based tests (marked with @pytest.mark.spark)
** [ ]tox -e doc # Build documentation with Sphinx