You have almost talked me into it.

FreeEMS topics that aren't specific to hardware development or firmware development.
User avatar
Fred
Moderator
Posts: 15431
Joined: Tue Jan 15, 2008 2:31 pm
Location: Home sweet home!
Contact:

Re: You have almost talked me into it.

Post by Fred »

Cool :-)

I like the checksum and strcpy changes, but I feel the comms one is best left verbose. I reckon it's easy to miss that ++ on the end, but impossible to miss the whole extra line. Besides, it compiles to the same binary, so nothing saved ;-) Lots of stuff is verbose over efficient by choice and design.

How would you feel about starting on a missing tooth decoder? That is something we could really do with that doesn't have an owner yet.

You know the old mantra :

Make it work
Make it right
Then make it fast ;-)

Fred.

PS, pretty graph : http://github.com/johnsop1/freeems-vanilla/network
DIYEFI.org - where Open Source means Open Source, and Free means Freedom
FreeEMS.org - the open source engine management system
FreeEMS dev diary and its comments thread and my turbo truck!
n00bs, do NOT PM or email tech questions! Use the forum!
The ever growing list of FreeEMS success stories!
johnsop1
DIP8 - Involved
Posts: 26
Joined: Tue Jan 13, 2009 1:56 am

Re: You have almost talked me into it.

Post by johnsop1 »

How would you feel about starting on a missing tooth decoder? That is something we could really do with that doesn't have an owner yet.
Thats my plan. The first step (I started last night) is writing a program that wiggles a bit on my PC LPT port to simulate missing tooth wheels at various rpms.
User avatar
AbeFM
Post Whore!
Posts: 629
Joined: Sat Feb 16, 2008 12:11 am
Location: Sunny San Diego
Contact:

Re: You have almost talked me into it.

Post by AbeFM »

Oh, god, the guilt!

I need to put in that regulator I found, and get Git set up. Certainly the paste it in the web front end works, but us useless folks like GUI's...
User avatar
Fred
Moderator
Posts: 15431
Joined: Tue Jan 15, 2008 2:31 pm
Location: Home sweet home!
Contact:

Re: You have almost talked me into it.

Post by Fred »

Good to see some cross team motivation happening there! Thanks Phil! ;-)

Fred.

EDIT Phil, just hit me up if you want any major structural changes to facilitate the decoder. I am planning to write a scheduler very soon to handle the event information parsing (not sure if it will be good or not) but stuff like init etc will need some attention. The init that is currently default should be fine for you though I guess. Probably best to wire your setup as per Seans new setup/idea. I'll mod mine to conform soon enough.
DIYEFI.org - where Open Source means Open Source, and Free means Freedom
FreeEMS.org - the open source engine management system
FreeEMS dev diary and its comments thread and my turbo truck!
n00bs, do NOT PM or email tech questions! Use the forum!
The ever growing list of FreeEMS success stories!
User avatar
Fred
Moderator
Posts: 15431
Joined: Tue Jan 15, 2008 2:31 pm
Location: Home sweet home!
Contact:

Re: You have almost talked me into it.

Post by Fred »

Phil, I don't think you have your global .gitconfig setup correctly (or at all) as your commits show up as philip rather than something more unique. I just noticed the same about Sean so I've told him to sort his out too :-)

Fred.
DIYEFI.org - where Open Source means Open Source, and Free means Freedom
FreeEMS.org - the open source engine management system
FreeEMS dev diary and its comments thread and my turbo truck!
n00bs, do NOT PM or email tech questions! Use the forum!
The ever growing list of FreeEMS success stories!
johnsop1
DIP8 - Involved
Posts: 26
Joined: Tue Jan 13, 2009 1:56 am

Re: You have almost talked me into it.

Post by johnsop1 »

I wrote some very rudimentary code that seems to be able to detect missing teeth. It is supposed to be very cold here this week so I should have some time for coding. With luck I will have something worth committing to git near the end of the week.

I will also take a look at .gitconfig before I do the next commit.
User avatar
Fred
Moderator
Posts: 15431
Joined: Tue Jan 15, 2008 2:31 pm
Location: Home sweet home!
Contact:

Re: You have almost talked me into it.

Post by Fred »

Awesome news Phil :-)

One thing though, make lots of commits! With git you don't have to worry about committing broken stuff because you only share with the world when you are ready and it works. The nice thing is that you can then keep a local commit history for the purposes of roll backs during development etc.

Lot's of small changes make merging easier too, but that shouldn't be an issue as you will be pretty much confined to one file doing that.

The small commits also give you a sort of incremental creation path too, rather than x file just appeared working all of a sudden :-)

Fred.
DIYEFI.org - where Open Source means Open Source, and Free means Freedom
FreeEMS.org - the open source engine management system
FreeEMS dev diary and its comments thread and my turbo truck!
n00bs, do NOT PM or email tech questions! Use the forum!
The ever growing list of FreeEMS success stories!
User avatar
Fred
Moderator
Posts: 15431
Joined: Tue Jan 15, 2008 2:31 pm
Location: Home sweet home!
Contact:

Re: You have almost talked me into it.

Post by Fred »

For example, right now you have a rudimentary working version which you should probably record so that if your next round of improvements don't work out you have something to go back to :-)

Fred.
DIYEFI.org - where Open Source means Open Source, and Free means Freedom
FreeEMS.org - the open source engine management system
FreeEMS dev diary and its comments thread and my turbo truck!
n00bs, do NOT PM or email tech questions! Use the forum!
The ever growing list of FreeEMS success stories!
User avatar
Fred
Moderator
Posts: 15431
Joined: Tue Jan 15, 2008 2:31 pm
Location: Home sweet home!
Contact:

Re: You have almost talked me into it.

Post by Fred »

http://www.diyefi.org/forum/viewtopic.php?f=8&t=577

New thread for the decoder and discussion about it :-)

Fred.
DIYEFI.org - where Open Source means Open Source, and Free means Freedom
FreeEMS.org - the open source engine management system
FreeEMS dev diary and its comments thread and my turbo truck!
n00bs, do NOT PM or email tech questions! Use the forum!
The ever growing list of FreeEMS success stories!
Post Reply