-
Notifications
You must be signed in to change notification settings - Fork 35
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
License? #8
Comments
Hey v4hn, thank you for asking! The MATLAB source is licensed with the BSD-3-Clause license. But since I generated the CPP version with the Student Version of MATLAB Coder, MATLAB Coder inserts these additional headers into the CPP sources. To get rid of these headers, I suggest to use a non-student version of MATLAB Coder to regenerate the CPP sources. See this issue for a tutorial. Best |
Thank you for explaining! I'm not familiar with MATLAB Coder and wrongly assumed the C++ code was an independent component.
It would definitely improve the usefulness of this repository in ROS if someone were to contribute generated code with a full license.
Maybe you should leave an issue open until someone did that because the BSD-3-Clause indicator for this repository is technically wrong at the moment. :/
Though personally I'm happy to use "academia-only" code as well. :-)
|
Thanks for the suggestion. I was able to update from the student version to the academic version which in addition to academic research and meeting course requirements at degree granting institutions also allows use in teaching. TLDR: If you have a commercial version of MATLAB, please follow this procedure to generate CPP files from the MATLAB sources and provide them to the public! |
Hey Marius,
I've basically just found this repository and did not use it (yet?). But I'll have to ask out of pure curiosity:
The repository includes a
LICENCE.md
for BSD-3-Clause, but the headers refer to a "Student License", although some of them also include a BSD header somewhere in the file.What license do you want to publish this code under?
The text was updated successfully, but these errors were encountered: