output status updated only every 250 ms

[t0mpr1c3]

in osPID-Firmware.ino the output status is written to the relay only every 250ms. This seems excessively granular in a default cycle period of 5s. (line 175, 250 is added to ioTime, meaning that the output status will not be updated for another 250ms).

[br3ttb]

This is because the PID Loop itself is evaluated every 250mS. There is no reason to do the io any faster than the PID is dealing with them.

[t0mpr1c3]

Perhaps I should clarify. I was referring to the writing of the output to the relay using WriteToOutputCard() which is not an asynchronous process. As far as I can see WriteToOutputCard() needs to be called repeatedly during the cycle until it is time for the relay to go high or low. For that reason calling WriteToOutputCard() every 250 ms seems too infrequent.

[br3ttb]

AH! that. is a legitimate point. sorry that I misunderstood. having separate input and output times instead of one ioTime is probably the way to go.

[t0mpr1c3]

Yes. Perhaps even do away with the output time altogether and call WriteToOutputCard in every iteration of the loop, because it is a high priority task and quick to execute.