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

FirmwareUpdater - Added new CLI options #683

Merged
merged 12 commits into from
Oct 6, 2020
Merged

FirmwareUpdater - Added new CLI options #683

merged 12 commits into from
Oct 6, 2020

Conversation

davidetome
Copy link
Contributor

@davidetome davidetome commented Oct 6, 2020

DESCRIPTION

This PR implements some new CLI options of the FirmwareUpdater tool; this need comes up from the new automated F/T sensor calibration procedure.
Anyway, these commands can be used also for other purposes.

DETAILS

The new CLI options implemented are :

  • -z or --load-dat-file : Loads the calibration .dat file into STRAIN2 eeprom (pass the file.dat with -l or --file option)
example : FirmwareUpdater -g -e ETH -i eth1 -t 10.0.1.1 -c 1 -n 13 -z -l calibrationDataSN003.dat
  • -w or --set-strain-sn <sn> : Sets the passed serialNumber (i.e. SN001) on STRAIN2
example : FirmwareUpdater -g -e ETH -i eth1 -t 10.0.1.1 -c 1 -n 13 -w SN001
  • -j or --set-strain-gains : Sets on STRAIN2 default gains to (8,24,24,10,10,24) , adjust the offset and check if some channel saturates
example : FirmwareUpdater -g -e ETH -i eth1 -t 10.0.1.1 -c 1 -n 13 -j
  • -b or --get-canboard-version <saveFile> : Gets Bootloader or Application version ( must be y or n to save or not a file containing fw info)
example : FirmwareUpdater -g -e ETH -i eth1 -t 10.0.1.1 -c 1 -n 13 -b y
  • -u or --save-dat-file : Saves the calibration .dat file from STRAIN2 EEPROM
example : FirmwareUpdater -g -e ETH -i eth1 -t 10.0.1.1 -c 1 -n 13 -u

ℹ️ All the example are running on a setup with an EMS4 board with IP=10.0.1.1 and a STRAIN2 board attached to it in the CAN1 line with ID=13

SCREENSHOT

image

cc @triccyx @simeonedussoni @julijenv @violadelbono @emilianob80 @maggia80 @MrAndrea @traversaro

@davidetome davidetome self-assigned this Oct 6, 2020
@pattacini pattacini changed the base branch from master to devel October 6, 2020 07:55
Copy link
Member

@pattacini pattacini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @davidetome for the very nice additions 🚀

@traversaro
Copy link
Member

traversaro commented Oct 6, 2020

This is definitely something that should go in the next distro spotlight! (Edit: actually if the feature is just for strain probably it is not something user facing).

I think it should be nice to also put the nice docs that are part of the PR description somewhere more permanent, but I am not sure if there is any place where we can put them, possibily @marcoaccame has some suggestions on this.

@pattacini pattacini merged commit 7b46d00 into robotology:devel Oct 6, 2020
@triccyx
Copy link
Member

triccyx commented Oct 6, 2020

For me a double rocket for @davidetome !!

@simeonedussoni
Copy link
Contributor

a mac-rocket menu, large 🥇 🍔 🤣

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants