-
Notifications
You must be signed in to change notification settings - Fork 553
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
add MT6701 I2C sensor configuration #398
Conversation
Thanks a lot for this contribution! I will gladly merge it, but we accept the PRs only against the dev branch of the library. I've changed the base of your PR, but there are some other parallel changes. I'll look into this, and merge it once I've sorted this out. |
thanks! let me know if you would like me to rebase and resolve if there are any conflicts. |
Hey, if you do not mind sending it again based on the "dev" branch then I could straight merge it. It's not your fault, we caused this by accidentally merging some changes to the master branch. Can I ask you to rebase your changes on the dev branch, then I can merge that. |
Sure thing, I have also extended example file for demonstrating MT6701 integration. But, somehow I cannot see dev branch in my fork, and cannot push a proposal branch to direct clone(not forked). is there any permission that needs to be granted ? |
Huh, that's weird... I'm not sure. Maybe there is a way to pull over the other branch into your forked repo on GitHub but not sure how... One way you could definately do it is to add the SimpleFOC repo as another remote to your local cloned copy, and then pull the dev branch from this. Then you can push that branch to your fork, and do the PR from there... |
I have cherry-picked your changes over in this PR: #406 |
No description provided.