Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Feb 15, 2023. It is now read-only.

Build astropy v1.3 #4

Merged
merged 8 commits into from
Jan 13, 2017
Merged

Build astropy v1.3 #4

merged 8 commits into from
Jan 13, 2017

Conversation

tomkooij
Copy link
Collaborator

@tomkooij tomkooij commented Jan 10, 2017

Build astropy v1.3 on appveyor and travis.

I have disabled the appveyor (windows) builds for python 3.6. Although installing python 3.6 from conda is possible (with a work-around) the tests fail (appveyor build)

Even after merging this will not upload the windows wheels to the wheelhouse. The provided password is copied from the matthew-brett appveyor account (from MacPython/cython-wheels and pytables-wheels). The password needs to be reencrypted using the MacPython appveyor account.

Closes #3

Miniconda3.6 is not released yet. We can build against 3.6 but tests fail.
@tomkooij
Copy link
Collaborator Author

tomkooij commented Jan 11, 2017

About the travis errors: It tries to download matplotlib-1.5.3 from pypi. This results in the building of the sdist which is notoriously difficult.
The previous successful builds fetched a wheel from https://nipy.bic.berkeley.edu/manylinux/ but matplotlib-1.5.3 is not available (yet).

I will pin matplotlib to the most recent avaiable at nipy, but I'll have to look into the other failures.

Installing matplotlib from sdist fails, so force installation from wheel.
This prevents pip selecting an sdist from pypi if the sdist version is higher
than the pre-built wheel in the wheelhouse.
@tomkooij
Copy link
Collaborator Author

tomkooij commented Jan 11, 2017

The test_api failures are caused by the remote_data test options. A known issue: astropy/astropy#5651

Unfortunately there a missing matplotlib python 3.6 wheel... (sigh). The MacPython wheelbuilder failed. Reported as: matplotlib/matplotlib#7799

remote_data='none' should be the default, but it is 'any' due to an astropy bug.
Do not use `python setup.py test`
Run tests from installed wheel, as on travis-ci.
The tests try to run cython/MSVC so CMD_IN_ENV is needed to find compiler.
@tomkooij
Copy link
Collaborator Author

tomkooij commented Jan 13, 2017

@matthew-brett: This is (almost) ready to merge.

Although there are many commits the changes are quite small:

  • build astropy-1.3
  • Python 3.6 test currently fail on windows: excluded from appveyor.
  • Use remote_data='none' to prevent API failures in test.
  • Install matplotlib from wheel only.

ToDo:

  • The password for travis-worker is wrong in appveyor.yml. The password needs to be reencrypted against the MacPython appveyor account and inserted. I do not have the password.

@matthew-brett
Copy link
Contributor

Great - thanks.

@matthew-brett matthew-brett merged commit b285396 into MacPython:master Jan 13, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants