#emc-devel | Logs for 2009-06-03

[01:30:49] <cradek> jepler: found it!
[01:32:32] <SWPadnos> a bug?
[01:34:06] <cradek> a mesa card
[01:34:45] <SWPadnos> heh
[01:35:22] <cradek> we don't have any more bugs, they're all fixed
[01:35:45] <SWPadnos> oh, excellent
[01:36:01] <skunkworks> the servo drive card>? for jepler?? :)
[01:36:09] <SWPadnos> did someone write the manual on all the unexpected features then? :)
[05:36:40] <CIA-48> EMC: 03cmorley 07TRUNK * 10emc2/src/emc/usr_intf/pncconf/ (pncconf.py pncconf.glade):
[05:36:40] <CIA-48> EMC: Add spindle signal names to stepper signal names variables.
[05:36:40] <CIA-48> EMC: fix charge pump signal name so it works
[05:36:40] <CIA-48> EMC: fix encoder index HAL pin numbering error
[05:36:40] <CIA-48> EMC: fix servo open loop test so it works - need to fix Estop button
[05:36:41] <CIA-48> EMC: maybe make the test controlled by keyboard too.
[11:17:12] <micges> jepler: cvs server is down
[12:57:47] <micges1> micges1 is now known as micges
[13:52:10] <micges> bbl
[14:54:10] <jepler> git patch series to get rid of hal and rtapi error constants and use unix errnos instead: http://emergent.unpy.net/files/sandbox/error-constants.patch
[14:58:43] <cradek> wow, that looks like it was a huge pain
[14:58:51] <jepler> nah, not really
[14:58:59] <cradek> are you unsure about committing it?
[14:59:05] <jepler> mostly done witregular expressions
[14:59:40] <SWPadnos> could you correct the spelling on "successful"? :)
[14:59:47] <jepler> no, probably not
[14:59:50] <SWPadnos> heh
[15:01:17] <SWPadnos> it's funny. I started to look at doing the same thing, but I couldn't find the kernel errno definitions
[15:01:40] <SWPadnos> I think I looked at it when someone had an error loading a module, and I couldn't tell them what it meant
[15:02:36] <jepler> /usr/include/linux/errno.h
[15:03:01] <jepler> it looks like my proposed change mostly gets rid of the "successfull"s
[15:03:08] <jepler> that's the spelling error you mean, right?
[15:03:12] <SWPadnos> yep
[15:03:23] <SWPadnos> it's there on the HAL_SUCCESS line
[15:03:47] <SWPadnos> and RTAPI_SUCCESS
[15:03:50] <jepler> ah, see, you didn't read all the way to the end
[15:04:01] <jepler> in the last commit, all those defines are removed altogether
[15:04:06] <SWPadnos> ah
[15:04:09] <SWPadnos> here I am in CVS mode
[15:05:00] <jepler> but you're right, maybe I should add a commit before all the others to correct the spelling error
[15:05:06] <SWPadnos> heh
[15:05:14] <SWPadnos> probably not necessary :)
[15:05:27] <jepler> I may be overdoing the commit splitting, to be honest
[15:52:08] <seb_kuzminsky> hi micges
[15:57:09] <micges> hi seb
[15:57:31] <seb_kuzminsky> did you get a chance to try the new hm2 stepgen driver & firmware?
[15:57:45] <micges> nope :/
[15:58:05] <micges> still waiting for hardware to test (as usual)
[15:58:17] <seb_kuzminsky> ok
[15:58:28] <micges> I'll surely let you know
[15:58:36] <seb_kuzminsky> ok, i'll stop nagging you ;-)
[15:58:51] <micges> aeb: another thing: today I was playing pwmgen from mesa
[15:59:33] <micges> you not nagging me it's the same important to you and for me
[16:00:17] <micges> and for some values .velocity from encoder was NAN
[16:00:35] <seb_kuzminsky> uh oh
[16:01:13] <seb_kuzminsky> that's a new one...
[16:01:15] <micges> version from emc2.3 beta but encoder.c didn't change since then
[16:01:45] <micges> all velocity estimate code is in encoder.c ?
[16:01:52] <seb_kuzminsky> for the encoder, yes
[16:02:03] <seb_kuzminsky> stepgen has its own velocity feedback
[16:03:01] <micges> I have nan once for minute and I switched to ddt's and everything works ok
[16:03:42] <seb_kuzminsky> the encoder vel estimator should be much better than ddt of position
[16:04:02] <seb_kuzminsky> this is from a pre-2.3 version of encoder.c? do you know how old?
[16:04:16] <micges> I'll later try find hal from my test machine
[16:06:21] <micges> seb: last commit to encoder.c was 3 months ago
[16:06:58] <seb_kuzminsky> yes, the encoder velocity should have been good since Jan 31 2009
[16:06:59] <micges> but ok I update to 2.3.1 tomorrow and recheck
[16:07:34] <seb_kuzminsky> what was the condition when it reported NaN for velocity? going fast or slow? changing speed? anything funky going on?
[16:08:12] <micges> slow and changing speed to -slow
[16:08:12] <seb_kuzminsky> if you update to the CVS top-of-tree in the 2.3 branch you'll get the stepgen fixes too
[16:08:42] <seb_kuzminsky> ok, i'll try to reproduce it tonight when i look at the encoder.reset thing chris morley reported
[16:08:43] <micges> I noticed that, thanks
[16:11:08] <micges> seb: I've looked into source and there is very little space to generate nan
[16:13:44] <micges> seb: that was on encoder.2 and scale was set to 1000
[16:15:11] <micges> velocities from +-20 to +-200 changed by setp
[16:15:56] <micges> bbl
[16:17:00] <jepler> seb_kuzminsky: I notice that you declare locals of type 'hal_float_t'. You should probably declare them as 'double' instead. You don't want the 'volatile' qualifier, and you probably don't want the reduced precision for temporaries in the case that hal_float_t is narrower than double.
[16:21:16] <seb_kuzminsky> jepler: oh yeah, good idea
[16:21:22] <seb_kuzminsky> my night is filling up...
[16:22:00] <jepler> problem is, i'm not sure I'd accept that change in 2.3, but that makes it near-impossible to automatically apply bugfixes from TRUNK to 2.3
[16:22:05] <jepler> ugh I hate everyone
[16:22:33] <seb_kuzminsky> heh, it's near impossible anyway with CVS, so nothing lost there ;-)
[16:22:41] <jepler> but soon!
[16:22:50] <seb_kuzminsky> * seb_kuzminsky swoons
[16:22:52] <jepler> (can't you cherry-pick, or whatever it's called, in bzr and then export to cvs?)
[16:23:04] <seb_kuzminsky> yeah that's what i do currently
[16:23:12] <seb_kuzminsky> bzr unfortunately doesnt merge-track cherrypicks :-(
[16:23:17] <seb_kuzminsky> but it works ok
[16:23:57] <jepler> I don't see how you can
[16:24:05] <jepler> cherrypicks aren't merges, so how can you track them as merges?
[16:24:19] <seb_kuzminsky> they're sort of partial merges
[16:24:32] <seb_kuzminsky> doesnt git track cherrypicks?
[16:27:26] <seb_kuzminsky> the "git cherry-pick -x" option suggests not :-(
[16:27:50] <jepler> "git rev-list --left-right --cherry-pick" and "git cherry" have some way of determining that a commit "introduces the same change as another commit on the 'other side'"
[16:28:42] <seb_kuzminsky> hmm
[16:31:12] <jepler> man git-patch-id
[16:31:35] <jepler> it seems to use this thing called a "patch id"
[16:42:17] <jepler> wow, there seems to be nearly a 10x speed difference running 'git filter-branch' between these two machines (old p4 2.4ghz slower and new phenom 2.3ghz faster)
[16:42:48] <jepler> (I'm trying to make an alternate copy of the emc history without any of the bitfiles, to see how much smaller it is)
[16:44:15] <jepler> one commit every two seconds vs multiple commits per second
[16:59:31] <jepler> which is nice because I keep missing some subtletly
[16:59:40] <jepler> and there are 17kcommits to do
[18:11:49] <jepler> Rewriting emc2's history without the .bit and .BIT files trims about 10 megs off; .git/objs goes from about 38 megs to about 28 megs.
[18:12:10] <jepler> I don't think this is a big enough benefit to justify destroying history
[18:14:24] <seb_kuzminsky> peter is supposed to mail me a replacement for every .bit that has stepgens in it...
[20:09:56] <cradek> micges: I added you as a developer - welcome! If you would try getting a devel checkout now, we can make sure it works.
[20:10:23] <cradek> micges: what is your sourceforge login?
[20:12:49] <micges> the same here
[20:13:09] <micges> wow :)
[20:14:50] <alex_joni> whee ;)
[20:14:59] <alex_joni> next round of beers is on micges
[20:15:36] <micges> devel checkout is working
[20:15:56] <cradek> ok good
[20:16:05] <cradek> now I am busy assigning all the bugs on sourceforge to you
[20:16:18] <alex_joni> cradek: there's an auto-assign feature
[20:16:30] <micges> oh well... ;)
[20:16:31] <SWPadnos> hmmm. interesting
[20:16:44] <SWPadnos> my Jaunty RTAI kernel has been up for a day now
[20:16:55] <alex_joni> jaunty is so yesterday
[20:17:01] <SWPadnos> I haven't changed it, but I also haven't run any RT programs
[20:17:03] <SWPadnos> heh
[20:17:04] <seb_kuzminsky> SHIP IT!!!1
[20:17:13] <SWPadnos> yeah, I should be using Karmic Koala now, huh?
[20:17:25] <alex_joni> karmic+1 ftw
[20:17:38] <alex_joni> at least you'd be the first one for a change :P
[20:18:01] <SWPadnos> heh
[20:24:00] <cradek> micges: I also added you to the bug and feature request trackers so you can open/close/assign things to yourself
[20:24:22] <alex_joni> especially new ones :P
[20:24:34] <alex_joni> I mean the ones you report :P
[20:25:40] <micges> ok cool
[20:56:46] <jepler> git marks all commits with a full name and e-mail address. For the cvs history import to git, should I use MichaƂ Geszkiewicz <micges@wp.pl> or some other address instead?
[20:58:11] <jepler> actually, that makes me think I should post this to the developers list
[21:00:09] <micges> jepler: yes
[21:03:16] <micges> use above
[21:06:26] <jepler> got it
[21:32:38] <CIA-48> EMC: 03micges 07TRUNK * 10emc2/src/emc/usr_intf/axis/extensions/emcmodule.cc: remove unused fields in nml
[21:32:38] <CIA-48> EMC: 03micges 07TRUNK * 10emc2/src/emc/nml_intf/ (emc.cc emc_nml.hh): remove unused fields in nml
[21:33:20] <micges> wow it works :)
[21:33:35] <SWPadnos> yay! :)
[21:57:37] <alex_joni> good night all
[21:58:36] <micges> good night
[22:19:48] <micges> jepler: I see where line is and comment is wrong
[22:20:01] <micges> thanks for advice
[22:20:07] <micges> good night all
[22:20:12] <jepler> jmkasunich: I dunno if we've talked about this before, but what do you think about getting rid of hal and rtapi error numbers and using unix errnos instead?
[22:20:15] <jepler> goodnight mc
[22:20:17] <jepler> micges