Skip to content
Snippets Groups Projects

aggregate_logger: fix bug with timestamp at beginning of the new year

Merged Alexander Hinz requested to merge (removed):master into master
All threads resolved!

@schaefed

The problem was, that a a aggregated year should start with the timestamp 00:30 when aggregated to 30 minutes and with the 01.01. when aggregated to a whole day.
The new solution seems to be a bit hacky.
In the writeData function a freq is needed (I renamed is to change_freq) for the iterDataFreq(data, change_freq) function in dacces.py Until now change_freq=y, that isn't even listed in freqs dictionary in the iterDataFreq function in dacces.py , but it works only for aggregation for 1d, not for the 30min aggregation.
For the 30min it works with change_freq=yearly.
Maybe you have an idea how to solve this better.

Merge request reports

Approval is optional

Merged by David SchäferDavid Schäfer 3 years ago (Feb 7, 2022 10:54am UTC)

Merge details

  • Changes merged into master with 6e1739a6 (commits were squashed).
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Alexander Hinz added 1 commit

    added 1 commit

    • dc5e8d1c - do_logger_aggregation: changes for the timestamp bug and change in dacces for skipping empty files

    Compare with previous version

  • David Schäfer resolved all threads

    resolved all threads

  • David Schäfer mentioned in commit 6e1739a6

    mentioned in commit 6e1739a6

  • Merged and deployed!

  • Please register or sign in to reply
    Loading