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

Template edits and Matlab main #518

merged 58 commits into from Dec 5, 2019


Copy link

FreyJo commented Nov 27, 2019

This PR contains:

  • fixes & edits to the templates
  • added templates: main_mex and corresponding compilation matlab function
    • this enables the possibility to test the solver obtained from the templated Code in Matlab/Octave
    • navigate into c_generated_code, run make_main_mex_<model_name>, run main_mex_<model_name>

This comment has been minimized.

Copy link

codecov bot commented Nov 27, 2019

Codecov Report

Merging #518 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #518   +/-   ##
  Coverage   75.57%   75.57%           
  Files          45       45           
  Lines       10732    10732           
  Hits         8111     8111           
  Misses       2621     2621
Copy link

zanellia left a comment

It looks good to me ;) (and thanks for the nice polishing!) How about merging before adding more changes?

tmmsartor and others added 8 commits Dec 4, 2019
Copy link
Member Author

FreyJo left a comment

Sorry @zanellia , I just saw your review after adding more Code.
Would you have another look?


This comment has been minimized.

Copy link

zanellia commented Dec 5, 2019

Also the latest changes look good! Merging.

@zanellia zanellia merged commit 5bc2b97 into master Dec 5, 2019
2 checks passed
2 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
continuous-integration/travis-ci/pr The Travis CI build passed
@FreyJo FreyJo deleted the GUC branch Dec 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
3 participants
You can’t perform that action at this time.