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

Appveyor Implementation #1320

Merged
merged 1 commit into from Mar 2, 2015
Merged

Appveyor Implementation #1320

merged 1 commit into from Mar 2, 2015

Conversation

blink1073
Copy link
Member

Builds upon #1057. Corrected dependency installation problems. Link to passing build.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling d64edf7 on blink1073:pr/1057 into 8955bbf on scikit-image:master.

@blink1073
Copy link
Member Author

@stefanv, rebased.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 92.69% when pulling 719276e on blink1073:pr/1057 into b27eb88 on scikit-image:master.

@stefanv
Copy link
Member

stefanv commented Feb 23, 2015

@stefanv
Copy link
Member

stefanv commented Feb 23, 2015

I can enable multi-status to include AppVeyor, but only once everything passes.

@blink1073
Copy link
Member Author

Yay Microsoft! Working on it now...

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 92.67% when pulling 94ccba0 on blink1073:pr/1057 into b27eb88 on scikit-image:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 92.67% when pulling 94ccba0 on blink1073:pr/1057 into b27eb88 on scikit-image:master.

@blink1073
Copy link
Member Author

Argh, os.pathsep is not : on Windows...

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.0%) to 92.68% when pulling fd18f25 on blink1073:pr/1057 into b27eb88 on scikit-image:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.0%) to 92.68% when pulling fd18f25 on blink1073:pr/1057 into b27eb88 on scikit-image:master.

@stefanv
Copy link
Member

stefanv commented Feb 24, 2015

Yikes!!

skimage.viewer.tests.test_widgets.test_ok_buttons ... alloc: invalid block: 05C98CF8: c0 5

@blink1073
Copy link
Member Author

I'm travelling today so I may not get to this until tomorrow.

@blink1073
Copy link
Member Author

That is a doozy. I don't even know where to start...

@blink1073
Copy link
Member Author

I might need to boot into Windows.

@blink1073
Copy link
Member Author

Great, it works fine on my machine with and without PyQt installed....

@blink1073
Copy link
Member Author

I used Windows 7 Anaconda Python 3.4 64 bit.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.0%) to 92.68% when pulling 2bc72a7 on blink1073:pr/1057 into b27eb88 on scikit-image:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.0%) to 92.68% when pulling ba1b775 on blink1073:pr/1057 into b27eb88 on scikit-image:master.

Implement Appveyor builds

initial test

Updated appveyor.yml

New nosetest call

With conda update conda

with pillow

without pillow

TST: Change case sensitivive JPG extension

Revert "TST: Change case sensitivive JPG extension"

This reverts commit 2deed7c.

Trying with Python 2.7

Trying with version env

Trying again with variables

Testing on all Python versions

don't allow failures

Allow failure

Do not actually use tests

Ignore failing tests

Removed Python 2.6 test

Testing only Python 2.6

Testing out more sklearn like AppVeyor CI

Added install to pip

Add artifacts

Enabled all permutations

Disable 2.6, add PIL

Python 2.6, 2.7 only with PIL

Testing with variable dependencies

Allow failure with IF ELSE

With Python 3.4

Scikit-learn like appveyor CI

Fixing paths

Undo path fix

path fix single line

path fix single line 2

Using Miniconda

More path fixes

New wheelhouse link

Added pillow to requirements.txt

Added networkx to requirements.txt

Add testing of 32/64-bit Python 2.7 and 3.4 to matrix

Debugging Cython compile

Retry with all 4 builds

Updated install.ps1 file

Updated based on latest python-appveyor-demo

Debugging pip install

Specify numpy 1.8.1 until whl is uploaded to rackspace

Use skimage-wide requirements.txt file

Minor comment change to trigger build

Install wheel and then install from WHEELHOUSE

Install six from pip

Install networkx from pip

Install pyparsing from pip

Install pytz from pip

Try using just find-links

Install the binary dependencies first, then the rest

Add pillow to the install list

Fix appveyor.yml syntax

Fix requirements.txt syntax

Fix requirements.txt syntax again

Fix appveyor call to initial install

Fix appveyor call to initial install again

Fix appveyor call to initial install yet again

Install wheel

Install wheel first

Install wheel and nose in the appveyor requirements.txt

Fix Python3 version to match python ftp site

Only use cleanup decorator if available

Add debug info to multiimage test

More debugging information

Fix handling of path separators on Windows

Add another warning guard

Fix warning handling for non-windows

Do not use TkAgg as it may be causing alloc error

Clean up echo command

Allow for unclosed file warning

Fix spacing in echo command
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 92.67% when pulling 78a0eb6 on blink1073:pr/1057 into 4156e20 on scikit-image:master.

@blink1073
Copy link
Member Author

Ready to roll. Not sure why coveralls is being mean.

stefanv added a commit that referenced this pull request Mar 2, 2015
@stefanv stefanv merged commit 8845169 into scikit-image:master Mar 2, 2015
@stefanv
Copy link
Member

stefanv commented Mar 2, 2015

Most excellent; thank you @blink1073!

# the binaries from http://www.lfd.uci.edu/~gohlke/pythonlibs/
# This is a temporary solution. As soon as numpy, scipy and cython provide
# official wheels for windows we can delete this --find-links line.
--find-links http://28daf2247a33ed269873-7b1aad3fab3cc330e1fd9d109892382a.r6.cf2.rackcdn.com/appveyor_pip_index.html
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@matthew-brett Just wanted to confirm with you whether this is the most current situation?

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.

None yet

4 participants