Re: Library update for OBD-II Adapter- DFRobot Forum

2013-10-04 13:14:57 Stanley,

I just pulled your latest update to megalogger.h. It now compiles. I see from the change log that you did other modifications, but I have not had time to test for improvements as it relates to the code being used with Kit#3.

Maybe later tonight or tomorrow I can provide that feedback.

Sb. sbirdranch

2013-10-04 12:54:37 Stanley,

I believe Mattt and I are both using your kits. I'm using your Kit#3.

I have all your current code pulled down from GitHub using TortiseGit. Your code is currently not what you think it is.

I suggest you review what you are pushing to the master branch and we can go from there.

Also...you code seems to have the display turned 180 degrees. That would suck for someone who has made a mount or case for the hardware.

Mattt,
When I get back from vacation I will be branching all of Stanley's code and starting a new development thread. I think with a little help from a phd friend, I can get it running. I can at least git something working with time. I had it modified so it would at least fire up if the OBD connector wasn't attached.
sbirdranch

2013-10-04 09:15:10 I found the compiling issue was to do with saving the files from GitHub by looking at them in raw and saving the file in chrome saves it to a file with HTML code inserted into the code. There is no easy way to save the files without copying and pasting each file.

The mega logger.ino in the mega logger archive on sourceforge does not compile still. mattt

2013-10-02 18:39:37 I have had the same problems since day one. I didn't realize that this is an older thread so there is no recent new update. Right now I have given up on this project as I don't have the programming knowledge to fix all of the bugs with this OBD adapter. There is also the issue of having to unplug and plug the OBD adapter from the UART in order for the screen to work in the vehicle. mattt

2013-10-02 15:33:24 Sorry....I meant 'serial' not 'spi'.

I have tried it and although it compiles and such, the stability is still terrible. Hitting reset is like rolling the dice (hmmm second dice reference). You may or may not get the acceloramete. You may or may not get the gps, even when it is locked. I got it to recognize the OBD connection to my truck (2012 Chevrolet) but never got it to actively retrieve data. I would get one RPM reading and that was it.

If there is a difference OBD library, it doesn't seem to be on GitHub.

Preston. sbirdranch

2013-10-02 15:10:51 Matt / Stanley

The problem is that the class COBDI2C does not exist in the libraries on GitHub. I have pulled down the zip file from SourceForge and the entire repo from GitHub. No dice.

You can make it compile if you change the class back to COBD like it was in the previous version, but I don't know what other issues that might reveal.

I'm going to try it, as the OBD connector from Kit#3 is SPI not I2C. sbirdranch

2013-10-02 14:43:50 I just did the scorched earth approach and loaded everything from scratch.

I get the same error as you.

I'll try updating the individual file that Stanley suggested and see what I get.

Sb. sbirdranch

2013-10-01 18:44:55 I downloaded the updated archive from [url=http://sourceforge.net/projects/obdcon/files/logger/megalogger-src-20130930.zip/download]http://sourceforge.net/projects/obdcon/files/logger/megalogger-src-20130930.zip/download[/url] I have overwritten the libraries from the archive. Still the same issue. I wonder if anyone else has the same problem? mattt

2013-09-30 21:02:51 Hi, just tried to upload megalogger.ino with updated OBD library. I get the following errors:

[code]megalogger:88: error: expected class-name before ',' token
megalogger.ino: In member function 'void COBDLogger::setup()':
megalogger:113: error: could not convert 'init()' to 'bool'
megalogger:113: error: in argument to unary !
megalogger:125: error: 'readSensor' was not declared in this scope
megalogger.ino: In member function 'void COBDLogger::loop()':
megalogger:175: error: 'isValidPID' was not declared in this scope
megalogger:181: error: 'isValidPID' was not declared in this scope
megalogger:191: error: 'isValidPID' was not declared in this scope
megalogger:216: error: 'errors' was not declared in this scope
megalogger.ino: In member function 'void COBDLogger::dataIdleLoop()':
megalogger:288: error: 'getState' was not declared in this scope
megalogger.ino: In member function 'void COBDLogger::logOBDData(byte)':
megalogger:427: error: 'readSensor' was not declared in this scope
megalogger.ino: In member function 'void COBDLogger::showECUCap()':
megalogger:467: error: 'isValidPID' was not declared in this scope
megalogger.ino: In member function 'void COBDLogger::reconnect()':
megalogger:481: error: could not convert 'init()' to 'bool'
megalogger:481: error: in argument to unary !
megalogger:485: error: 'write' was not declared in this scope
megalogger.ino: In function 'void setup()':
megalogger:640: error: 'class COBDLogger' has no member named 'begin'[/code] mattt

2013-09-19 09:14:40 Has this updated library been included in the bundle you reference at : [url=http://sourceforge.net/projects/obdcon/files/logger/]http://sourceforge.net/projects/obdcon/files/logger/[/url] ?? sbirdranch