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

acc buggy on redmi note 6 pro(tulip) #113

Closed
kevin01523 opened this issue Aug 31, 2021 · 5 comments
Closed

acc buggy on redmi note 6 pro(tulip) #113

kevin01523 opened this issue Aug 31, 2021 · 5 comments

Comments

@kevin01523
Copy link

kevin01523 commented Aug 31, 2021

acc seems too buggy for my device on the latest update
acc -wizard is bugged
acc doesnt show current
acc seems to control current fine via checking on ru.andr7e.deviceinfohw.pro app on its battery tab

also kinda breaks acca the current version and its previous release on github

Screenshot_20210831-200949
Screenshot_20210831-200958

acc-logs-tulip.tar.bz2.zip

@VR-25
Copy link
Owner

VR-25 commented Aug 31, 2021

What's wrong with the wizard?

ACC is not showing the correct current value because your device is reporting two - yes, two current_now values - negative and positive.
This breaks the logic.
I'll have to rewrite part of it.

The AccA issue is a bit weird.
It does not seem to be related to acc.
Otherwise, the app would not be able to read the config at all.
Below "Custom Settings", the config is correct - meaning, acc is ok.
This is under investigation.
I haven't tested the previous AccA version with the latest acc.

@kevin01523
Copy link
Author

kevin01523 commented Aug 31, 2021

oh the config seems to have disappeared from /data/media/0/documents folder i think that causes the config issue cannot be read by acca
it now defaults into /data/adb/vr25 i didn't reset the config before upgrade but it seemed to have been moved there

ah what i meant is acc -wizard command on termux tsu seems to bug out or its just a wrong command that kinda bugs acc command interface with flashing values

@VR-25
Copy link
Owner

VR-25 commented Aug 31, 2021

The config is not the issue.
The path changed a while ago for security reasons.

Another user just reported that the latest acc is working with the previous AccA.
The only issue is "CURRENT_NOW=0 mA".

acc -wizard is an invalid command.
What you want is just acc (without the -wizard part).

@kevin01523
Copy link
Author

kevin01523 commented Aug 31, 2021

oh i guess acca is having issues reading the config from its default location now it was ok on its previous version oh well tnx for the info

oh now i get what made me think thatlatest acc broke acca
this is the steps i made during upgrade

  • i upgraded acc from magisk

  • restarted phone

  • opens acca sees 0 current(didn't check the edit config)
    i still assume at this point that config is still in document

-updates acca to latest release from few hours ago

opens acca sees still 0 and now opens edit config and seems it shows all 0 although i think it can read the default current but still shows 0 unless you click the edit button fro current and voltage

so the issue is more on the acca app and the only issue for acc is 0 current

just clarifying the issues ive faced so no misunderstandings etc

@VR-25 VR-25 closed this as completed Aug 31, 2021
@kevin01523
Copy link
Author

thanks it works now ill test for more bugs

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

2 participants