[config] Implementation of new configuration management #1505

Peguen · 2024-04-04T13:40:37Z

Description

Introduced config management via structs which opens the possibilty for users to configure eCAL in code. In addition, evaluation of the inputs will be easily possible.

If assigned configuration values can be evaluated and are incorrect, the program will exit with an error.

Internally also the new config structs will be used.

…tializer.cpp

…lue.

…a types.

…unsigned int.

…tructor. Some renaming.

…ubscriber config.

…mes.

github-actions

clang-tidy made some suggestions

ecal/core/src/config/ecal_config_initializer.cpp

ecal/core/src/ecal_global_accessors.cpp

ecal/core/src/ecal_global_accessors.h

github-actions · 2024-05-22T14:48:08Z

ecal/core/src/pubsub/ecal_publisher_config.cpp

-    Configuration::Configuration() :
-      share_topic_type(eCAL::Config::IsTopicTypeSharingEnabled()),
-      share_topic_description(eCAL::Config::IsTopicDescriptionSharingEnabled())
+    Configuration::Configuration() 


warning: constructor does not initialize these fields: tcp, share_topic_type, share_topic_description [cppcoreguidelines-pro-type-member-init]

ecal/core/include/ecal/config/ecal_publisher_config.h:137:

- TCP::Configuration tcp; + TCP::Configuration tcp{};

ecal/core/include/ecal/config/ecal_publisher_config.h:139:

- bool share_topic_type; //!< share topic type via registration - bool share_topic_description; //!< share topic description via registration + bool share_topic_type{}; //!< share topic type via registration + bool share_topic_description{}; //!< share topic description via registration

…_implementation

github-actions

clang-tidy made some suggestions

doc/rst/configuration/src/publisher_config/main.cpp

github-actions · 2024-05-24T09:29:27Z

ecal/core/src/pubsub/ecal_publisher_config.cpp

-    Configuration::Configuration() :
-      share_topic_type(eCAL::Config::IsTopicTypeSharingEnabled()),
-      share_topic_description(eCAL::Config::IsTopicDescriptionSharingEnabled())
+    Configuration::Configuration() 


warning: constructor does not initialize these fields: tcp, share_topic_type, share_topic_description [cppcoreguidelines-pro-type-member-init]

ecal/core/include/ecal/config/publisher.h:137:

- TCP::Configuration tcp; + TCP::Configuration tcp{};

ecal/core/include/ecal/config/publisher.h:139:

- bool share_topic_type; //!< share topic type via registration - bool share_topic_description; //!< share topic description via registration + bool share_topic_type{}; //!< share topic type via registration + bool share_topic_description{}; //!< share topic description via registration

…_implementation

ecal/core/src/config/ecal_config_initializer.cpp

github-actions

clang-tidy made some suggestions

github-actions · 2024-06-04T11:46:14Z

ecal/core/include/ecal/config/configuration.h

+          std::string                    ecal_ini_file_path{};
+
+        private:
+          ECAL_API void Init(std::vector<std::string>& args_);


warning: function 'eCAL::Configuration::Init' has a definition with different parameter names [readability-inconsistent-declaration-parameter-name]

ECAL_API void Init(std::vector<std::string>& args_); ^

Additional context

ecal/core/src/config/ecal_config_initializer.cpp:238: the definition seen here

void Configuration::Init(std::vector<std::string>& arguments_) ^

ecal/core/include/ecal/config/configuration.h:77: differing parameters are named here: ('args_'), in definition: ('arguments_')

ECAL_API void Init(std::vector<std::string>& args_); ^

github-actions · 2024-06-04T11:46:14Z

ecal/tests/cpp/config_test/src/config_test.cpp

+
+  eCAL::Config::CmdParser parser;
+
+  std::vector<std::string> arguments;


warning: variable 'arguments' is not initialized [cppcoreguidelines-init-variables]

Suggested change

std::vector<std::string> arguments;

std::vector<std::string> arguments = 0;

github-actions · 2024-06-04T11:46:15Z

ecal/tests/cpp/config_test/src/config_test.cpp

+{
+  eCAL::Config::CmdParser parser;
+
+  std::vector<std::string> arguments;


warning: variable 'arguments' is not initialized [cppcoreguidelines-init-variables]

Suggested change

std::vector<std::string> arguments;

std::vector<std::string> arguments = 0;

hannemn

Reviewed together with @Peguen. Further restructuring of configuration object is needed but can be handled in a subsequent PR.

Peguen added 30 commits February 20, 2024 16:23

First working implementation for config management.

382d9ef

Merge branch 'master' into feature/user_config_implementation

0e9c039

Resolved some merging problems.

49e8ade

Fixed copy paste issue values for receivingOptions in ecal_config_ini…

70366d5

…tializer.cpp

Config Adaptions: Set LimitSize default paramaeter to template MIN va…

00b2ec1

…lue.

Refactored options and config access.

eb31c72

Changed defines to constexpr in ecal_dev_ini.h

548037f

Removed doubled constexpr.

5b07971

Removed former ecal_config cpp and h from core/CMakesLists

9cd3594

Removed ms_t from registration_options and handling in using functions.

9e1be9a

Fixed ecal_registration cpp time config import.

c5ab1dc

Refactored config options in application.

05ec616

Added process_options.terminal_emulator to config.

a861b01

Refactored sys- and processoptions as sub to application options.

e35df5a

Refactored default config to use ecal_def.h defines.

4a0ce13

Refactored ini config to use ecal_def.h

153ee9e

Moved IpAddressV4 definitions to private space.

94f91ae

Removed unused fields.

b196ab4

Fixed some GH actions compiling issues.

afe2332

Removed unnecessary comments in header files.

4887b02

Changed #include <limits> to correct header.

c768ba9

Merge branch 'master' into feature/user_config_implementation

6306d99

Some updates regarding new configuration management to ecal_core.

b4690a0

Updated submodules

4694927

Checked out versions of thirdparty modules equal to master branch.

388f5bc

Implemented termination, when wrong values are assigned to custom dat…

9303795

…a types.

Changed g_ecal_config() return value (PathWarning)

848c572

Added basic configuration tests.

2370814

Added ECAL_API macro for config calls.

e20d510

Added operator overloading of == for LimitSize(LS), LS == int, LS == …

7cafa3e

…unsigned int.

Peguen added 8 commits May 21, 2024 09:01

Enable shm for pubsub acknowledge test.

295f998

Included ecal_subscriber_config

915a7f7

Renamend Configuration member.

a2aca3e

Moved config to globalgalactic space. Implemented default config cons…

cf2148c

…tructor. Some renaming.

Restructure eCAL Application Options and default init for publisher/s…

80ef2a0

…ubscriber config.

Use custom types in publisher config.

cafb419

Restructered config file structure, config namespaces and and file na…

25116ea

…mes.

Added comments to config structs from ini file.

f7b8470

github-actions bot reviewed May 22, 2024

View reviewed changes

Peguen added 7 commits May 23, 2024 12:11

Some file renaming.

f6f3dd5

Updated docu for config.

913fee0

Changes for transportlayer config.

fd80bad

Try taking .ini file in normal initializing, as it was before.

e126b12

Merge remote-tracking branch 'origin/master' into feature/user_config…

f876c26

…_implementation

Added doc for publisher config.

25dd3e0

Defaulted publisher test again.

7be0dd9

github-actions bot reviewed May 24, 2024

View reviewed changes

Peguen added 3 commits May 27, 2024 07:30

Removed unncessary constructor/destructor in custom config classes.

aa1d1ca

Some ecal_cmd_parser cleanup.

db16ff6

Merge remote-tracking branch 'origin/master' into feature/user_config…

9613db0

…_implementation

Peguen changed the title ~~[DRAFT][config] Implementation of new configuration management~~ [config] Implementation of new configuration management May 28, 2024

Peguen marked this pull request as ready for review May 28, 2024 11:37

Peguen added 3 commits May 28, 2024 15:49

Fix ecal_cmd_parser checkForValidConfigFilePath function.

11044f1

Fixed ecal_cmd_parser problem with path separator attachments.

a9a4a41

Merge remote-tracking branch 'origin/master' into feature/user_config…

d1c606d

…_implementation

hannemn reviewed Jun 4, 2024

View reviewed changes

ecal/core/src/config/ecal_config_initializer.cpp Outdated Show resolved Hide resolved

Changed handling of cli arguments to std::vector<string> by default.

23ac095

github-actions bot reviewed Jun 4, 2024

View reviewed changes

hannemn approved these changes Jun 11, 2024

View reviewed changes

hannemn merged commit 5122562 into eclipse-ecal:master Jun 11, 2024
11 of 13 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[config] Implementation of new configuration management #1505

[config] Implementation of new configuration management #1505

Peguen commented Apr 4, 2024

github-actions bot left a comment

github-actions bot May 22, 2024

github-actions bot left a comment

github-actions bot May 24, 2024

github-actions bot left a comment

github-actions bot Jun 4, 2024

github-actions bot Jun 4, 2024

github-actions bot Jun 4, 2024

hannemn left a comment


		eCAL::Config::CmdParser parser;

		std::vector<std::string> arguments;

	std::vector<std::string> arguments;
	std::vector<std::string> arguments = 0;

[config] Implementation of new configuration management #1505

[config] Implementation of new configuration management #1505

Conversation

Peguen commented Apr 4, 2024

Description

github-actions bot left a comment

Choose a reason for hiding this comment

github-actions bot May 22, 2024

Choose a reason for hiding this comment

github-actions bot left a comment

Choose a reason for hiding this comment

github-actions bot May 24, 2024

Choose a reason for hiding this comment

github-actions bot left a comment

Choose a reason for hiding this comment

github-actions bot Jun 4, 2024

Choose a reason for hiding this comment

github-actions bot Jun 4, 2024

Choose a reason for hiding this comment

github-actions bot Jun 4, 2024

Choose a reason for hiding this comment

hannemn left a comment

Choose a reason for hiding this comment