Skip to content

Conversation

@JWyman482
Copy link
Contributor

Reference Issue

Problem(s)

  1. The round(baro, 1) call on line 277 in bds62.py was being called when baro was 'None'.
  2. (minor) Padding in _print() wasn't sufficient for the baro readout
  3. (minor) Output would display units when value was 'None.'

Fix Actions Taken

  1. Wrapped the else clause with the round call so it was only called if baro held a numeric value.
  2. Updated the output padding from 20 chars to 28 chars
  3. Added a conditional to remove the "millibars" if baro is 'None.'

@xoolive xoolive merged commit d3607a1 into junzis:master Aug 15, 2022
@JWyman482 JWyman482 deleted the bds62_baro_rounding_fix branch August 25, 2022 04:37
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.

2 participants