Skip to content

[OPTIMIZATION] Optimize AxoTask to provide lightwight version no messaging#642

Merged
PTKu merged 13 commits intodevfrom
641-optimization-optimize-axotask-to-provide-lightwight-version-no-messaging
Apr 3, 2025
Merged

[OPTIMIZATION] Optimize AxoTask to provide lightwight version no messaging#642
PTKu merged 13 commits intodevfrom
641-optimization-optimize-axotask-to-provide-lightwight-version-no-messaging

Conversation

@PTKu
Copy link
Copy Markdown
Contributor

@PTKu PTKu commented Apr 2, 2025

This pull request includes several updates across different namespaces and files, primarily focusing on increasing the maximum length of log messages and improving error handling in logging mechanisms. Additionally, there are some changes related to class inheritance and configuration templates.

Log Message Length Update:

  • Increased the maximum length of the _message field from 80 to 254 characters in multiple files, including IAxoLogger.st, Mock.st, AxoLogEntry.st, AxoLogger.st, NULLs.st, and Abstractions.st. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12]

Error Handling Improvements:

  • Added a try-catch block to handle exceptions during the log dequeuing process in AxoLogger.cs. [1] [2]

Class Inheritance Change:

  • Changed the inheritance of the AxoStep class from AxoTask to AxoTaskLight in AxoStep.st.

Configuration Template Update:

  • Updated the CycleCommunicationLoad value in the 6ES7516-3AP03-0AB0_v3_1.hwl.yml configuration file to use a variable. [1] [2]

Miscellaneous:

  • Added a call to SetSuspendCyclicExecuteIsNotCalledCheck(TRUE) in AxoAlert.st to improve the cyclic execution check.
  • Commented out unnecessary lines in AxoDialog.st and AxoRemoteTask.st. [1] [2]

closes #641

@PTKu PTKu linked an issue Apr 2, 2025 that may be closed by this pull request
@PTKu PTKu marked this pull request as ready for review April 3, 2025 13:45
@PTKu PTKu merged commit 8f0c32e into dev Apr 3, 2025
1 check passed
@PTKu PTKu deleted the 641-optimization-optimize-axotask-to-provide-lightwight-version-no-messaging branch April 3, 2025 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[OPTIMIZATION] Optimize AxoTask to provide lightwight version no messaging

1 participant