Skip to content

Comments

Fix wrong metric description (copy-past from previous)#618

Merged
davidv1992 merged 1 commit intopendulum-project:mainfrom
pohlm01:fix-metric-description
Feb 27, 2025
Merged

Fix wrong metric description (copy-past from previous)#618
davidv1992 merged 1 commit intopendulum-project:mainfrom
pohlm01:fix-metric-description

Conversation

@pohlm01
Copy link
Member

@pohlm01 pohlm01 commented Feb 21, 2025

Apparently, the description of time_source as been copied from ptp_timescale, the metric one above. I tried to come up with a meaningful description based on Section 8.2.4.9 of the IEEE Std 1588-2019. Feel free to adopt it if necessary.

@davidv1992 davidv1992 enabled auto-merge February 27, 2025 07:30
@codecov
Copy link

codecov bot commented Feb 27, 2025

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 61.58%. Comparing base (758aa8b) to head (adbb8dd).
Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
statime-linux/src/metrics/format.rs 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #618   +/-   ##
=======================================
  Coverage   61.58%   61.58%           
=======================================
  Files          62       62           
  Lines        8415     8415           
=======================================
  Hits         5182     5182           
  Misses       3233     3233           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@pohlm01 pohlm01 force-pushed the fix-metric-description branch from 149593a to adbb8dd Compare February 27, 2025 07:51
@davidv1992 davidv1992 added this pull request to the merge queue Feb 27, 2025
Merged via the queue into pendulum-project:main with commit 6d27d58 Feb 27, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants