Add Timestamp.get_time and improve Timestamp.seconds_since_unix_epoch (…

…#111)

src/value.rs

@@ -194,16 +194,12 @@ impl _Timestamp {
         }
     }
     #[getter]
-    pub fn seconds_since_unix_epoch(&self) -> PyResult<f64> {
-        match self
-            .0
-            .get_time()
-            .to_system_time()
-            .duration_since(std::time::UNIX_EPOCH)
-        {
-            Ok(o) => Ok(o.as_secs_f64()),
-            Err(e) => Err(e.to_pyerr()),
-        }
+    pub fn get_time(&self) -> u64 {
+        self.0.get_time().0
+    }
+    #[getter]
+    pub fn seconds_since_unix_epoch(&self) -> f64 {
+        self.0.get_time().as_secs_f64()
     }
 }
 

zenoh/value.py

@@ -114,11 +114,18 @@ class Timestamp(_Timestamp):
     def _upgrade_(this: _Timestamp) -> 'Timestamp':
         return _Timestamp.__new__(Timestamp, this)
     @property
+    def get_time(self) -> int:
+        """
+        Returns the time part, as generated by the Zenoh HLC in NTP64 format (See https://datatracker.ietf.org/doc/html/rfc5905#section-6).
+        """
+        return super().time
+    @property
     def seconds_since_unix_epoch(self) -> float:
         """
         Returns the number of seconds since the Unix Epoch.
 
-        You shouldn't use this for comparison though, and rely on comparison operators between members of this class.
+        Considering the large number of seconds since the Unix Epoch, the precision of the resulting f64 is in the order of microseconds.
+        Therefore, it should not be used for comparison. Directly comparing Timestamp objects is preferable.
         """
         return super().seconds_since_unix_epoch