- 
                Notifications
    You must be signed in to change notification settings 
- Fork 4
Examples updated to match MTK v9 macros syntax #16
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
Conversation
Updated examples to match MTK v9 macros syntax
| @bradcarman Just following up here? | 
| Yeah, I think this looks good. Can you just remove Revise from the Project.toml? I don't think it needs to be there. | 
Removed Revise from Project.toml
| @bradcarman Done | 
| @bradcarman Just following up here? | 
| Hi @matthew-kapp this looks good, I'll approve. Thanks for your contribution! | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All looks good
| @bradcarman sorry, I'm still learning git approvals - is there still a final step to merge merge request? | 
| Sorry, I was just waiting for tests to pass, all merged now. | 
The examples have been updated in such a way that the exact same model is created, but the syntax matches that used by MTK v9, where @mtkmodel and other macros are used.