Skip to content

Add example for Assembly Mapping Object#15

Open
AnJoha wants to merge 5 commits intohms-networks:mainfrom
AnJoha:assembly-mapping-object
Open

Add example for Assembly Mapping Object#15
AnJoha wants to merge 5 commits intohms-networks:mainfrom
AnJoha:assembly-mapping-object

Conversation

@AnJoha
Copy link
Collaborator

@AnJoha AnJoha commented Jan 8, 2026

No description provided.

@AnJoha AnJoha requested review from cldhms and Copilot January 8, 2026 11:00
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request adds example code demonstrating how to use the Assembly Mapping Object feature in the ABCC driver. The changes extend the existing network data parameters example by introducing assembly mapping capabilities with multiple read and write assemblies.

Key Changes:

  • Extended the ADI setup from 2 to 4 instances (adding torque and reference torque parameters)
  • Implemented assembly mapping with 3 write assemblies and 2 read assemblies
  • Added EtherNet/IP producing/consuming instance configuration to support the assembly mappings

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
src/example_application/abcc_network_data_parameters.c Extended example with assembly mapping functionality, added two new ADI entries, implemented assembly mapping instances and related callback functions
src/abcc_adaptation/abcc_driver_config.h Enabled assembly object support and added EtherNet/IP instance configurations for producing and consuming assemblies

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@AnJoha AnJoha requested review from cldhms and Copilot January 8, 2026 15:21
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

cldhms
cldhms previously approved these changes Jan 9, 2026
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