Skip to content
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

Added wiring diagrams, datasheet and comments on code #18

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

pcgayan
Copy link

@pcgayan pcgayan commented May 25, 2017

Added Datasheet of PZEM004T
Added Wiring diagram
Added comments for RX, TX on Arduino

@mandulaj
Copy link

Hey, I am maintaining the PZEM004T-v30 library https://github.com/mandulaj/PZEM-004T-v30 which is very similar in terms of the hardware. I have many users being confused about the wiring. Would you mind adding your drawing also there so that the confusion can be resolved for once. Thank you

@pcgayan
Copy link
Author

pcgayan commented Jan 15, 2022 via email

@mandulaj
Copy link

mandulaj commented Jan 15, 2022

Yeah so would you like to add it to my library instead. I am not going to ignore the pull request 😉

@olehs
Copy link
Owner

olehs commented Jan 15, 2022

Hi. I'm sorry, I completely forgot about this request.

The reason I didn't accept it at the first place was that it not only adds datasheets as stated in the title, but also modifies example's workflow.
These changes should be split to different PR's.

Also I doubt I can accept example's changes without further discussion. With these changes, when AC power is off, there will be last positive reading in the freezed log. This can be confusing.

@vortigont
Copy link
Contributor

Hello @olehs, @mandulaj,
all lib devs in one place, huh :) Just wanted you to suggest adding a links to docs and pages in different repo's. Would be much easier for users to search for connection diagrams and proper libs. I.e. Oleh has a very nice WiKi pages describing hardware setup with esp8266. I've made a 3-phase setup diagram not so long ago. Also I found a really good repo with PZEM's reverse-engineered schematics.
This is all scattered pieces of information, would be nice to have references one to another.
Cheers!

@mandulaj
Copy link

When I originally wrote my version of the library this was my goal. Just haven't had the time to sit down and collect all the pieces. If you have time and motivation then you are welcome to take the initiative 😇

@vortigont
Copy link
Contributor

Time is the most precious resource for all of us :) OK, I may come up with a list of "useful links" next time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants