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

Improve information of firmware binary choice #66

Open
krichardsson opened this issue Jun 14, 2017 · 4 comments
Open

Improve information of firmware binary choice #66

krichardsson opened this issue Jun 14, 2017 · 4 comments

Comments

@krichardsson
Copy link
Contributor

The firmware update mechanism gets a list of the binaries from the latest release in our github release repository. A list of binaries is displayed to enable the user to pick the correct one for the update. Currently it is not possible to understand which binary to use and this should be improved, possibly by displaying the file name in the list or something similar.

An example is the 2007.05 release that adds two lines to the list saying
"CF1 & CF2 2017.05"

@fredg02
Copy link
Member

fredg02 commented Jun 27, 2017

Yes, different binaries (e.g. one with the Kalman estimator activated) are currently not covered. We need to find an easy file name scheme that allows to add infos like "kalman".
For example, crazyflie-<info>-YYYY.MM.zip. WDYT?

We could add much more info in the manifest.json, but I'd like to create the release list drop down menu without downloading all release zip files first.

Side note: if the release notes text could be improved, this would automatically be reflected in the app (when you press the info button after selecting a firmware).

@krichardsson
Copy link
Contributor Author

Right now this problem is not acute as we have improved the firmware and do not have the need to release multiple binaries, but I think it would be nice to solve it for the future.

Agree, we don't want to download the zips to populate the drop down. I think your suggestion for a naming scheme looks good. To clarify, I would like to add the possibility of skipping the info since this is the format we have today, crazyflie[-info]-YYYY.MM.zip

Examples:

  • crazyflie-2018.02.zip
  • crazyflie-kalman-2018.02.zip
  • crazyflie-rocket-engine-support-2018.02.zip - can we use '-' in the info part?

@ataffanel
Copy link
Member

Yes we can use - in info part apparently, it works with the regexp:

^[a-zA-Z0-9]+(?:-([a-zA-Z0-9\-]+))?-([0-9.]+)\.zip$

@fredg02
Copy link
Member

fredg02 commented Aug 10, 2017

Thanks for the regexp Arnaud! :)

Unfortunately there will be a limit of characters for the info part (I need to check how many...), otherwise the drop down will overflow or mess up the layout.

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

No branches or pull requests

3 participants