Skip to content
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

yc/add matrix demo - Adding a server example for MATRIX Creator/Voice boards #69

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

yoelrc88
Copy link
Contributor

@yoelrc88 yoelrc88 commented Jun 5, 2018

  • added configuration files ready for this example
  • added small guide on how to install/run the demo
  • added a gif as a reference of what the demo does

@FrancoisGrondin
Copy link
Member

Thank you for this code! I see you are using json-c to parse from JSON. I try to avoid using external libraries as much as possible and so yesterday I just wrote a json parser to integrate directly in odas. As soon as I have push this in a dev branch, I'll look how I can merge your code to odas.

@yoelrc88
Copy link
Contributor Author

yoelrc88 commented Jun 7, 2018

Oh great! As soon as you push that json code I will update this example.
Thank you

@FrancoisGrondin
Copy link
Member

Here is the code for the homemade JSON parser. I'm still working on the server side implementation... I'll merge to master when I'm done and I clean a bit, but you can already have an idea of the way the json parser works here: https://github.com/introlab/odas/blob/dev/src/odas/utils/json.c

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.

2 participants