-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add timedelta verification to merge_double_ended_times #175
Conversation
…le ended to double ended
"Add timedelta verification to If you change the PR title to that and do "squash and merge" when merging the PR you can make the commit message on the main branch fit the max. line length |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice that you managed to implement this verification :)
Just some small comments from me
Oh and don't forget to update the changelog! |
timedelta64[ns] to timedelta64[s] Co-authored-by: Bart Schilperoort <[email protected]>
Rearrange print statements. Suggested by @BSchilperoort Co-authored-by: Bart Schilperoort <[email protected]>
…b.com/dtscalibration/python-dts-calibration into merge-single-ended-to-double-check-dt
…le ended to double ended