hamishcunningham / fishy-wifi

Scripts, notes and the odd subaquatic gizmo for the ESP8266 and what-have-you.
GNU Affero General Public License v3.0
25 stars 13 forks source link

Edit pin numbers to match COM3505 schematic #9

Closed cgmcintyr closed 6 years ago

cgmcintyr commented 6 years ago

Wiring up the board the waterelf32 sketch does not seem to be reading the correct pin numbers for the following sensors:

The humidity sensor is setup with correct pins as of c7e3cb415f0ee15786c1d1fe610efdc433e5f889

hamishcunningham commented 6 years ago

Hmmm, c7e3cb415f0ee15786c1d1fe610efdc433e5f889 was intended to fix this -- does that shed any light?

layerzerolabs commented 6 years ago

I know that the ultrasonic sensor needs a pin change - I'll do that in the next ten min so a git pull should resolve that one... as to the waterproof temp sensor I'll have another look and see if there is another bug there...

G

On 15 Nov 2017 16:10, "Hamish Cunningham" notifications@github.com wrote:

Hmmm, c7e3cb4 https://github.com/hamishcunningham/fishy-wifi/commit/c7e3cb415f0ee15786c1d1fe610efdc433e5f889 was intended to fix this -- does that shed any light?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/hamishcunningham/fishy-wifi/issues/9#issuecomment-344641801, or mute the thread https://github.com/notifications/unsubscribe-auth/AFFUF55ECVBY-e8kZBEAqP9awd2Y_0WYks5s2wzrgaJpZM4QfJwD .

layerzerolabs commented 6 years ago

I've fixed the ultrasonic pin on the diagram, the waterelf32 sketch in fishy-wifi was already correct...

I can't see any bugs in the instructions for connecting the water temp and light sensor however..

Let's dig into it tomorrow?

Kind regards

Gareth

On 15 November 2017 at 18:09, Gareth Coleman shef.gareth@gmail.com wrote:

I know that the ultrasonic sensor needs a pin change - I'll do that in the next ten min so a git pull should resolve that one... as to the waterproof temp sensor I'll have another look and see if there is another bug there...

G

On 15 Nov 2017 16:10, "Hamish Cunningham" notifications@github.com wrote:

Hmmm, c7e3cb4 https://github.com/hamishcunningham/fishy-wifi/commit/c7e3cb415f0ee15786c1d1fe610efdc433e5f889 was intended to fix this -- does that shed any light?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/hamishcunningham/fishy-wifi/issues/9#issuecomment-344641801, or mute the thread https://github.com/notifications/unsubscribe-auth/AFFUF55ECVBY-e8kZBEAqP9awd2Y_0WYks5s2wzrgaJpZM4QfJwD .

--

“Stand for something, or you will fall for anything.” R.M.S.

Gareth Coleman

Eroc33 commented 6 years ago

I don't think this is a pin assignment issue. Ultrasonic seems to work for me, but note it needs to be plugged into USB to get 5v (and I assume this means it won't work on battery), whereas I had it plugged into the common 3v rail. Light level definitely seems to work for me. With Water temperature it's often reading -127C, but does give readings that seem correct occasionally.

layerzerolabs commented 6 years ago

Glad to hear that it's not pins! Yes, the ultrasonic sensor needs 5v but battery operation wasn't a priority for waterelf as the aquaponics systems need mains power for the pumps.

With the water temp sensor, does running a test sketch like Example > DallasTemperature > simple work more reliably?

If so, then that's an issue someone could look into....

Kind regards

Gareth

On 15 November 2017 at 18:45, Euan Rochester notifications@github.com wrote:

I don't think this is a pin assignment issue. Ultrasonic seems to work for me, but note it needs to be plugged into USB to get 5v (and I assume this means it won't work on battery), whereas I had it plugged into the common 3v rail. Light level definitely seems to work for me. With Water temperature it's often reading -127C, but does give readings that seem correct when warmed up.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/hamishcunningham/fishy-wifi/issues/9#issuecomment-344689276, or mute the thread https://github.com/notifications/unsubscribe-auth/AFFUFzm_zBImtlVFp629S4qBUtc053gRks5s2zFdgaJpZM4QfJwD .

Eroc33 commented 6 years ago

I still get readings of -127C with the basic example, so I assume it's my wiring or the sensor, not the pin assignments. Hopefully I should be able to resolve that tomorrow.

layerzerolabs commented 6 years ago

Ok thanks for the feedback, I'll close this one for now.