TimurDela

Results 19 comments of TimurDela

Yes, this is still a draft. I am waiting for #11106 to be merged so I can rebase this one and make it more consistent with the new syntax

Hi @srebhan, I was not aware that some devices required the address 0 to be queried. However, my problem is that I have a device with around 1300 signals. My...

For the `holding` registerType, maxBatchSize is 125. Most of my entries are float32 and take 2 addresses each, so that is maximum 62 fields per request. 1300 / 62 =...

Hello @srebhan , is there something else you want me to modify in this pull request?

Thank you @srebhan I will have a look at it as soon as I can but it may take a few days.

I have left a few `fmt.Printf` here and there for comparing with #11273 but of course they are meant to be removed

Just to be clear, do you want me to rebase this pull request on top of [srebhan:modbus](https://github.com/srebhan/telegraf/tree/modbus) or do you want me to add my changes into pull request #11273...

I am sorry but I do not understand the error message from the failing job. Locally, when I run `make lint-branch` none of the messages concern files in `plugins/inputs/modbus`. I...

Hello @srebhan, any chance you take a look at this PR?

Hello @srebhan , did you have time to look at this: https://github.com/influxdata/telegraf/pull/11106#discussion_r982052477 ?