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

Multi Instance Driver #9

Open
akd1608 opened this issue Dec 31, 2023 · 11 comments
Open

Multi Instance Driver #9

akd1608 opened this issue Dec 31, 2023 · 11 comments

Comments

@akd1608
Copy link

akd1608 commented Dec 31, 2023

Hi, I am currently trying to integrate two HX711 modules into my application. This cannot be implemented as the driver only supports one instance of the load cell.

Is an implementation of the multi-instance driver pending?

@nobodyguy
Copy link
Owner

Hi, it should just be an initialization rework to support this. I can push this change over the weekend, but more thorough testing is up to you :)

@akd1608
Copy link
Author

akd1608 commented Jan 5, 2024

Thank you, i'll try my best! :D

@akd1608
Copy link
Author

akd1608 commented May 16, 2024

Hi, where ca I find the commit for the multi instance driver?

@nobodyguy
Copy link
Owner

Hi, where ca I find the commit for the multi instance driver?

https://github.com/nobodyguy/HX711_zephyr_driver/tree/multi_instance
And rate GPIO is needed for every instance. To make it optional would require more time consuming refactoring.

@akd1608
Copy link
Author

akd1608 commented Aug 27, 2024

Hey is it possible to merge the multi instance driver into main?

@nobodyguy
Copy link
Owner

What's the result of the testing?

@akd1608
Copy link
Author

akd1608 commented Aug 27, 2024

It works fine, if test for two hx711 is enough.

@nobodyguy
Copy link
Owner

Integrating it properly would require quite a bit of work because of the optional rate control and newly added output filters.
I'll try to find a few hours in the following weeks and get it done.

@akd1608
Copy link
Author

akd1608 commented Nov 17, 2024

Hi,
have you had time to merge the multi-instances in the driver into the main branch?

@ClausOeh
Copy link

Hi @nobodyguy,

I hope you're doing well! I wanted to kindly follow up on the status of the multi-instance driver implementation. Have you had the chance to work on merging it into the main branch, or is there an updated timeline for when this task might be completed?

Thank you for your efforts, and please let me know if I can assist with testing or provide any additional feedback to help move things along.

Best regards,
Claus Oehler

@ClausOeh
Copy link

ClausOeh commented Dec 2, 2024

Hello @nobodyguy, any updates here? Thanks

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

No branches or pull requests

3 participants