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

RFC29_Optimize Nano Setup by Removing Unused Services #41

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Ahelsamahy
Copy link
Collaborator

The current setup for the Nano includes 11 services: following, ftpd, httpd, inference, mongodb, pilot, rosnode, teleop, vehicle, wireguard, and zerotier. When all these services are active, the CPU and memory are nearly fully utilized even when idle. This results in performance bottlenecks, especially when resource-intensive services like FOL (Following) or INF (Inference) are running.

To address this, we are removing services that are not currently needed, such as Wireguard and ROS. This change aims to optimize resource usage and improve the overall performance of the Nano during operations. While a hardware upgrade may be necessary in the future, this update focuses on improving efficiency with the existing setup.

Changes

  • Removed the Wireguard and ROS services from the Nano's setup.
  • Adjusted configuration files to reflect the removal of these services.
  • Updated documentation to detail the changes in the service setup and their impact on performance.

Impact

  • Reduced idle CPU and memory usage by eliminating unnecessary services.
  • Improved performance for critical services like FOL and INF, minimizing bottlenecks.
  • Delayed the need for an immediate hardware upgrade by optimizing current resources.

OrestisTS and others added 2 commits May 22, 2024 13:29
@Ahelsamahy Ahelsamahy marked this pull request as draft October 18, 2024 12:14
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