-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
CI: Setup 3.9 Travis Build #33505
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
CI: Setup 3.9 Travis Build #33505
Conversation
Test failures currently related to: #33507 |
@alimcmaster1 did we determine this needs to wait on an upstream fix? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let us know when ready
ci/build39.sh
Outdated
python -c "import hypothesis" | ||
|
||
# TODO: Is there anything else in setup_env that we really want to do? | ||
# ci/setup_env.sh |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add a line at the end
minor comments
Yes waiting on a fix to the current cython alpha - cython/cython#3544. (We cant use current cython release due to a different issue - cython/cython#3395) Will ping on here when ready. |
@alimcmaster1 you may need to manually skip the clipboard tests on this build. |
/azp run |
Commenter does not have sufficient privileges for PR 33505 in repo pandas-dev/pandas |
Azure pipelines don't seem to be triggering |
@pandas-dev/pandas-core 3.9 build is ready. (Azure seems to be playing up - all jobs have ran but doesn’t seem to be reporting back to my PR) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can u add an issue to revert this once conda builds are good
a question about wheels
ci/build39.sh
Outdated
pip install cython --pre # https://github.com/cython/cython/issues/3395 | ||
|
||
# Alternative for getting numpy: | ||
# pip install --pre -f https://7933911d6844c6c53a7d-47bd50c35cd79bd838daf386af554a83.ssl.cf2.rackcdn.com/ numpy |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do these wheels work yet?
ci/build39.sh
Outdated
python setup.py install | ||
cd .. | ||
rm -rf numpy | ||
pip install --pre -f https://7933911d6844c6c53a7d-47bd50c35cd79bd838daf386af554a83.ssl.cf2.rackcdn.com numpy |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this might not be the right address iirc this changed
maybe have to look on the numpy issue list
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Its the same address we use in the numpy dev build? - https://github.com/pandas-dev/pandas/blob/master/ci/deps/azure-37-numpydev.yaml#L19
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
these might be older:
https://anaconda.org/scipy-wheels-nightly/numpy/files
i think is the new location (includes scipy as well)
let’s update
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ahh nice - thanks wasn't aware! Will update the numpy dev build too in a separate PR.
thanks!
all green & raised an issue for later follow ups @jreback |
Resorted to building numpy from source:
|
@jreback @jbrockmendel any comments here? All green |
pip install cython --pre # https://github.com/cython/cython/issues/3395 | ||
|
||
git clone https://github.com/numpy/numpy | ||
cd numpy |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we use the wheel instead?
iirc the existing dev build was already changed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried above - but there are no 3.9 wheels available yet on scipy-wheels-night or the old rackcdn url
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmm ok
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will update when available - tracking in #33948
thanks @alimcmaster1 |
* Python 3.9 build * Python 3.9 build * Python 3.9 build * Python 3.9 build * Update build39.sh minor comments * Update .travis.yml Skip clipboard tests * Python 3.9 tests * Dummy commit * Use numpy master manylinux wheel * use scipy nightly wheels * Build numpy from source
* Python 3.9 build * Python 3.9 build * Python 3.9 build * Python 3.9 build * Update build39.sh minor comments * Update .travis.yml Skip clipboard tests * Python 3.9 tests * Dummy commit * Use numpy master manylinux wheel * use scipy nightly wheels * Build numpy from source
* Python 3.9 build * Python 3.9 build * Python 3.9 build * Python 3.9 build * Update build39.sh minor comments * Update .travis.yml Skip clipboard tests * Python 3.9 tests * Dummy commit * Use numpy master manylinux wheel * use scipy nightly wheels * Build numpy from source
closes BUG: Test failures with CPython master #33395
In a very similar fashion to @jbrockmendel work for 3.8 - might be useful to have start a 3.9 build so we can start eliminating the issues @TomAugspurger mentioned in BUG: Test failures with CPython master #33395.
This is using currently using
Python 3.9.0a5+
Currently failing with