Skip to content

Conversation

blink1073
Copy link
Collaborator

Make the messenger non-blocking, and allow Matlab GUI to refresh, making visible plots interactive.
Added the ability to toggle visibility through the set_plot_settings method or using the --gui flag in the magic. The Matlab splash screen will now pop up at the start, but the figures are invisible by default.

Also fixes a bug where "clear all" killed the Server.

Finally, adds numpy as a dependency.

Tested with both Matlab and Octave in Linux.

Fixes #101. Fixes #70. Fixes #148. Fixes #96. Fixes #99. Fixes #132.

@blink1073
Copy link
Collaborator Author

Ugh I hate you Coveralls!

@blink1073
Copy link
Collaborator Author

Do not merge yet! I have to change messenger.c again.

@blink1073
Copy link
Collaborator Author

Good to go. Octave plots were coming back from the dead so I had to add an explicit close all hidden at the start of each eval.

@blink1073 blink1073 closed this Feb 19, 2015
@blink1073 blink1073 reopened this Feb 19, 2015
@blink1073
Copy link
Collaborator Author

Sorry, please wait again.

@blink1073
Copy link
Collaborator Author

Nevermind, I tried to avoid closing the figures by using pause instead of drawnow in messenger.c but the Octave figures still pop up. This is ready to merge.

@arokem
Copy link
Owner

arokem commented Feb 19, 2015

Just to be sure - these newly compiled messengers are now compiled against zmq 4?

@blink1073
Copy link
Collaborator Author

I compiled against libzmq3, since that is what is available via apt-get in Ubuntu 14.04. I'd contrast this to the OSX situation where you have v4 available on Homebrew, and Windows, where you can get the installer on the ZMQ website. (I built against v4 on Windows).

@isbadawi
Copy link
Collaborator

I went in the coveralls settings and changed "Coverage decrease threshold for failure" to "10%", so (if I understand the setting correctly) it should stop failing PRs for reducing coverage by a tiny amount like this. (You should both have access if you'd like to mess with the settings more: https://coveralls.io/r/arokem/python-matlab-bridge).

Anyway, this LGTM. I'll commit new versions of the OS X messengers for Matlab and Octave (built against v4) after merging.

isbadawi added a commit that referenced this pull request Feb 21, 2015
@isbadawi isbadawi merged commit 79c06d4 into arokem:master Feb 21, 2015
@arokem
Copy link
Owner

arokem commented Feb 21, 2015

Cool cool. I am working on setting up the messengers right now, so you can
hold off on that.

On Fri, Feb 20, 2015 at 5:15 PM, Ismail Badawi notifications@github.com
wrote:

Merged #150 #150.


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

@isbadawi
Copy link
Collaborator

Oh sorry! I just pushed them. Feel free to override if you have something you want to add.

@arokem
Copy link
Owner

arokem commented Feb 21, 2015

No problem. The ones you pushed are compiled against zmq 4? I'll add (as
backup) the zmq 3 ones.

On Fri, Feb 20, 2015 at 5:17 PM, Ismail Badawi notifications@github.com
wrote:

Oh sorry! I just pushed them. Feel free to override if you have something
you want to add.


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

@isbadawi
Copy link
Collaborator

Yep, I built them against 4.0.5.

@arokem
Copy link
Owner

arokem commented Feb 21, 2015

OK. Working on those zmq3 messngers for the mac. Will have a PR with that
soon.

On Fri, Feb 20, 2015 at 5:21 PM, Ismail Badawi notifications@github.com
wrote:

Yep, I built them against 4.0.5.


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

@blink1073
Copy link
Collaborator Author

I should build some zmq 4 ones for linux as well. 


Sent from Mailbox

On Fri, Feb 20, 2015 at 8:24 PM, Ariel Rokem notifications@github.com
wrote:

OK. Working on those zmq3 messngers for the mac. Will have a PR with that
soon.
On Fri, Feb 20, 2015 at 5:21 PM, Ismail Badawi notifications@github.com
wrote:

Yep, I built them against 4.0.5.


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


Reply to this email directly or view it on GitHub:
#150 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants