Hi Alia, Thanks for your comment, see inline...
> -----Original Message----- > From: Pce [mailto:[email protected]] On Behalf Of Alia Atlas > Sent: 15 September 2016 19:13 > To: The IESG <[email protected]> > Cc: [email protected]; [email protected]; > [email protected] > Subject: [Pce] Alia Atlas' No Objection on > draft-ietf-pce-pcep-service-aware-12: (with COMMENT) > > Alia Atlas has entered the following ballot position for > draft-ietf-pce-pcep-service-aware-12: No Objection > > When responding, please keep the subject line intact and reply to all email > addresses included in the To and CC lines. (Feel free to cut this introductory > paragraph, however.) > > > Please refer to > https://www.ietf.org/iesg/statement/discuss-criteria.html > for more information about IESG DISCUSS and COMMENT positions. > > > The document, along with other ballot positions, can be found here: > https://datatracker.ietf.org/doc/draft-ietf-pce-pcep-service-aware/ > > > > ---------------------------------------------------------------------- > COMMENT: > ---------------------------------------------------------------------- > > I am concerned that the 24 bit values of microseconds are being represented > in IEEE 32-bit floating point. > A quick look at conversions indicates that all integers will up to 6 > significant figures can be converted without loss of precision. That > implies that values of over 1 second may not be accurately sent. It would > be useful to at least refer to the precision issue in the document. I don't > expect that the loss of precision at the microsecond level is critical. > > [Dhruv] I could add a sentence for delay and delay variation - The conversion from 24 bit integer to 32 bit IEEE floating point could introduce some loss of precision. Will this be okay? Regards, Dhruv > _______________________________________________ > Pce mailing list > [email protected] > https://www.ietf.org/mailman/listinfo/pce _______________________________________________ Pce mailing list [email protected] https://www.ietf.org/mailman/listinfo/pce
