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

Check direct current #24

Closed
pattacini opened this issue Mar 23, 2023 · 5 comments
Closed

Check direct current #24

pattacini opened this issue Mar 23, 2023 · 5 comments
Assignees
Labels
domain-modeling Pertains to model-based design team-fix Activity performed by Team FIX

Comments

@pattacini
Copy link
Member

pattacini commented Mar 23, 2023

We're required to verify that the direct current $I_d$ is kept to $0$ effectively by the FOC PID.
To this aim, we may rely on a temporary hack for the CAN protocol.
To be checked on the setup.

(Aside from $I_d$, it might be convenient to check the bare three-phase currents too.)

cc @Nicogene

@pattacini pattacini added team-fix Activity performed by Team FIX domain-modeling Pertains to model-based design labels Mar 23, 2023
@pattacini
Copy link
Member Author

cc @maggia80 @fiorisi

@mfussi66
Copy link
Member

The Id could be quite easily inserted into the STATUS message payload without removing other quantities, there is still plenty of space and it can legitimately be useful info:

image

@pattacini
Copy link
Member Author

For the time being yes, certainly, but in the mid/long term that message is already allocated. See #15.

@pattacini pattacini self-assigned this Mar 26, 2023
@pattacini
Copy link
Member Author

pattacini commented Mar 26, 2023

Provided the reading of the direct current in the test branch https://github.com/robotology/icub-firmware-models/tree/test/direct_current.

The format can be retrieved from below:

image

cc @mfussi66

@pattacini pattacini changed the title Check direct and three-phase currents Check direct current Mar 26, 2023
@mfussi66
Copy link
Member

Today, we successfully measured the direct current Id on CAN as above, and confirmed its truthfulness when printing the 3-phase current on the firmware debug viewer.

Here is an example:
currents_id

Id is correctly controlled to 0 on all three boards. As highlighted above, however, the closeness of the hall sensors of AMBCLDC 0 to the motors influences readings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain-modeling Pertains to model-based design team-fix Activity performed by Team FIX
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

3 participants