Merge Freeze to Develop #131
Merged
Conversation
This file contains 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
In order to prepare the different architectures that the CDevice to be used on startup can be configured via CMake options in this commit some clean-ups and generalisations have been performed.
add a base for testing the ctf traces. It has classes representing all possible events being traced, a way of reading the trace files, and a minimal example of comparison of a small FB network.
Add a virtual accessor function for adapters to enable subclasses to determine the memory location of adapter instances.
Refactor monitoring event counters in preparation for separating dynamic interface creation from the function block base class.
Move the dynamic interface setup and required variables to the generic function block and make accessor functions pure virtual.
….1 with reset functionality by Franz Hoepfinger.
…put and connection settings accordingly.
Updated the copyright notice to reflect 2024 for HR Agrartechnik GmbH in the E_TON_fbt.cpp and E_TON_fbt.h files.
…put and connection settings accordingly.
Include a reset feature in Timer Function Blocks for better control and flexibility. This update enhances the existing pulse functionality.
…ables to default values uniformly.
- Renamed timeout-related files with "_adp" suffix - Updated CMakeLists to include new file names
Changed include paths for timeout classes in event adapter files to match the updated file names. This ensures proper linking and consistency within the codebase.
- Updated source files to include new function block types for timeouts. - Renamed header and source files for E_TimeOut and E_RTimeOut.
With this change CDevice provides a static method that will create an isntance of the device to be used by this 4daic FORTE instance. This reduces code duplication and wrong device creation. Furthermore it is the basis that different device types can be configured via CMake.
With this commit it is now possible to select the device to be used for a specific 4diac FORTE build. As part of this change OPCUA_DEV is currently not build able. Will be fixed in a seperate commit.
* Choose device based on macro This change chooses the device based on a macro instead of conditional compilation. It also works again with OPCUA_DEV. * Choose device based on configure file and type alias This change chooses the device based on a configure file with a type alias instead of a macro.
* We have to 'delete' anything created using 'new' * Swap a vector with a new empty vector will effectively delete it from memory. Same result as assigning a new empty vector to a vector but swap more readable.
ernstblechaPT
added a commit
to ernstblechaPT/4diac-forte
that referenced
this pull request
Jun 7, 2024
In eclipse-4diac#131 the memset was removed, this results in random values showing up on the event counters
ernstblechaPT
added a commit
to ernstblechaPT/4diac-forte
that referenced
this pull request
Jun 7, 2024
In eclipse-4diac#131 the memset was removed, this results in random values showing up on the event counters
azoitl
pushed a commit
that referenced
this pull request
Jun 7, 2024
In #131 the memset was removed, this results in random values showing up on the event counters
kaartee
added a commit
to kaartee/4diac-forte
that referenced
this pull request
Jul 2, 2024
kaartee
pushed a commit
to kaartee/4diac-forte
that referenced
this pull request
Jul 2, 2024
dok-net
added a commit
to dok-net/org.eclipse.4diac.forte
that referenced
this pull request
Jul 3, 2024
dok-net
added a commit
to dok-net/org.eclipse.4diac.forte
that referenced
this pull request
Jul 16, 2024
dok-net
added a commit
to dok-net/org.eclipse.4diac.forte
that referenced
this pull request
Jul 18, 2024
azoitl
pushed a commit
that referenced
this pull request
Jul 18, 2024
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.
No description provided.