Skip to content

Conversation

blink1073
Copy link
Collaborator

87 -> 92%, and caught some bugs!


def _program_name(self):
def _program_name(self): # pramga: no cover
Copy link
Collaborator

Choose a reason for hiding this comment

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

pragma, not pramga

@@ -22,8 +22,6 @@
import pymatbridge as pymat
from .compat import text_type

ipython_version = int(IPython.__version__[0])
Copy link
Owner

Choose a reason for hiding this comment

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

Probably safer to use IPython.version_info

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Not needed any more, I used a version-agnostic method.

Copy link
Owner

Choose a reason for hiding this comment

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

Oh yes. This is red. Silly me.

@isbadawi
Copy link
Collaborator

There seems to be an issue with the Travis build. If the tests fail, it waits for 10 minutes and times out.

@blink1073
Copy link
Collaborator Author

@isbadawi, yeah, I'm not sure what to do about that. The Python 3.4 version had the same behavior even when passing.

@blink1073
Copy link
Collaborator Author

We could add a kill switch alarm to make it stop sooner...

@blink1073
Copy link
Collaborator Author

Oh great, now they are all hanging...

@arokem
Copy link
Owner

arokem commented Feb 16, 2015

It looks like the test failure leaves it hanging there.

On Mon, Feb 16, 2015 at 9:57 AM, Steven Silvester notifications@github.com
wrote:

Oh great, now they are all hanging...


Reply to this email directly or view it on GitHub
#142 (comment)
.

@blink1073
Copy link
Collaborator Author

Hold on before merging, I'm adding two more tests.

@blink1073
Copy link
Collaborator Author

The hanging behaviour was a result of me removing __del__ in the magic without putting the explicit stop() call back in unload_ipython_extension. Fixed now, and ready to merge!

isbadawi added a commit that referenced this pull request Feb 16, 2015
More streamlining, fault tolerance, and coverage increasing
@isbadawi isbadawi merged commit 3b7024a into arokem:master Feb 16, 2015
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.

3 participants