Skip to content

Various adjustments and optimizations#635

Merged
PTKu merged 18 commits intodevfrom
various-updates-adjustemets-due-local-storage
Apr 3, 2025
Merged

Various adjustments and optimizations#635
PTKu merged 18 commits intodevfrom
various-updates-adjustemets-due-local-storage

Conversation

@PTKu
Copy link
Copy Markdown
Contributor

@PTKu PTKu commented Mar 28, 2025

This pull request includes several changes across multiple files to enhance the functionality and fix errors in the AXOpen.Components.Cognex.Vision namespace. The most important changes involve updates to the sequencer steps, error descriptions, and workflow improvements.

Sequencer Steps and Methods:

  • Increased the array size of Steps from 4 to 10 in multiple components to accommodate additional steps. [1] [2] [3] [4]
  • Added new steps for restoring state and moving to the next step in the sequencer methods. [1] [2] [3] [4]
  • Removed redundant method calls and comments to clean up the code. [1] [2] [3] [4] [5] [6]

Error Descriptions:

  • Corrected error descriptions for writing user data in AxoDataman and AxoInsight components. The previous description incorrectly stated "1286 bytes" instead of "128 bytes". [1] [2] [3]

Workflow Improvements:

  • Updated the GitHub Actions workflow to fetch and compare stored application run results, ensuring that steps are skipped only if both the commit SHA and previous result match.

Configuration Adjustments:

  • Changed the default values for InfoTime and ErrorTime in the AxoInsight_Config structure to provide more appropriate timing settings.

Device and IO System Configuration:

  • Removed the configuration for plc_line devices and IO systems from the plc_line.hwl.yml file, indicating a possible deprecation or reconfiguration of these systems.

PTKu and others added 16 commits March 28, 2025 16:32
… methods in AxoContext; update PlcTextList formatting in AxoRemoteTask for improved readability
@TomKovac TomKovac self-requested a review April 2, 2025 10:35
@PTKu PTKu changed the title Change method visibility from private to protected for Open and Close… Various adjustments and optimizations Apr 3, 2025
@PTKu PTKu merged commit 7f343d6 into dev Apr 3, 2025
1 check passed
@PTKu PTKu deleted the various-updates-adjustemets-due-local-storage branch April 3, 2025 11:59
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.

2 participants