-
Notifications
You must be signed in to change notification settings - Fork 70
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
docs needed for nn_device #145
Comments
I took a lil stab at this today, but found I'm still lacking the complete mental models to describe it aptly. Perhaps in a few months I'll be able to revisit this, but I think it'd be preferred if someone else could document this. |
@yoshuawuyts see #147, you know we ought to get this documented |
yeah, definitely. But I haven't gone deep enough on this to do it quite yet. I'm going to be give a talk about |
oh nice! We need more talks on Also can you update this thread with any links about your talk? Or maybe we put a link to resources in the docs (oh but now that's a different PR) Anyway yea a month is fine, if you see any issues with nn_device implementation also let us know so we can fix, I haven't tried using it personally in a long time |
hah, alright! I'll try and not forget to contribute back, don't hesitate to ping me if I haven't gravitated back here in a month. Cheers! |
nice talks notes, great diagrams.. like: ┌───────────┐
│ Publisher │
├─────┬─────┤
│ REP │ PUB │
└▲──┬─┴──┬──┘
1 2 3
┌┴──▼─┬──▼──┐
│ REQ │ SUB │
├─────┴─────┤
│Subscriber │
└───────────┘ can we add stuff like that on the node-nanomsg readme? also my 3 cents: yoshuawuyts/speaking#2 |
Oh yeah, that'd be cool! I've got a whole collection of 'em on https://github.com/yoshuawuyts/knowledge/blob/master/arch/patterns.md |
The nanomsg device api is useful should be documented
Related to #143, we have a device implementation and some tests for it, but none of it is documented.
The text was updated successfully, but these errors were encountered: