Skip to content

Conversation

@silverjam
Copy link
Contributor

@silverjam silverjam commented Jun 7, 2019

Per #681 we don't plan to support this any time soon, so we're going to deprecate the message to avoid confusion.

Note: there's some additional change to sbp.jit folded into this since it wasn't (until now) part of the all target for updating generated code.

@silverjam silverjam requested a review from anth-cole June 7, 2019 22:30
Going to assume that this was written when we didn't have a seperate
TCPDriver (thus we were forced to use PySerialDriver's TCP support) and
switch this test to TCPDriver.

This is in the hopes that it'll improvce the stability of this test.
Copy link
Contributor

@anth-cole anth-cole left a comment

Choose a reason for hiding this comment

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

lgtm

@silverjam
Copy link
Contributor Author

Travis failure looks like a (hopefully) transient issue with the sbp2json benchmark

@silverjam silverjam merged commit 02ae20e into master Jun 10, 2019
@silverjam silverjam deleted the silverjam/deprecated-msg-init-base branch June 10, 2019 23:17
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