SWPadnos_ is now known as SWPadnos
i've pushed some cleanups in load_handlers code
psha: buon giorno - ah, great
check that it's still working :)
why you need inspect.ismethod?
is callable not enought?
i seem that it's lost in transition :)
ah, not lost
that' snot good enough to discern method from function - will send example
what is method? it's function bound to class instance
if you have class instance in your hands then getattr(obj, method) will return bound function
callable will give you a class object as well
i know difference between methods and functions :)
that's good point :)
but do you really need class named as signal handler? :))
dont think so
so no problems in class names
also that only applies to nested classes
so i think that inspect.ismethod is excessive check (won't change anything in current code)
it's clearly anally retentive
also i've added 'dbg' functions to simplify debugging
also do you really need adjustable 'window1' name?
whoa! example still works post-merge ;-)
no, I just hate replicating strings
we were fixing the debug flag in parallel ;-)
merge one small patch
i've moved hdl_functions into load_handlers
no need to pollute module level namespace
all your changes are in
I started working on share/gladevcp/examples
repo is current
[13:53:00] <Jymmm> http://www.rockstarprogrammer.org/post/2008/apr/06/differences-between-mercurial-and-git/
cradek_ is now known as cradek
well this is a crazy idea: http://repo.or.cz/h/mob.html
jepler: you know you like the idea =)
I like it
I kinda do too
fwiw this seems to be the implementation of the 'mob' branch concept: http://repo.or.cz/w/girocco.git/blob/768398d585b185d1ca5fae33d11d97fffcf00c87:/hooks/update
jthornton: are you 'BigJonhT'@forum?
psha: yes he is
he's forum admin?
good morning )
psha: yes I am
jepler: i've replaced idle_add() with sleep with timeout and it's working fine
jepler: may you take a look on this patch
[16:58:11] <psha> http://psha.org.ru/cgit/psha/emc2.git/commit/?h=gremlin-fix&id=b5eb1f16181d7b673cc62f3efa34511367837f55
it's all needed to add it as gladevcp widget
and it's parent - which is simple splitting patch
Are the .pdf versions of the documentation equal to the html version, but simply apportioned into "manuals" and delivered in PDF?
pretty much. AFAIK
I don't know if the manpages are in any PDF though
mhaberler: any progress on examples/docs?
(they are available in the HTML docs
psha: some - http://git.mah.priv.at/gitweb/emc2-dev.git/tree/bf36f5a48e85d24c4cadf07b82b3602f32dd9d44:/share/gladevcp/examples
working on the mdi example now - not in there yet
SWPadnos: Thanks! I'm looking at the build system now to see what I can tell from that.
psha: what do you mean, they work for you or what ;-)
few things are still broken ingladevcp/util (former inifile), so patience leaze
they looks fine :)
i'm picking features for merge
so let stage inifile for next merge
and now userfuncs will go in
your merge, I assume... too early for a master merge request - hold on for an hour with inifile which is now named util(!)
i think it's better to stage inifile for next merge
while userfuncs are pretty stable now i think that inifile will introduce too much noise...
fair enough, gladevcp is good to go, yes
btw maybe to move gladevcp-test to share?
psha: I'm having a look at the patch..
psha: I'll start by saying that I think that's cool
psha: and then pick some nits
psha: as the world blew up, you could hear an engineer say "this will never happen" ;-)
psha: I'd rather not have poll_file_change in the base class. AXIS has never automatically reloaded the preview, and if I ever get around to writing an axis-like gui in gtk I'll want the same
psha: is there a derived class of gremlin that gladevcp could use to add that?
which implement hiding DRO strings for example
[17:20:44] <psha> http://psha.org.ru/cgit/psha/emc2.git/commit/?h=gladevcp-gremlin&id=939d2c996bf090ceb9c71ce3eb8aa6339b2295a6
it's in separate branch
jepler: you prefer rewriting that patch or introducing new? :)
Note: On the web site's documentation page, the reference to "Component and command reference (pdf)" points to http://linuxcnc.org/docs/2.2/EMC2_Manual_Pages.pdf.
It should point to http://www.linuxcnc.org/docs/EMC2_Manual_Pages.pdf.
psha: I'd prefer you rewrite the patch to take out that part
I am not doubting this change, but can you explain to me in what way .queue_draw() will behave differently than .expose()?
queue draw is not rewriting it immidiate
i think :)
i'll check it once again for sure
[17:37:32] <psha> http://www.pygtk.org/docs/pygtk/class-gtkwidget.html#method-gtkwidget--queue-draw
queue_draw invalidates widget area and not redrawing
ok, thanks -- that explains it
it was my guess but I wasn't sure
The advantage of adding to the invalidated region compared to simply drawing immediately is efficiency; using an invalid region ensures that you only have to redraw one time.
in the code you'll move to gladevcp for automatic loading, you may also want to check for modifications to the file (the st_mtime, for instance)
yes, would be nice to handle file update
i'm testing patch with removed poll and then pushing it
besides that patch is ok?
yes, I think so
i think it does not make sense to merge this branch since i think i'll request merge of gladevcp branch today
I think there will be some 'make install' and debian/emc2.files.in changes necessary if we want the runnable gremlin program to get all the way into the packages..
but I'm not sure whether the gremlin program is useful enough to bother
there is usr/bin/* in emc2.files.in
that's fine then
my memory is obviously faulty. But now I am enlightened and remember commit af705da
it was verbose :)
so gremlin is now stable for you, even when you have multiple of them in a panel? that is really cool.
[17:51:50] <psha> http://psha.org.ru/cgit/psha/emc2.git/commit/?h=gremlin-fix2&id=14ab3e5668dc92661d6eaf4043985243021c7fe2
this how it's looks like now
patch is pretty minimal
you can do what you need to in the derived class without the load/_load split?
just wrap call in with_context
also i may wrap load call in derived class
may you look into gladevcp-cleanup branch?
yes I can..
just simple sanity checks
that it starts
there are no 'new' userfuncs still
it's cleanup branch to base all subsequent work on it
good, will pull and come back shortly, but I dont see problems yet
i'm on feature merging stage so you'll need just to wait a bit
ok, then go ahead, looks good to me
btw do you need several debug levels?
its really just verbosity
so can be useful at times
so let leave bool for a while until some tasks will raise
well, I have a few compares for debug > const, so..
heh, you'd better write less nested code :))
my old pascal habit
universities really can ruin style;-)
heh, habits when moving from one imperetive language to another are not that bad when jumping on something pure functional :)
it takes a year for me to start writing real code in haskell when i've first found it
haskell.. duh. I heard it exists..
pretty nice language :)
i'm still using it for prototyping
for some large stuff
first writing sketch in haskell then translate it to something else
i've tried to use erlang but without any success :(
I'll have a look at it .. after this is done
mhaberler: update branch
i've merged last handlers
got my first merge conflict :-/
i've reverted gladevcp modularization
dont worry, have to learn that anyway
i usually abandod feature branches after merge
for example i have 3 gremlin-fix and 2 gladevcp-gremlin branches :)
needed for rewriting patches
then merge and drop
surely for my local project i don't bother with this :)
nothing bad in tons of oneliners
but for public project i'd better do more rewriting and show only clean patches :)
ok, got it resolved
I'll push so you see where I am, but its WIP on probe example; several changes on util.py
fast SSD on sale at NewEgg today (if they have any left)
fast + (relatively) expensive though, $250/128GB
util won't land in this merge so feel free to do what you need :)
I thought about putting a SSD in my D510MO box but don't you have to have a hard drive too for read write data?
JT-Hardinge: recent versions of ssd are pretty resistant to rw
as i recall they try to use whole area for writes - not exactly same block as HDD drives do
you need to enable TRIM support for the highest longevity
that lets the OS tell the drive that an area is no longer in active use
(which lets the drive use that area for wear leveling)
jepler: if you have some time may i request gladevcp merge?
today merge consists of improved userfuncs and Gremlin (also tons of cleanup in gladevcp and small fix for HAL Bar widgets)
psha: If I don't get to it today, please ask someone else -- I'm about to go on a trip for a week.
if you'll find time -- it's 'gladevcp' branch in my repo
if not - i'll ask someone else tomorrow
EMC: 03jepler 07master * rddca18e0e31e 10/ (6 files in 4 dirs): Merge branch 'gremlin-fix' into gladevcp-gremlin3
EMC: 03jepler 07master * rdc967ecc9011 10/lib/python/gladevcp/ (hal_gremlin.py hal_python.xml hal_pythonplugin.py): gladevcp: Add Gremlin widget
EMC: 03jepler 07master * r2c830d5ca166 10/lib/python/gladevcp/widget-gladevcp-hal_gremlin.png: gladevcp: Added icon for Gremlin widget
EMC: 03jepler 07master * r2da021370a21 10/src/hal/user_comps/gladevcp.py: gladevcp: Switch from getopt to optarse
EMC: 03jepler 07master * r748c935c2dd9 10/lib/python/gladevcp/hal_bar.py: gladevcp: Added template string to bar
EMC: 03jepler 07master * r5274f152c2db 10/lib/python/gladevcp/led.py: gladevcp: Backport HAL Led to work on Hardy
EMC: 03jepler 07master * re555a8747180 10/src/hal/user_comps/gladevcp.py: gladevcp: Remove global variables and buildertype
EMC: 03jepler 07master * r2f686f50d74e 10/ (8 files in 6 dirs): Merge remote branch 'origin/master' into gladevcp
EMC: 03jepler 07master * r5a130eebdeb9 10/src/ (4 files in 2 dirs): gremlin: Add make rules for gremlin
EMC: 03jepler 07master * ra0fe3c3f815f 10/src/emc/usr_intf/gremlin/gremlin.py: gremlin: Remove sleeps from idle functions and...
EMC: 03jepler 07master * rb44dfb136531 10/src/hal/user_comps/gladevcp.py: gladevcp: Add get_handlers function in userfuncs
EMC: 03jepler 07master * ra12d55803915 10/src/hal/user_comps/gladevcp.py: Merge branches 'gladevcp-gremlin' and 'gladevcp-cleanup' into gladevcp
EMC: 03jepler 07master * r6aac6a97f2e8 10/ (11 files in 4 dirs): Merge remote branch 'psha/gladevcp'
psha, mhaberler: thanks, merged.
uh-oh, buildbot doesn't like you guys
what's the trouble?
found it: http://emc2-buildbot.colorado.edu/buildbot/builders/hardy-i386-trunkish-sim/builds/86/steps/compile/logs/stdio
I'll look into it
it looks like a "python too old" error
I dont get yet why.. why python version is this`?
I'm using Python 2.6.5 here
the 'except ... as' syntax was added to python2.6 and is not in python2.5.
[22:16:48] <jepler> http://docs.python.org/whatsnew/2.6.html#pep-3110-exception-handling-changes
I see.. my python is too young.. is it all to be 2.5 compat for now?
even though I don't think we're going to ship emc 2.5 packages on ubuntu hardy, we are still not making gratuitous breakage
so, yes, your code needs to work back to python2.5.2
ok, will get me one for testing, thanks
EMC: 03jepler 07master * r02abdee62aa9 10/src/hal/user_comps/gladevcp.py: gladevcp: python2.5 compatibility
.. of course, I didn't actually test that on hardy :-/