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

minipack3n: Initial config for sensor_service #320

Closed

Conversation

brandonchuang
Copy link
Contributor

@brandonchuang brandonchuang commented Jan 20, 2025

Summary:
Add sensor_service.json for the minipack3n sensor_service.

** Description **

  • Include all sensor devices in sensor_service.json for minipack3n.
  • This configuration has been tested on minipack3n.

Test Plan

  • Run the sensor_service with this configuration on minipack3n.
  • Validate the generated logs to ensure functionality.

20250214_mp3n_sensor_service_log.txt

@kimdo8736
Copy link

Can we remove all the sensors with "sysfsPath" /run/devmap/sensors/SMB_VRM[1..11]/* for now?

We can add them back later when they are successfully created by Platform Manager.

Summary:
Add `sensor_service.json` for the minipack3n sensor_service.

** Description **
- Include all sensor devices in `sensor_service.json` for minipack3n.
- This configuration has been tested on minipack3n.

Test Plan
- Run the sensor_service with this configuration on minipack3n.
- Validate the generated logs to ensure functionality.

[20250214_mp3n_sensor_service_log.txt](https://github.com/user-attachments/files/18795107/20250214_mp3n_sensor_service_log.txt)
@facebook-github-bot
Copy link
Contributor

@kimdo8736 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@kimdo8736 merged this pull request in 2573232.

@brandonchuang brandonchuang deleted the mp3n_sensor_service branch February 20, 2025 06:29
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.

3 participants