SparkplugNet icon indicating copy to clipboard operation
SparkplugNet copied to clipboard

Sequence Number is missing in NodeDataReceivedAsync & NodeBirthReceivedAsync

Open ankitagojiya opened this issue 1 year ago • 7 comments

According to the Sparkplug specification, we must submit a REBIRTH request if we get a message out of order. However, the NodeData which we receive in the host application(using NodeDataReceivedAsync ) just includes metrics not sequence number.

image

ankitagojiya avatar Dec 05 '23 12:12 ankitagojiya

Need to check this.

SeppPenner avatar Dec 07 '23 16:12 SeppPenner

Hello @SeppPenner
any updated on this . Need this to implement reorder timeout functionality using this.

ankitagojiya avatar Jan 31 '24 07:01 ankitagojiya

Hello @SeppPenner any updated on this . Need this to implement reorder timeout functionality using this.

Sorry, I haven't started with this yet. The plan is to work on this today and tomorrow.

SeppPenner avatar Feb 10 '24 15:02 SeppPenner

Do you want this to be within the list of metrics or do you want to have this as seperate property like I did it in https://github.com/SeppPenner/SparkplugNet/commit/611afa9cdc044419c80abd78bc8b5550ed10e78c?

SeppPenner avatar Mar 22 '24 16:03 SeppPenner

This should be fixed in version > 1.3.5+. Check https://github.com/SeppPenner/SparkplugNet/commit/9cb2deb43492926293c49338667bbd83eb6a286b that implicitely fixed this.

SeppPenner avatar Mar 24 '24 20:03 SeppPenner

Hello @SeppPenner Thanks for the above updates. I have verified it in the latest release (1.3.8) but i am still not getting Seq Number in NBIRTH or NDATA event. For the clarification i am referring to Seq number like below image not the bdseq(sessionNumber) metrics.

image

ankitagojiya avatar May 31 '24 05:05 ankitagojiya

Hello @SeppPenner Thanks for the above updates. I have verified it in the latest release (1.3.8) but i am still not getting Seq Number in NBIRTH or NDATA event. For the clarification i am referring to Seq number like below image not the bdseq(sessionNumber) metrics.

image

Thanks for the clearification, need to check this.

SeppPenner avatar Jul 02 '24 13:07 SeppPenner