#ARRUS-543: Added GPU prototxt setting to enable customisation of MemoryPool Usage and RAM Limit for ProcessingRunner#428
Open
#ARRUS-543: Added GPU prototxt setting to enable customisation of MemoryPool Usage and RAM Limit for ProcessingRunner#428
Conversation
change logging level in log_gpu_info to DEBUG
zkLog
reviewed
Feb 4, 2026
| """ | ||
| Set the GPU memory limit for the memory pool as a percentage of total GPU memory. | ||
|
|
||
| :param gpu_memory_limit_percentage: Percentage of total GPU memory to use (0.0 to 1.0) |
Collaborator
There was a problem hiding this comment.
There is following line in the docs/content/user_guide.python.rst:
'
The memory_limit_percentage is the percentage of the GPU memory that should be used by the memory pool. Values between 0.01 and 0.95 are allowed.
'
Here is " (0.0 to 1.0)".
Is it inconsistency or these are other things?
| # Here starts communication with the device. | ||
| medium = arrus.medium.Medium(name="water", speed_of_sound=1490) | ||
| with arrus.Session("us4r.prototxt", medium=medium) as sess: | ||
| with arrus.Session("/home/pjarosik/us4r.prototxt", medium=medium) as sess: |
Collaborator
There was a problem hiding this comment.
Probably the recent path was better for the example.
| ) | ||
|
|
||
| arrus.set_clog_level(arrus.logging.INFO) | ||
| arrus.set_clog_level(arrus.logging.DEBUG) |
Collaborator
There was a problem hiding this comment.
Do we need debug level here in the example script?
| sample_range=(0, 4096), | ||
| delays=[0]*n_elements), | ||
| Rx(aperture=np.ones(n_elements).astype(bool), | ||
| sample_range=[420, 420+512], |
Collaborator
There was a problem hiding this comment.
This is example script - it would be 'more naturally' (probably) if sample range would be rather larger, like [0, 1024], to be ready to see 'the wire in the water' or similar simple object.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is a follow-up PR based on the submission prepared by @gordon-n-stevenson; the original PR can be found here.
Many thanks to @gordon-n-stevenson! Original description:
I have introduced the following changes compared to the base PR:
GpuSettingsfromUs4RSettingstoSessionSettings, as the GPU can be treated as a device independent of the connected Us4R. This results in the following changes:.prototxtfile: the gpu settings should be defined at the top-level of the entire file (previously they were part ofus4rsettings) see e.g., these test .prototxtSessionBuildernow has an additional methodaddGpu.GpuSettingsis now immutable (setters have been removed).INFO(in accordance with other users’ requirements).