Closed aaeegg closed 3 weeks ago
@brendandburns I believe that PR #100 represents the most complete and accurate DTC table we have at this time. Do you have any objection to this being merged instead of PR #99?
@fenugrec I'm sorry to put you in the uncomfortable position of reviewing two "competing" PRs, but as we haven't heard from @brendandburns, would you take a look at #99 and #100?
Well commits are cheap so I could start with this one which is closer to what I want to see. The python script is certainly more elegant but IIRC it needed a bit of work. It can certainly be refined and merged later; even if it only does 90% of the job requiring a few extra sed lines, that's still an improvement vs hand-editing the whole thing. My priority here is "how will someone pull updated DTC lists from external sources, and update freediag's lists with high confidence and minimal headache".
For this PR I think it would be a good idea to preserve the steps you posted as a shell script, probably in scantool_850/ ? Even if not really usable as-is, it'll be more useful than hidden away in a github comment that may someday be hard to find.
@fenugrec I've added the script as scantool_850/makedtc.sh.
Merged. Thanks for your patience. Will leave '99 open since a "proper" script would be a nice eventual upgrade.
This adds a DTC table based on those recently provided by Richard H. Jones and Aleksi Venäläinen. The new table is a merge of the two "upstream" tables. Tips have been split off from DTC descriptions, and there is a new option to the 850 dtc command to display tips.
I processed the incoming tables as shown in the shell session below. This is less readable than @brendandburns's Python script, but it results in valid output, which the Python script doesn't. I then hand edited the result to merge and correct the lists. I placed the J1979 DTC cross-references in the tips; they seem superfluous to me, but Volvo's first-party scan tool displays such, so I left them in. I removed useless tips like "see OTP 850 DVD" - of course there's information on the OTP 850 DVD - there's information on every DTC on the OTP 850 DVD. But I left in specific references like "see OTP 850 DVD, TP-2308202, p669 and p78 and p316".