David Novakovic

Results 85 comments of David Novakovic

Ah great thanks, now I'm starting to join the dots. On Jan 3, 2017 10:19 PM, "Paulo Pires" wrote: > More info https://en.wikipedia.org/wiki/OBD-II_PIDs#Mode_01. Look for > *RPM*. > > —...

Weird. This was the data I was getting before: `BUSINIT:OK84F110410C0C04E283F1187F01` I added something to remove the ":OK" - but now I'm not being send the OK anymore. Strange. I'm being...

``` 01-07 15:34:17.527 32248-32248/dpn.carby.carby I/obd1: BUSINIT:OK84F110410C0C1CFA83F1187F01 01-07 15:34:17.528 32248-32248/dpn.carby.carby I/obd2: 84F110410C0C1CFA83F1187F01 01-07 15:34:17.528 32248-32248/dpn.carby.carby D/btutil: RPM: 1040 01-07 15:34:17.722 32248-32248/dpn.carby.carby I/obd1: 84F110410C0C230183F1187F01 01-07 15:34:17.723 32248-32248/dpn.carby.carby I/obd2: 84F110410C0C230183F1187F01 01-07 15:34:17.723 32248-32248/dpn.carby.carby...

``` --- a/src/main/java/com/github/pires/obd/commands/engine/RPMCommand.java +++ b/Users/dpn/AndroidStudioProjects/Carby/app/src/main/java/com/github/pires/obd/commands/engine/RPMCommand.java @@ -12,6 +12,8 @@ */ package com.github.pires.obd.commands.engine; +import android.util.Log; + import com.github.pires.obd.commands.ObdCommand; import com.github.pires.obd.enums.AvailableCommandNames; @@ -43,6 +45,7 @@ public class RPMCommand extends ObdCommand { @Override...

I think my approach will be to create a patch that allows setting a byte offset either statically on the parent ObdCommand or allow it to be passed into the...

I fond that stripping the preceding byte pattern fixes the issue mostly (still figuring out some things like VIN etc though) I'm happy to contribute back to the lib, I'll...

FWIW it'd be nice to have a reasonably sane way to work around this issue: https://twistedmatrix.com/trac/ticket/7552

You're completely correct. I'm reasonably "safe" in my case - not that it excuses it. I'm mostly just wanting to be able to dev :)

also here - xmonad+picom on xubuntu btw the same error is mentioned on #34

`CORS(app, automatic_options=True)` - I did this to get things working. Doesn't fix the real issue here, but at least is a work around.