Codecov Report

Merging #1379 (876dacb) into develop (2da28c1) will increase coverage by 0.01%.
The diff coverage is 90.90%.

❗ Current head 876dacb differs from pull request most recent head f64b6e7. Consider uploading reports for the commit f64b6e7 to get more accurate results

@@             Coverage Diff             @@
##           develop    #1379      +/-   ##
===========================================
+ Coverage    66.21%   66.22%   +0.01%     
===========================================
  Files           86       86              
  Lines         9021     9027       +6     
===========================================
+ Hits          5973     5978       +5     
- Misses        3048     3049       +1     

Remaining tasks:

• If the append option is requested, then open the compressed Writer with append mode
• Get the compressed BLFWriter functional with the append option

The current changes make it so that blf.gz files are written with a complete header. The solution is not long term, because it simply ensures that nothing is written until the rollover occurs.

import can
from datetime import datetime


def u_to_dt(ts):
    return datetime.utcfromtimestamp(ts).strftime("%Y-%m-%d %H:%M:%S")


def main():
    file_name = "private_2022-08-24T084948_#000.blf.gz"

    with can.LogReader(file_name) as reader:
        for m in reader:
            print(m.timestamp)

    reader = can.LogReader(file_name)
    print(
        f"start: {u_to_dt(reader.start_timestamp)} "
        f"stop: {u_to_dt(reader.stop_timestamp)}"
    )


if __name__ == "__main__":
    main()

I have modified the description of issue #1378 and this PR now contains changes that are out of scope. A combination of #1383, #1384 and #1385 will cover the material previously intended to be addressed in this PR.