tx_haggis

Results 96 comments of tx_haggis

@TBAMax I may have a way to remove the virtual function calls. Is this PR still being actively worked on? @noisymime Is this PR likely to be accepted?

@TBAMax I'm working on a patch for you. Question: the new fuel scheduler adds a new check when setting the next*Compare members: there has to be 400 timer ticks remaining...

@TBAMax Teemo - I agree with this approach to the timers and the need to recode against latest master. What can I do to help? A different branching strategy could...

@TBAMax I fixed the unit tests with the PR I submitted to your main branch: https://github.com/TBAMax/speeduino/pull/11

@VitorBoss could this be closed, with pr #735 being merged recently?

Sure. I'd be interested to hear the results, especially if I need to improve/change performance testing. BTW, a couple of Ardustim log compares. Bright green is master, light green is...

Might be related to InitVar instances never comparing equal? ``` a = InitVar[str] b = InitVar[str] print(a==b) # False ```

Serendipity - I hit the same problem: PR to fix has been submitted.

I'll take a look - I don't have a Teensy handy, so it will require some guesswork. TS is sending the 'r' cmd, 0x30 (send live values), offset 0, length...

It works OK on a Mega? I wonder if the Teensy serial code has an issue: the code is reliant on `availableForWrite()`.