-
-
Notifications
You must be signed in to change notification settings - Fork 19.2k
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
[FR] Multiple INA226 support #23206
Comments
I have working code for this, but need comments before formatting a PR. why are we pulling in Wire and INA226 for 5 i2c reads in ammeter.cpp? that locks us to SDA/SCL. i propose to pull SlowSoftI2CMaster instead, to allow remapping pins, as the MKS GEN L v1 i'm tinkering on for this laser doesn't expose SDA/SCL. I feel it would probably be more useful that way? rewriting the existing INA226 code to use the same is fairly trivial, but i would like some feedback before cooking something up. |
Thank you both, @toxicpsion and @thisiskeithb . Today I was rummaging around my profile in GIThub and noticed this FR tied to my profile. -Hmm whats this? I look forward to seeing what you come up with and you incorporate! -My best hope was to at least get an option to output to serial where even if the display and software didn't use the data, I might be able to monitor manually and maybe even write a script to also follow serial and log the data with time stamp. Now if nifty toys are added to the firmware, even better! Note: I don't just invent the wheel I make it into turbine thrusters. Is marlin ready for this much data? lol |
Is your feature request related to a problem? Please describe.
Feature Request: Inspired by the addition of the I2C amperage checking function via a singular INA226. I would like to ask for the ability to monitor one chip or channel per Axis. INA226 have one per chip. INA3221 have 3 per chip.
Are you looking for hardware support?
INA226
INA3221
Describe the feature you want
I would like to be able to use the INA226 or INA3221 to monitor the voltages and amperage of multiple axis.
In particular this may be useful in monitoring CNC axis load. Normally to do so a shunt or clamp must be placed inline. An extra monitor installed at the CNC router motor 120V or Spindle DC line to monitor router load. Monitoring load can improve the quality of cut parts.
I am certain the function was introduced to aid in monitoring current to the laser drivers. But I think it can be used for much more given that a clever person can derive a lot of data from these chips.
To be clear I am not requesting any changes to interface. Only serial output, maybe a clever way to view the data as a sub menu of the dot matrix type screens.
Additional context
I do believe the request, at least right now is of very low priority. I do not expect to see this request implemented. But I am hopeful that some one might see the value and work in a way to implement some expanded functionality for multiple I2C voltmeter/Ammeter channels.
The text was updated successfully, but these errors were encountered: