Skip to content

Commit

Permalink
chore(config): added a warning for systems with imus #1000
Browse files Browse the repository at this point in the history
added a warning for when a user tries configuring a system with an IMU,
explaining it is still under development and not all calibration scripts
are functional
  • Loading branch information
Kazadhum committed Jan 21, 2025
1 parent 448bc70 commit c1d505c
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions atom_calibration/scripts/configure_calibration_pkg
Original file line number Diff line number Diff line change
Expand Up @@ -591,6 +591,16 @@ if __name__ == "__main__":
atomError('Config file has duplicate transform(s) ' + Fore.BLUE +
str(duplicate_transforms) + Style.RESET_ALL + '. Invalid configuration.')

# NOTE: Development of IMU calibration will progress throughout the calibration pipeline, meaning some of the scripts created during configuration might not work if there is an IMU sensor and depending on the current stage of development. Adding a warning...

imu_present = False
for sensor_key, sensor in config['sensors'].items():
if sensor['modality'] == "imu":
imu_present = True

if imu_present == True:
atomWarn('There is at least one sensor with modality "imu". At the moment, the calibration of systems involving IMU sensors is in development. As such, some of the scripts generated by this configuration might not be functional.')

# --------------------------------------------------------------------------
# Create dot calibration graph (used for printing out a summary)
# --------------------------------------------------------------------------
Expand Down

0 comments on commit c1d505c

Please sign in to comment.