refactor(python/timing): refactor timing (wip)

Author: hakula139

binding/timing_binding.cpp

@@ -29,10 +29,6 @@ auto TimeFunctionWrapper(const std::function<void()> &func) -> std::int64_t {
 }  // namespace
 
 void BindTiming(py::module &m) {
-  // Bind utility functions
-  m.def("to_human_readable", &ToHumanReadable);
-  m.def("time_function", &TimeFunctionWrapper);
-
   // Bind Timer class
   py::class_<Timer>(m, "Timer")
       .def(py::init<>())
@@ -43,10 +39,9 @@ void BindTiming(py::module &m) {
       .def("get_elapsed_us", &GetElapsedWrapper<std::chrono::microseconds>)
       .def("get_elapsed_ms", &GetElapsedWrapper<std::chrono::milliseconds>)
       .def("get_elapsed_s", &GetElapsedWrapper<std::chrono::seconds>)
-      .def("get_elapsed_string", &Timer::GetElapsedString)
-      .def("__str__", &Timer::GetElapsedString)
-      .def("__repr__", [](const Timer &self) {
-        return std::format("<Timer(elapsed='{}') at {}>", self.GetElapsedString(),
-                           static_cast<const void *>(&self));
-      });
+      .def("get_elapsed_str", &Timer::GetElapsedString);
+
+  // Bind utility functions
+  m.def("to_human_readable", &ToHumanReadable);
+  m.def("time_function", &TimeFunctionWrapper);
 }

include/timing/timer.hpp

@@ -109,6 +109,18 @@ class Timer {
    */
   void Stop() noexcept { end_time_ = Clock::now(); }
 
+  /**
+   * @brief Reset the timer to start a new measurement
+   *
+   * Resets the timer by capturing the current time as the new start point and clearing any
+   * previously recorded end time. This allows reusing the same timer instance for multiple
+   * measurements.
+   */
+  void Reset() noexcept {
+    start_time_ = Clock::now();
+    end_time_ = TimePoint{};
+  }
+
   /**
    * @brief Get the elapsed time in the specified duration type
    *
@@ -166,18 +178,6 @@ class Timer {
    */
   [[nodiscard]] auto GetElapsedString() const noexcept -> std::string;
 
-  /**
-   * @brief Reset the timer to start a new measurement
-   *
-   * Resets the timer by capturing the current time as the new start point and clearing any
-   * previously recorded end time. This allows reusing the same timer instance for multiple
-   * measurements.
-   */
-  void Reset() noexcept {
-    start_time_ = Clock::now();
-    end_time_ = TimePoint{};
-  }
-
  private:
   TimePoint start_time_;  ///< The time point when timing started
   TimePoint end_time_;    ///< The time point when timing stopped