The actual playback time is much larger than the timestamp

[zixuan-su]

Describe the bug

I have a 120 s long message ，then I find that It takes me 324.2 s to play. The Timestamp is still about 120 s, but the difference from the actual playback time is nearly 2.7 times . The file is https://github.com/hardbyte/python-can/blob/develop/can/player.py

To Reproduce

Expected behavior

The actual time spent is the same as the timestamp duration

Additional context

OS and version: Ubuntu 1804
Python version: 3.9.9
python-can version: 4.0.0
python-can interface/s (if applicable):

Traceback and logs

def func():
    return "hello, world!"

[zariiii9003]

Could you replace this

python-can/can/io/player.py

Line 154 in 3a7c80e

sleep(sleep_period)

with this

            if sleep_period > 1e-3:
                sleep(sleep_period)

and check if it gets better? You have a very high message rate, i assume the sleep() call slows you down too much.

[zariiii9003]

try this branch please: https://github.com/zariiii9003/python-can/tree/message_sync

[zixuan-su]

Could you replace this

python-can/can/io/player.py

Line 154 in 3a7c80e

sleep(sleep_period)

with this

            if sleep_period > 1e-3:
                sleep(sleep_period)

and check if it gets better? You have a very high message rate, i assume the sleep() call slows you down too much.

I replace

with _create_bus(results, **additional_config) as bus:
with LogReader(results.infile, **additional_config) as reader:

        in_sync = MessageSync(
            cast(Iterable[Message], reader),
            timestamps=results.timestamps,
            gap=results.gap,
            skip=results.skip,
        )

with

Msg_List = list(BLFReader(results.infile))

print(f"Can LogReader (Started on {datetime.now()})")
     
try:
    for message in Msg_List:
        #print(message)  
        if message.is_error_frame and not error_frames:
            continue        
        if verbosity >= 3:

It only takes 44 s to play a 120 s BLF file

Then I will try your suggestion

[zixuan-su]

try this branch please: https://github.com/zariiii9003/python-can/tree/message_sync

I follow your suggestion,that`s cool !!!

[zariiii9003]

Thank you for your feedback, then i'll merge it into develop.