PPG Events

[griffincollaboration]

PPG Events

[VinzenzBildstein]

I think that is the old interpretation of the PPG format, now the "new" PPG pattern is the requested PPG pattern and the "old" PPG pattern is the actual PPG pattern (as read back from the PPG).

[r3dunlop]

Ah, we should put some comments or change the naming scheme in the GRSISort data parser then.

On Nov 13, 2015, at 1:35 PM, Vinzenz Bildstein [email protected] wrote:

I think that is the old interpretation of the PPG format, now the "new" PPG pattern is the requested PPG pattern and the "old" PPG pattern is the actual PPG pattern (as read back from the PPG).

—
Reply to this email directly or view it on GitHub #5 (comment).

[SmithJK]

This is inconsistent with some of the PPG patterns from the last run:

0x820ffff0 0xdd771329 0x0008c008 0x9008c008 0xa0000000 0xb0000000 0xe0000000
0x820ffff0 0xd0000000 0x0002c002 0x9002c002 0xa8f0d1c0 0xb0000000 0xe008c008
0x820ffff0 0xd0000000 0x0001c001 0x9001c001 0xaee6b2c2 0xb0000000 0xe002c002
0x820ffff0 0xd0000000 0x0004c004 0x9004c004 0xa03e0562 0xb0000001 0xe001c001

Based on these, I think that the last word is now 28 bits of the Old PPG. Obviously, this doesn't leave us with a corruption check, though I suppose the repetition within the pattern itself is a decent check.

[VinzenzBildstein]

That is exactly what I said (though I left out that the old PPG status is now in the trailer). We used to read the new PPG, then the old PPG, and find the new PPG repeated in the trailer. This was (I think) the case for the Cd/In experiments in August.
Since then the format has changed (though not the TPPG class). Now we get the requested new PPG pattern, then the read back new PPG pattern, and the old PPG in the trailer. The only thing we changed in the GRSISort code was to omit the check of the trailer vs the new PPG, meaning we're storing the requested PPG as new PPG, and the read back PPG as old PPG.