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

Question Wiegand Reader #30

Closed
SamNitro opened this issue Dec 10, 2017 · 22 comments
Closed

Question Wiegand Reader #30

SamNitro opened this issue Dec 10, 2017 · 22 comments

Comments

@SamNitro
Copy link

Hey,
Is it possible to include the wiegand readers?

@omersiar
Copy link
Collaborator

It is indeed. Maybe @nardev has some taughts about it.

@nardev
Copy link
Collaborator

nardev commented Dec 13, 2017

If @omersiar wants to merge it, i'll make it as new commit, but in that case, @omersiar should change the project description and plan?

@omersiar would you do that?

At the moment, it's not 100% stable, just changed the current code to work with wiegand readers and that works great.

@omersiar
Copy link
Collaborator

Project itself getting more complex with each feature that is supported, each feature should can be enabled or disabled at compile time, but that needs more robust build system. I have a plan to change WebUI structure to support plugins like MQTT, Wiegand, etc.

@nardev I can merge it on another branch, so people can test it more widely, so we can fix bugs if there is any then it can be merged to master branch. I will make changes at description and change log.

@nardev
Copy link
Collaborator

nardev commented Dec 14, 2017

@omersiar yea.. that is good.

Regarding UI.. i believe it can be much more simple...

@omersiar have you thought about moving to platformio structure?

@SamNitro
Copy link
Author

That looks like good, thanks for the great work.

@amp83
Copy link

amp83 commented Dec 17, 2017

2 questions is there a way to get esp to load settings if MFRC522 is not connected. (Haven’t arrived yet)
2nd any way of using ibuttons (onewire devices) instead of MFRC522

@nardev
Copy link
Collaborator

nardev commented Jan 12, 2018

@omersiar i did some code, just tell me what is your final intention:

  1. Only wiegand based branch
  2. Make it optional in settings, to choose a reader and reader pins

I can make both things.. currently i have installed few wiegand based readers, works fine but i use interrupts

Is there any particular reason not to use interrupts for you?

@omersiar
Copy link
Collaborator

Let's do 2, because the other branches won't get any updates unless someone manages it. The reason we did not use interrupt on MFRC522 is because of complexity. I never managed it working.

https://github.com/miguelbalboa/rfid/blob/master/examples/MinimalInterrupt/MinimalInterrupt.ino
miguelbalboa/rfid#311

@akeilox
Copy link

akeilox commented Jan 13, 2018

its great to hear Wiegand support in the horizon, there is so much potential in converting old oem readers into modern mqtt

@SamNitro
Copy link
Author

@nardev I have read you are testing the Wiegand Reader... if you have a compiled bin I can test it too :)

@nardev
Copy link
Collaborator

nardev commented Jan 15, 2018

@SamNitro you can test it now: https://github.com/nardev/esp-rfid

To me it worked for both readers.

@SamNitro
Copy link
Author

Hey, I can't save the Wiegand settings. After Reboot it is unchecked.

@nardev
Copy link
Collaborator

nardev commented Jan 16, 2018

@SamNitro i'll test it, just a sec...

@nardev
Copy link
Collaborator

nardev commented Jan 16, 2018

@SamNitro oh, it saves the settings it was just when i did clean up.. i checked if the readerType is 0 istead of "0"... before i used == and there was a warning in commit..

feel free to continue... it actually save it.. i'll pus the change now...

@SamNitro
Copy link
Author

Thanks :)

@nardev
Copy link
Collaborator

nardev commented Jan 16, 2018

@SamNitro please tell me if it works as a reader.. is everything fine?

@nardev nardev closed this as completed Jan 16, 2018
@SamNitro
Copy link
Author

SamNitro commented Jan 16, 2018

No I'm sorry, he lose the GPIO information. He set D-0 and D-1 back to GPIO-5
I will use
D-0 -> GPIO 14
D-1 -> GPIO 12
on Wemos D1 mini

@nardev
Copy link
Collaborator

nardev commented Jan 16, 2018

@SamNitro
ah.. it's just predefined in form... sorry... i'll fix that too

once you choose it and set it.. just "save/reboot" and it should work.. despite what it shows in settings.

@nardev
Copy link
Collaborator

nardev commented Jan 16, 2018

@SamNitro thank you for testing... it works now...i just pushed.. there was slight mistake in HTML form

@SamNitro
Copy link
Author

Perfect, it works :)
Thx for support and work.

@nardev
Copy link
Collaborator

nardev commented Jan 16, 2018

@SamNitro glad to hear that....

@omersiar sorry.. as the functions pile up... more testing is required... :) probably once we simplify the frontent, it will be less mistakes of this kind...

@JuculianD
Copy link

I would like the wiegand reader function as well. I got an 125khz wiegand reader pcb with antenna from aliexpress and I would like to use it. However, the prebuilt builds do all not contain the wiegand settings on the settings page. I will wait for the new frontend.

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

No branches or pull requests

6 participants