This repository was archived by the owner on Oct 12, 2022. It is now read-only.
This repository was archived by the owner on Oct 12, 2022. It is now read-only.
Some initial improvements #2
Description
Hello,
First thanks for this really nice library (and to make the source code available).
From initial usage I'd like to point out some potential first improvements (please note I'll be very happy to help with some of those):
- Add gitignore : Don't think it requires any explanation, I'll hapilly push a standard templete for this (ignore bin/obj and visual studio temporary files like .suo)
- Update to public sdk version, otherwise you will have coordinate mapper errors (this is also really simple, only question is should dll be included as dependency as it is now, or should project just point to kinect sdk environment variable).
- For now syntax error in script will throw some null assertion exception while visiting syntax tree, which makes it quite difficult to know. Some compilation exception with line number would be really handy for authoring.
- Consolidate gesture sample : Simple error message in case of invalid syntax (for now application just crashes).
Kind regards
Julien