Skip to content

sending Isobus type PGN#1164

Closed
Pat-I wants to merge 12 commits intoAgOpenGPS-Official:readyforreleasefrom
Pat-I:sending-ISOBUS-type-PGNs
Closed

sending Isobus type PGN#1164
Pat-I wants to merge 12 commits intoAgOpenGPS-Official:readyforreleasefrom
Pat-I:sending-ISOBUS-type-PGNs

Conversation

@Pat-I
Copy link
Copy Markdown
Contributor

@Pat-I Pat-I commented Jan 17, 2026

This change the AOG PGN 0xF2 to also send the CANBUS PGN number, his priority, source and destination address.

It now also send all 8 data byte of the ISOBUS Process Data Message (DDEntity)

The AOG message is now 7 bytes longer.

Right now it only send the "ISOBUS Process Data Message (DDEntity)", PGN 0xCB00(51968):
image
image

The goal of this change is to be able to exchange more isobus data in the future without having to create other PGNs (at the cost of more data sent today).

Pat-I added 3 commits January 16, 2026 22:50
This change the AOG PGN 0xF2 to also send the CANBUS PGN number, his priority, source  and destination address.

It now also send all 8 data byte of the ISOBUS Process Data Message (DDEntity)

The AOG message is now 7 bytes longer.
1:1 J1939 CANBUS message
fist 4 bytes make the identifier
8 next bytes are the payload
reversed the Frame identifier byte order.
@Pat-I
Copy link
Copy Markdown
Contributor Author

Pat-I commented Jan 19, 2026

I made some changes
It's an 1:1 J1939 CANBUS message. These two integer would make it ready to send with the teensy library
id: the first 4 bytes make the identifier
data: the 8 next bytes are the payload

Edited to remove the inaccurate test results.

Pat-I added 2 commits January 19, 2026 21:48
corrected a comment after further tests
This extra byte will allow to send some instruction with the CAN message.
Like a command to send on the machine bus for example.

This PGN could eventually be used from and to any device in AOG.
@Pat-I
Copy link
Copy Markdown
Contributor Author

Pat-I commented Jan 20, 2026

Description of the proposed modified AOG PGN F2:
0x80,0x81,SRC(AOG),0xF2,13(length),0(future use), ID3, ID2, ID1, ID0, DATA 0 TO 7, crc

Example: ISOBUS PGN: 0xCB00, priority: 3, DA(PS): F7, SA: 82
Data: Command: 3, element: 5, DDE: 75, data: 3847700

On CANBUS: 0xCCBF782, 53, 0, 4B, 0, 14, B6, 3A, 0
In AOG:
image

Overall the cost is 7 more bytes (+58%) and the benefit is to be able send (almost?) any CAN J1939 message between controllers in AOG

@Pat-I Pat-I marked this pull request as draft January 21, 2026 02:15
Pat-I added 3 commits January 22, 2026 00:09
Allow sending variable length CANBUS message

The ISOBUS process data message is still fixed to 8 bytes
allow variable data length for 0xCB00

Some parameters are optional
@Pat-I
Copy link
Copy Markdown
Contributor Author

Pat-I commented Jan 23, 2026

The last commit allow "private void SendCanbusMessage(int id, byte[] data)" to send a variable number of "data" bytes. There's no limit of bytes for this PGN although CANBUS is 1 to 8 bytes.

@Pat-I Pat-I marked this pull request as ready for review January 23, 2026 04:42
Pat-I added 3 commits February 3, 2026 22:11
Removed F1 to replace by F2 DDE160
Output the ISOBUS setpoint Condensed Work State DDE290 +
@Pat-I Pat-I marked this pull request as draft February 5, 2026 02:28
fix to be able to read 8 byte standard length PGNs
Fixed section painting behavior if the sections are ON while a field is opened or closed.
number |= 1ul << j;
}
}
if (!isJobStarted) number = 0; //
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can the variable number be changed to something appropriate?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I did not choose this name. I only set it to 0 in 2 more situations

@richardklasens richardklasens deleted the branch AgOpenGPS-Official:readyforrelease April 2, 2026 16:13
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