Skip to content

Optimizations and refactoring#83

Open
slgrobotics wants to merge 11 commits into
flynneva:mainfrom
slgrobotics:pr_to_flynneva
Open

Optimizations and refactoring#83
slgrobotics wants to merge 11 commits into
flynneva:mainfrom
slgrobotics:pr_to_flynneva

Conversation

@slgrobotics

Copy link
Copy Markdown

Here are proposed changes based on my testing

  • Bump to 0.5.1 to prep for release
  • Using smbus2 instead of smbus
  • Optimized get_sensor_data() for speed, only publish necessary topics based on operation mode (fused or raw)
  • Parameter i2c_address: The hexadecimal I2C addresses to use; default=[0x28,0x29] - now will try addresses from a list
  • Added launch files for raw data and imu_tools madgwick filter, and one to avoid config file inclusion.

* Minor changes after AI review

* WIP: launch files

* WIP: launch files

* Update bno055_params_i2c.yaml

* Update bno055_I2C.launch.py

* WIP: launch files

* Update bno055.launch.py

* Comments added to clarify operation mode

* Update bno055.launch.py

* WIP: mag publishes zeroes

* Update SensorService.py

* Update SensorService.py

* Clarified "with FMC" vs NDOF_FMC_OFF in comments

* Update SensorService.py

* Update SensorService.py

* Cleanup and comments

* Added launch/bno055_raw.launch.py

* Update bno055_raw.launch.py

* Bumped version to 0.5.1

* Update CHANGELOG.rst

* Update SensorService.py

* Update SensorService.py

* Refactored bno055.py - moved code from main() to class body

* non-fusing mode: orientation covarience unknown, calib and grav data not published

* Update SensorService.py

* parameter 'i2c_addr' changed to INTEGER_ARRAY, e.g. [0x28, 0x29]

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thankyou for creating your first PR! Contributions like yours is what it is all about. Keep it up!

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.

1 participant