-
-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Y505-A Aquaculture Optical Dissolved Oxygen Sensor support? #34
Comments
False alarm. It is working on Y504 config. |
@primus192, thanks for letting us know that the Y504 functions work well with the Y505 sensor. I expected that they might, but I've never had the modbus manual for the Y505 nor one to test, so I just didn't know. What are the first 3 digits of your serial number on the Y505? With that info, I could get this to work with the Y505 explicitly |
Hello. Meanwhile i am currently using mainly Y4000 sondes. Yosemitech succesively Single reference readings stays the same tho and new measurements are being
|
Thanks for that info!!! Let's spin this into a new issue and work together to get a code update to work with all models. I have one of the first sondes that they made, and I worked with some clients in 2021 to deploy a bunch of other sondes. Are you able to do a pull request with your updated code? |
Sure i will try to do this tomorrow :) Maybe we can work it out by adding
Y4000V2 or Y4000NEW variant and create separate switch case for it or
completely new getValues overloaded function. Will try to implement
something.
wt., 17 wrz 2024, 16:24 użytkownik Anthony Aufdenkampe <
***@***.***> napisał:
… Thanks for that info!!!
Let's spin this into a new issue and work together to get a code update to
work with all models. I have one of the first sondes that they made, and I
worked with some clients in 2021 to deploy a bunch of other sondes.
Are you able to do a pull request with your updated code?
—
Reply to this email directly, view it on GitHub
<#34 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AVLNJLC3MICYP64BPL7MX5TZXA3RTAVCNFSM6AAAAABOKGP4A2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJWGAYDINRYG4>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
I made pull request with my changes in regards with Y4000v2 sondes. Please be easy on me as it is one of the first contributions that i made. |
Hi. Is Y505-A sensor supported? I see it in the yosemitechModel but it is commented and i appears not to be working on Y504 config. Thanks for any help.
The text was updated successfully, but these errors were encountered: