Skip to content
Snippets Groups Projects

fix bug calculate find_tps_tm when the end time fall into the last 5 minute of a day

Merged Lan Dam requested to merge i81_tps_second_marker into master

closes #81 (closed)

Logic changes are only at the end of handling_data.py find_tps_tm_idx() to handle the case that x_idx is None.

Other changes are variables and functions name to be more descriptive and unittests are added to test the 2 functions involve in tps calculating: get_start_5mins_of_diff_days() and find_tps_tm_idx()

Edited by Lan Dam

Merge request reports

Pipeline #2427 passed

Pipeline passed for 21b9a0f1 on i81_tps_second_marker

Merged by Kien LeKien Le 1 year ago (Apr 24, 2023 5:50pm UTC)

Loading

Pipeline #2487 passed

Pipeline passed for 6202d7f5 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading