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

[Estimator] Move estimators for BLDC rotor into their own class #6

Merged
merged 1 commit into from
Oct 9, 2023

Conversation

sahil-kale
Copy link
Owner

Topic: estimator-move
Relative:
Reviewers:

Topic: estimator-move
Relative:
Reviewers:
@sahil-kale
Copy link
Owner Author

Reviews in this chain:
#6 [Estimator] Move estimators for BLDC rotor into their own class

@sahil-kale
Copy link
Owner Author

sahil-kale commented Oct 9, 2023

# head base diff date summary
0 01686a3e c09feb75 diff Oct 9 15:43 PM 6 files changed, 30 insertions(+), 245 deletions(-)
1 4d5846cb c09feb75 diff Oct 9 15:49 PM 8 files changed, 280 insertions(+), 30 deletions(-)

@sahil-kale sahil-kale force-pushed the sahil.kale1/revup/main/estimator-move branch from 01686a3 to 4d5846c Compare October 9, 2023 13:49
@sahil-kale sahil-kale merged commit 4c51d84 into main Oct 9, 2023
2 checks passed
@sahil-kale sahil-kale deleted the sahil.kale1/revup/main/estimator-move branch October 9, 2023 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant