rest() from Iec6205621Client

[youserj]

def rest(self, duration=None):
    """
    The protocol needs some timeouts between reads and writes to enable the device
    to properly parse a message and return the result.
    """

    _duration = duration or (self.reaction_time * 1.25)
    logger.debug(f"Resting for {_duration} seconds")
    time.sleep(_duration)

For self.use_short_reaction_time = True dont work. At first need decide time of writing message:

def time_write(self, msg):
    """
    return need time of write
    """
    return (1 / self.port.get_settings()['baudrate'] * len(msg)) 
    #or return (1 / self._current_baudrate * len(msg)*10) 

where msg is the ack_message.to_bytes() or 6

After self.rest(self.reaction_time +self.transport.time_write(ack_message.to_bytes()))
or

[Krolken]

Hi.

Are you proposing a change? I don't have any devices that uses short reaction time and cannot test against it.
Do you have one?

[youserj]

yes. I develop soft and powermeter at the moment.
And more one problem:
(Acknowledgement/option select message = 20ms) + (short_reaction_time = 20ms) = 40ms
however - self.port.flush() - give delay about 50ms(in windows 10).
can I remove flush?
and make delay use time.time() into rest method

[Krolken]

If you want to try out some differnet settings i sugest you subclass and override.
I don't see how flush should be a problem since it returns when all data is sent. You don't want to close the port while stil having data in the outbuffer.

Also the timings are mostly not a problem, since you write and then do a blocking read. So the only time you have to make sure to wait is when you receive something and then send it back.

What meter are you using?

[Krolken]

Closing due to inactivity. Please open again if needed.