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

Ramping Bug/Issue? #5

Closed
BrightZin opened this issue Nov 3, 2022 · 11 comments
Closed

Ramping Bug/Issue? #5

BrightZin opened this issue Nov 3, 2022 · 11 comments
Assignees

Comments

@BrightZin
Copy link

I flashed the latest kr4 no fet hex on 2 e17a d4v2's and I noticed there's a fluttery type of double blink happening when smooth ramping up through mid ramp. Not sure it it was intentional or not but I don't think non fet anduril is supposed to blink mid ramp.

@starryalley
Copy link
Owner

Yes I noticed this as well on my D4v2 lights but I haven't got time to figure out. Might be a bug or a configuration issue. I'll let you know when I find out why. Thanks!

@starryalley starryalley self-assigned this Nov 3, 2022
@starryalley
Copy link
Owner

Doing some quick bisecting and found the faulty commit: 1923dd27ae3be10f48dc7926dc1b647815a78071

Apparently I forgot to enable the phase sync config for KR4 (all variants). Fixed with this commit

Thanks again for brining this to my attention. I'll create a new release shortly.

@BrightZin
Copy link
Author

That was quick. Thanks for your work on this. Gonna flash some more lights once when its released. That blink was really annoying me

@starryalley
Copy link
Owner

I agree. I haven't EDC'd my D4v2 for the past few months (I've been using my SP10 pro and TS10 lately). All my D4v2's are stationary at home so that's my excuse not to have a motivation to analyse behind that annoying blink. Thanks again for this issue or I might never get it fixed.

I've personally verified on 4 of my D4v2 currently on my desk.

Just created a new release here

@BrightZin
Copy link
Author

I flashed the no fet kr4 version and the flickering during ramping is gone but now the light flickers right when I turn it on at mid ramp or higher

@starryalley
Copy link
Owner

Any chance it is the startup mode default being set to ramp up (mode 1)?
See details: https://github.com/starryalley/Anduril2#start-up-mode

Set to mode 0 for the default Anduril2 behaviour. I prefer mode 1 so that's why I set it in this commit

@BrightZin
Copy link
Author

No I just checked and I'm in mode 0. It turns on instantly but flickers the same way it used to during ramping

@starryalley
Copy link
Owner

Sure thanks for reporting. I'll see what's wrong there! SP10 pro doesn't seem to have it so I assume it's another missing part/config. I'll update when I find out.

@starryalley
Copy link
Owner

I can replicate the issue on my nofet and fet D4v2. This is clearly the phase sync issue, which results in the same flickers of the original problem like in the ramping up. But this is not the same as the previous fix. This issue actually exists before the latest release.

It is a bug in my startup mode code introduced in this commit.

Fix here.

I've attached kr4-nofet HEX. If you can give it a try and let me know. Thanks again for the report and test!

anduril.noctigon-kr4-nofet.hex.zip

@BrightZin
Copy link
Author

Looks good, no more blinking on start up. Thanks

@starryalley
Copy link
Owner

Thanks again for the bug report and testing. I've just published another release.

Closing.

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