8000 Add a configuration for garbage_collection_mbyte_limit by Johannes-Thiel · Pull Request #300 · zauberzeug/rosys · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add a configuration for garbage_collection_mbyte_limit #300

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

Merged
merged 1 commit into from
Jun 4, 2025

Conversation

Johannes-Thiel
Copy link
Contributor
@Johannes-Thiel Johannes-Thiel commented Jun 4, 2025

Motivation

In our project we have a limited amount of Memory. Due to the limit of 300 mb the collection is run every minute. This will stop all processing. To fix this issue we want to start the garbage_collection in times where our system is not doing anything. For this we introduced this feature to set the garbage_collection_mbyte_limit to 0. This is also relevant for systems with smaller memory, to set the limit to the apropriate threshold.

Implementation

To make it confiurable in time, we introduce the value garbage_collection_mbyte_limit in the config.py.

Progress

  • I chose a meaningful title that completes the sentence: "If applied, this PR will..."
  • I chose meaningful labels (if GitHub allows me to so).
  • The implementation is complete.
  • Pytests have been added (or are not necessary).
  • Documentation has been added (or is not necessary).

@Johannes-Thiel Johannes-Thiel added this to the 0.25.0 milestone Jun 4, 2025
@Johannes-Thiel Johannes-Thiel added the enhancement New feature or request label Jun 4, 2025
@Johannes-Thiel Johannes-Thiel merged commit 7b153a3 into main Jun 4, 2025
7 checks passed
@Johannes-Thiel Johannes-Thiel deleted the changeable_garbage_collection_limit branch June 4, 2025 14:18
@falkoschindler falkoschindler mentioned this pull request Jun 4, 2025
2 tasks
pascalzauberzeug added a commit to zauberzeug/field_friend that referenced this pull request Jun 11, 2025
### Motivation
When using the normal rosys garbage collection all functions will be
stopped each minute. To prevent this we are calling the garbage
collection manually, before enabling the implement when starting a new
row and after the automation is finished. If no atomation is running we
are starting the garbage collection every 5 minutes.

### Implementation
We choose to use the garbage collection in the `field_navigation.py` at
this point, because the robot is standing still while the implement will
be enabled. For this reason a second of additional waiting time is okay.
### Progress

- [x] I chose a meaningful title that completes the sentence: "If
applied, this PR will..."
- [x] I chose meaningful labels (if GitHub allows me to so).
- [x] The implementation is complete.
- [x] Tests with a real hardware have been successful (or are not
necessary).
- [x] Pytests have been added (or are not necessary).
- [x] Documentation has been added (or is not necessary).
- [x] Wait for zauberzeug/rosys#300
- [x] Also wait for zauberzeug/rosys#302

edit: the commit message "update to RoSys 0.25.1" is not correct, it is
an update to 0.25.0

---------

Co-authored-by: Pascal Schade <pascal@zauberzeug.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0