-
-
Notifications
You must be signed in to change notification settings - Fork 157
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 testing on Windows has stopped working #2204
Comments
@martinholmer This seems like another fun build error. It looks like the issue comes up when bokeh imports PIL:
I found this issue in the PIL repo: python-pillow/Pillow#2945 It seems like people found multiple solutions to the problem:
Fixing these build errors seems like more art than science most of the time. I hope one of these commands resolves the issue. If they aren't successful, we could remove the windows build while we are still digging into the problem. We have a good number of users at AEI who are using windows. So, it's nice to know that Tax-Calculator is passing the tests on a windows machine. |
@hdoupe said:
@hdoupe, thanks for the diagnosis. It seem like the Anaconda people haven't been able to resolve this problem in over a year! |
I don't understand this line of thinking. If the tests run smoothly on a Mac under Python 3.6 and run smoothly on Linux user both Python 3.6 and Python 3.7, why would there be a problem with Tax-Calculator code running under Windows? @hdoupe also said:
I'm surprised to hear this. There has not been even one download of Windows |
@hdoupe said in Tax-Calculator issue #2204:
Yes, it would be "nice to know" is the costs of running the Windows tests were less than the benefits of knowing for certain. But running Windows tests does not seem to be a PSL requirement. Besides Tax-Calculator and Behavioral-Responses, there are now two other "PSL cataloged" models: OG-USA and B-Tax, neither of which run Windows tests under AppVeyor. Because the costs have recently risen substantially and the benefits are marginal and I don't have the knowledge or time to fix this problem, I think I'll just remove the AppVeyor service from the Tax-Calculator and Behavioral-Responses repositories. @MattHJensen, are you OK with this plan? |
@martinholmer Dropping the windows appveyor build makes since to me given these issues. I think the most common problem for writing python code that is compatible with windows, mac, and linux is keeping the different file path separators straight. Tax-Calculator uses the |
Removing AppVeyor makes sense to me. |
@hdoupe said in Tax-Calculator issue #2204:
@hdoupe, thanks for you thoughts on this. |
A week after opening this issue AppVeyor began working again (on 2019-01-29) without any changes in the the code in this repository. This suggests what many have suspected: that AppVeyor is not only very slow but a bit flaky. |
About 25 months ago, @talumbau introduced in PR #1111 the execution of
pytest
on Windows for each GitHub pull request using the AppVeyor service. The testing has worked ever since. It is far slower than the Travis-CI execution ofpytest
on Linux; so much so that sometimes the development process is slowed down waiting for AppVeyor to run the tests. But it has worked.A few days ago, it stopped working. The Travis-CI tests under both Python 3.6 and 3.7 continue to work fine.
After spending some time trying to diagnose and fix this problem, I've concluded that I don't know nearly enough about how AppVeyor works to fix this problem. For example, I can't even figure out how to make it use Python 3.6 to run the tests. It seems to want to use the latest Python 3.7 version. Look at the AppVeyor test failures for Tax-Calculator pull request #2203 for an example of the (cryptic) messages being generated over the past few days since this problem appeared.
If this problem can't be fixed soon, I vote to drop the whole AppVeyor service and rely on the Travis-CI service.
@MattHJensen @hdoupe
The text was updated successfully, but these errors were encountered: