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.

Sign up for GitHub

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

Jump to bottom

AMCBLDC – Read Vcc on the fly #10

Closed
pattacini opened this issue Feb 22, 2023 · 4 comments · Fixed by #73
Closed

AMCBLDC – Read Vcc on the fly #10

pattacini opened this issue Feb 22, 2023 · 4 comments · Fixed by #73
Assignees
@pattacini @sgiraz
Labels
domain-firmware Pertains to FW development domain-modeling Pertains to model-based design team-fix Activity performed by Team FIX type-board-amcbldc Relevant to the AMCBLDC development

Comments

@pattacini
Copy link
Member

It's be super useful to have the AMCBLDC board capable of reading the supply voltage Vcc on the fly in order to adapt wrt. changing conditions (network supply, batter, setup...).

To this end, we're required to develop a dedicated API and expose the value to the architectural model.
See also #8.
@pattacini pattacini added team-fix Activity performed by Team FIX domain-firmware Pertains to FW development domain-modeling Pertains to model-based design labels Feb 22, 2023
@pattacini
Copy link
Member Author

cc @marcoaccame @sgiraz @fgarini @valegagge

@pattacini pattacini changed the title Let AMCBLDC read Vcc on the fly AMCBLDC – Read Vcc on the fly Feb 22, 2023
@pattacini pattacini added the type-board-amcbldc Relevant to the AMCBLDC development label Feb 22, 2023
@pattacini
Copy link
Member Author

@marcoaccame provided a way to read Vcc on the fly aboard AMCBLDC.
When we're also done with AMC, I'll update the model to have this reading as input.

cc @sgiraz @Nicogene @mfussi66 @ale-git @valegagge

pattacini added a commit that referenced this issue Dec 28, 2023
@pattacini
Made Vcc an input
660c40f 
See #10
@pattacini pattacini self-assigned this Dec 28, 2023
@pattacini pattacini mentioned this issue Dec 28, 2023
Merged
@pattacini pattacini linked a pull request Dec 28, 2023 that will close this issue
Make Vcc an input as a new field of SensorsData; Disable Temperature Estimation #73
Merged
pattacini added a commit that referenced this issue Dec 30, 2023
@pattacini
Made Vcc an input
70c2a1a 
See #10
pattacini added a commit that referenced this issue Dec 30, 2023
@pattacini
Made Vcc an input
e59b8b8 
See #10

Ported ed9d3de
pattacini added a commit that referenced this issue Dec 30, 2023
@pattacini
Made Vcc an input
ea41849 
See #10

Ported ed9d3de
@pattacini pattacini removed a link to a pull request Jan 15, 2024
Make Vcc an input as a new field of SensorsData; Disable Temperature Estimation #73
Merged
@pattacini
Copy link
Member Author

Test report by @sgiraz

Varying the Vcc from 24V to 44V and viceversa here are the results:

image (1)

MATLAB_DYwhivPItX.mp4

As we can see now, we got the expected the results. If we lower the Supply Voltage we need a increased PWM% (Vq) to keep the desired current target

@pattacini
Copy link
Member Author

cc @DanielePucci

@pattacini pattacini linked a pull request Mar 13, 2024 that will close this issue
Make Vcc an input as a new field of SensorsData; Disable Temperature Estimation #73
Merged
@mfussi66 mfussi66 mentioned this issue Mar 13, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pattacini pattacini

@sgiraz sgiraz

Labels
domain-firmware Pertains to FW development domain-modeling Pertains to model-based design team-fix Activity performed by Team FIX type-board-amcbldc Relevant to the AMCBLDC development
Projects
None yet
Milestone
No milestone
2 participants
@pattacini @sgiraz