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

Need for easy mempool configuration #13

Closed
Karsten1987 opened this issue Dec 3, 2019 · 5 comments
Closed

Need for easy mempool configuration #13

Karsten1987 opened this issue Dec 3, 2019 · 5 comments
Labels
enhancement New feature

Comments

@Karsten1987
Copy link
Contributor

Brief feature description

Motivated from ros2/rmw_iceoryx#5

Detailed information

The current mempool configuration is hardcoded in https://github.com/eclipse/iceoryx/blob/master/iceoryx_posh/source/mepoo/mepoo_config.cpp#L44-L54 and should be changed to be set dynamically on startup.

@mossmaurice mossmaurice added the enhancement New feature label Dec 4, 2019
@mossmaurice
Copy link
Contributor

Hi @Karsten1987,

thanks for your feature request. Having a hard-coded compile-time mempool config is quite unpleasant. The feature "run-time config file for RouDi" is definitely on our roadmap (see 2nd point in https://github.com/eclipse/iceoryx/blob/master/PLANNED_FEATURES.md). You can expect it to arrive in Q1 2020.

Simon

@budrus
Copy link
Contributor

budrus commented Dec 10, 2019

work on that just started

@elBoberido
Copy link
Member

it's almost ready. expect a pull request within the next days

@Karsten1987
Copy link
Contributor Author

Is there any update on this? #12 also seems to be in need for this.

elBoberido added a commit to elBoberido/iceoryx that referenced this issue Jan 29, 2020
Signed-off-by: Kraus Mathias (CC-AD/ESW1) <mathias.kraus2@de.bosch.com>
elBoberido added a commit to elBoberido/iceoryx that referenced this issue Jan 30, 2020
Signed-off-by: Kraus Mathias (CC-AD/ESW1) <mathias.kraus2@de.bosch.com>
elBoberido added a commit that referenced this issue Jan 30, 2020
…orings

iox-#13 add config file parser and some refactorings
@budrus
Copy link
Contributor

budrus commented Jan 30, 2020

#38

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature
Projects
None yet
Development

No branches or pull requests

4 participants