tagyoureit / nodejs-poolController

An application to control pool equipment from various manufacturers.
GNU Affero General Public License v3.0
315 stars 94 forks source link

Intelliflo VS 3050 read as VF #292

Closed Telemachus1977 closed 3 years ago

Telemachus1977 commented 3 years ago

Describe the bug VS 3050 pump is read as VF. GPM are not reported, but watts are.

To Reproduce Just shows up on home screen.

Expected behavior I would expect to see VS with RPM

Packet Capture Attached

Pool Equipment

Desktop (please complete the following information):

Smartphone (please complete the following information):

Additional context Add any other context about the problem

replay.zip

rstrouse commented 3 years ago

Just to be sure on this could you please check your setup in the EasyTouch panel. This will be under Settings --> IntelliFlo --> Pump 1 the signature that is coming from EasyTouch is actually a VF pump.

@tagyoureit for reference here is the Action 24 message for that pump. Byte(1) is 255. [255, 0, 255][165, 20, 15, 16, 24, 31][1, 255, 0, 255, 255, 6, 7, 1, 13, 11, 12, 0, 3, 0, 3, 0, 3, 0, 3, 0, 3, 3, 8, 122, 28, 232, 232, 232, 232, 232, 232][10, 95]

Telemachus1977 commented 3 years ago

20210509_204824 20210509_204820 20210509_204810

rstrouse commented 3 years ago

Actually what I want you to verify is the settings on the main panel. I am not sure whether these actually exist in the handheld controller but the pump type not the speed is what I want you to verify. image

Telemachus1977 commented 3 years ago

For sure I will verify that.

The label on the pump itself says "VS-3050"

But as the handheld is reading RPM shouldn't njs-pC also display that data?

Screenshot_20210510-133939_Chrome

rstrouse commented 3 years ago

But as the handheld is reading RPM shouldn't njs-pC also display that data?

Yes it should but the panel is reporting what we have seen in the past as a VF pump and RPM is not valid for that pump. It sends out garbage in that spot. I believe what we have found is simply a difference in the EasyTouch firmware. Double checking that information will help us to map the signature to the proper pump type. It is very odd that it is returning 255 which is way out of range for what we have seen VS pumps at in the past. In fact 255 is typically what Pentair reports when it doesn't have a definition. Either way we can coerce it if we are sure that the byte is actually set on the panel.

Telemachus1977 commented 3 years ago

I do not even see anywhere on the menus where it gives me a pump type. VS was the only option back in 2009 AFAIK. RPM is all I have ever been able to program.

I have recieved my programmer and plan on updating my firmware in the next few weeks so this may be a moot issue.

rstrouse commented 3 years ago

That was my suspicion. 255 means that the byte never gets set. I think it is safe for us to call that a VS pump. I’ll let you know when we add it. Probably tomorrow.

Telemachus1977 commented 3 years ago

20210513_060713 20210513_060701

Thanks. Certainly a VS pump.

Telemachus1977 commented 3 years ago

20210513_060914

Telemachus1977 commented 3 years ago

Screenshot_20210514-091629_Chrome

Updating the firmware to the newest available from Pentair has resolved this problem.

rstrouse commented 3 years ago

Thanks for letting us know.