Skip to content

Conversation

carolFrohlich
Copy link
Contributor

Please don't accept the pull request, this is just a test.

Copy link
Member

@chrisgorgo chrisgorgo left a comment

Choose a reason for hiding this comment

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

This is just a test.

@chrisgorgo
Copy link
Member

Just to make it clear - the issue reported in #1692 is not confined just to tests and is not a problem with tests themselves. It has also been reported by users in their production pipelines (I also have experienced it). The relevant multiproc tests should not be reenabled until a solution the the multiproc bug will be found.

@ccraddock
Copy link
Collaborator

Do you have any examples that might help us reproduce the problem? We do not see these issues with our pipeline and were only able to reproduce it with nosetests.

@codecov-io
Copy link

codecov-io commented Dec 7, 2016

Current coverage is 70.82% (diff: 75.00%)

Merging #1739 into master will decrease coverage by <.01%

@@ master #1739 diff @@ ========================================== Files 1030 1030 Lines 51873 51876 +3 Methods 0 0 Messages 0 0 Branches 7341 7341 ========================================== + Hits 36741 36743 +2  - Misses 14031 14039 +8  + Partials 1101 1094 -7 

Powered by Codecov. Last update 2dcda51...1e213f8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants