Oops, wrong sender mail address - trying again...
Hi, it's me again - the guy from the sourceforge forum with
the plan to teach the Harmonys Philips Pronto hex codes...
While looking at the IR learning code, I could not help
noticing that libconcord.cpp is cluttered all the same file
reading ops o
[ 1898492 ] LearnIR makes next run fail on H785:
After any run of harmony -l.. to learn IR command, any next run of harmony
fails. Any subsequent run after the failed one is OK again (until the next
'-l').
Anyone willing to try at least if this is reproducible with
other Harmonys, and if yes, veri
On Wednesday 26 March 2008, Phil Dibowitz wrote:
> Your patch looks very solid. However, your changing things that Stephen's
> API patch is ALSO going to change (in fact, his patch will make your patch
> simpler, you won't have to deal with that silly static-size vs dynamic-size
> thing).
> Can you
To allow insertion of IR codes from other sources (e.g. from the
Philips Pronto hex code archives), I have made up a draft of an
extended Learn_IR API for libconcorde.h.
It separates the learning and POSTing operations, to allow the
API client to modify the IR code before POSTing, as well as
acces
Well, thanks for your precious time...
Seems like all this 'Harmony vs. Concordance', 'LINUX vs. Windows'
and 'ANSI vs. C99' stuff has settled enough to try and merge my
other two patches, so here just a few short notes to your comments:
On Monday 31 March 2008, Phil Dibowitz wrote:
> OK, I spen
On Monday 31 March 2008, Phil Dibowitz wrote:
> Phil Dibowitz wrote:
> > 1. It's generated against 0.13, but CVS has changed DRASTICALLY since
> > then - can you please re-gen against CVS, and also check our new
> > SubmittingPatches file for further details.
done (hopefully conforming the rules no
On Monday 31 March 2008, Phil Dibowitz wrote:
> Now that Stephen's patch has been merged, did you want to re-submit your
> patch against the latest CVS? In theory it should make your patch a bit
> smaller...
Yup - piece of cake now, with all that \0 termination or not stuff gone.
Makes some functi
On Tuesday 01 April 2008, Phil Dibowitz wrote:
> Applied. I made a tweak and renamed read_from_file to be _read_from_file,
> but other than that, I applied it as-is.
>
> Though I think I may re-org that file a bit.
Feel free as you like - it's just my 2p to keep things going and
tidy (same for the
On Wednesday 09 April 2008, Phil Dibowitz wrote:
> Marcel de Jong wrote:
> > Hello all,
> >
> > I'm afraid I have some bad news, not sure if others have had this as
> > well. When I tried to set up my 555, I needed to have the remote learn
> > some new trick via IR (basically because it refused to
On Thursday 10 April 2008, Phil Dibowitz wrote:
> No need to initialize that err there once the if() is removed. And yes, I
> removed the if() before I sent the email. The commit message was in my
> inbox at 16:46 on 4/8.
Phil,
you're right - CVS just fooled me since I didn't update recently and
it
On Thursday 10 April 2008, Marcel de Jong wrote:
> I grabbed the CVS version yesterday, and had it run, but it only took
> one command in the file. I had to download the learnIr file for each
> and every command separately. I kinda expected it to read through all
> the key commands selected in the
On Thursday 10 April 2008, Phil Dibowitz wrote:
> Thanks. That's a whole lot of work when you coulda just opened firefox in
> linux, selected the keys you wanted, downloaded the XML file from the site
> and sent it along rather than piecing it together with wireshark. :)
Well, I was just curious t
On Wednesday 07 May 2008, JDN wrote:
> Can someone take this poor dumb fellow in hand and help me figure out how
> to compile the concordance in Windows XP using Visual Basic 2008 Express
> 9.0 ?
Not at all, I'm afraid - libconcord is C++, so you will need to install
the Visual C++ part of Visual
Joe,
On Thursday 08 May 2008, N Farms wrote:
> I can't get Visual Basic C++ 6.0 to install on my XP system.
> Would it be possible to send me the executable in a zip file so I can try
> it to see if it is even worth all the effort to compile on my own ?
Note that 'Visual Basic C++ 6.0' is a cont
HI all,
I have found some time to do some work on the LearnIR
interface, which I have almost completed by now:
- allow LearnIR.EZtut file with more than one key
(several keys tagged in the web interface)
- added callback for key name and IR code information:
this removes the output of the key
ally test it yet.
Actual changes going to follow soon in separate messages..
Regards, Andreas Schulz
-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open
Here come the first patches for libconcord and concordance,
providing resp. using the new IR learning API.
Patches are against the current (2008-06-06) CVS versions.
Regards, Andreas Schulz
Index: concordance.1
===
RCS file
d the changelog, everything
else is as in congruity v7.
Regards, Andreas Schulz
congruity8-learn-20080606.tgz
Description: application/tgz
-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell se
ible in case it should kill your cat...
Regards, Andreas Schulz
libirremotes-0.20.tar.gz
Description: application/tgz
-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just
Last, but not least, here comes the patches for congruity
and concordance to utilize libIRremotes for reading Pronto
codes.
Patches are against congruity8 resp. CVS concordance plus
patch for new IR learning (from previous messages).
Regards, Andreas Schulz
diff -prc congruity8/Changelog
Sorry, please don't take congruity 8/8.1 as my final words - looks like I've
been a little too keen to get it out..
As I found out during testing my stuff with WindowsXP this weekend, I
apparently managed to get a few bugs in while adapting to my last version of
libconcord - partly I messed up
First, my apology to Stephen - I didn't surely intend to interfere
with your release policy for congruity, but I just needed a new version,
and mindlessly decided that I would do well enough to it to deserve
a new major number.
Since I already told everyone around here to please ignore my
congruit
Next, here comes the latest version of my IR learning patch for
libconcord. As my first try, it changes the API from one
'learnIR' function to four functions:
char *get_key_name(uint8_t *data, uint32_t size, uint32_t index);
int learn_from_remote(uint32_t *carrier_clock, uint32_t **pulses,
otecentral.com.
.SH AUTHOR
-\fBconcordance\fP was written by Kevin Timmerman and Phil Dibowitz
+\fBconcordance\fP was written by Kevin Timmerman and Phil Dibowitz; IR learning update by Andreas Schulz
.br
http://www.phildev.net/concordance/
Index: conco
On Sunday 15 June 2008, Andreas Schulz wrote:
> Next, here comes the latest version of my IR learning patch for
> libconcord. As my first try, it changes the API from one
> 'learnIR' function to four functions:
oops - that should have been deleted - shouldn't start edit
+++ Changelog 15 Jun 2008 22:25:44 -
@@ -1,3 +1,17 @@
+* 2008-06-13 Andreas Schulz <[EMAIL PROTECTED]> - patch
+- Tested with LINUX/KDE and WinXP
+- Added zoom in/out to IR signal plot
+- Changed invocation of 'program' - action_func made a parameter
+- Added support for
On Sunday 15 June 2008, Phil Dibowitz wrote:
> I think you missed the point. It wasn't the 8.x that was the problem. It
> was assigning a version number.
Well, advocacy was never my business - but I'm getting it..
OK then, I'll keep my hands off any number that does not go into some
computation,
Hi Stephen (and other subscribers),
On Thursday 19 June 2008, Stephen Warren wrote:
> I'm slowly reading through your version/patches. My plan is to initially
> integrate any desired changes that aren't related to IR learning; these
> could form part of any release prior to a new libconcord being
On Sunday 22 June 2008, Mike Fletcher wrote:
> Then, I
> clicked "Build" menu on the menu bar, then selected "Built Solution"
> or you could press F7. After Visual Studio completed its process I
> found the exe in (extracted archive folder)\concordance\win\Debug.
Ah, yes, it looks like I forgot to
On Sunday 22 June 2008, Stephen Warren wrote:
> Phil Dibowitz wrote:
> > Stephen Warren wrote:
> >> It'd be nice if all symbols (types, values, functions, etc.) were all
> >> prefixed with e.g. "IRR_"/"irr_" to ensure they never conflict with any
> >> other library.
> > ACK for the public functions
OK, I plead guilty of lazyness and sloppy design in a few cases..
On Saturday 21 June 2008, Phil Dibowitz wrote:
> I have a request for future submissions of patches that have multiple
> parts (i.e. patchsets). I'd like to follow the general LKML standard
> here for this...
A valid point - I wi
(I re-added some cuts by Phil to answer both posts in one)
> On Sunday 22 June 2008, Phil Dibowitz wrote:
> > On Sunday 22 June 2008, Stephen Warren wrote:
> > > A "pulse" is usually a low->high->low transition, whereas these docs
> > > talk about pulses and spaces. There should be differentiation
On Saturday 21 June 2008, Phil Dibowitz wrote:
> (hmm - feel like copy-n-pasting a session of use?)
Here's your sample session - the key names are just '0'..'5'.
Note the rendering of the 2nd IR signal is not really accurate -
it should have some gaps of different length in between, but
both gap
So, maybe we can agree on the following interface:
/*
* IR-stuff
* ===
* Data structure information:
*
* carrier_clock: in Hz, usually ~36000..4
* ir_signal: IR mark/space durations (alternating) in microsconds
* ir_signal_length : total number of mar
On Tuesday 24 June 2008, Andreas Schulz wrote:
> So, maybe we can agree on the following interface:
...
> void destroy_key_names(char **garbage);
...
> void destroy_ir_signal(uint32_t *garbage);
...
> void destroy_post_string(char *garbage);
As I just recognized, better make that del
On Tuesday 24 June 2008, Phil Dibowitz wrote:
> Andreas Schulz wrote:
> > As I just recognized, better make that delete_*, to be consistent
> > with delete_blob..
>
> Delete blob always deletes the same thing - in your case you have lots
> of different things..
Sure,
On Tuesday 24 June 2008, Stephen Warren wrote:
> On Tue, June 24, 2008 1:04 am, Andreas Schulz wrote:
> In the delete_* functions, I'd use a more descriptive name for the
> parameter than "garbage", e.g.:
>
> void delete_key_names(char **key_names);
> ...
>
Just to support the significance of the IR signal plot-
all signals are from the same original remote key:
- 1st : way too far away from the Harmony, nothing received at all
- 2nd : just a little bit too far away from the Harmony, almost right
- 3rd/4th : perfect
- 5th : between 1st and 2nd, receiv
OK, this should finally cover all of Phil's and Stephen's comments:
- though, now that I'm writing this message, some more of the stuff in
the 'API change' section below should probably go into libconcord.h...
This patchset has been build and tested with a Harmony 785, on the
following platforms:
(see also message:
[patch 0/2] Change of libconcord API for IR codes learning - Build 20080628)
Patch installation:
===
This patch should be applied to libconcord CVS release 0.20.
Install via:
cd /libconcord
patch -p1 < libconcord-IRLearn-20080628.patch
TODO:
=
While I ma
(see also message:
[patch 0/2] Change of libconcord API for IR codes learning - Build 20080628)
Note on concordance UI:
===
To make use of the changed IR learning API of libconcord, after this patch
concordance will:
- show the number of keys to be learned
- prompt for the nex
On Saturday 28 June 2008, Stephen Warren wrote:
> Stephen Warren wrote:
> > Attached is an updated version
>
> Attached is another (final) updated version that removes a debug print I
> was using. Sigh.
Looks like some great work.. Considering the amount of work that it caused
you, I'm already sta
On Sunday 29 June 2008, Phil Dibowitz wrote:
> Did you see my comments/suggestions on the output for IR learning? This
> code doesn't seem to follow that...
The only comment I found was:
> * Hmm - should be be showing pulses and details in non-verbose
> mode? I would think that _dump_new_code() s
On Wednesday 02 July 2008, Stephen Warren wrote:
> I believe that congruity SVN now includes everything in your proposed
> congruity changes that isn't related to IR learning. Note that I didn't
> simply merge your patch directly; I took the opportunity to do various
> other cleanup and changes at
Phil Dibowitz wrote:
> You missed this email in response to your sample output
Phil, this updated patch (based again on CVS 0.20) should include
your lost comments:
--
Concordance 0.20+CVS
Copyright 2007 Kevin Timmerman and Phil Dibowitz
This software
On Monday 14 July 2008, Stephen Warren wrote:
> 1) How was MAX_IR_SIGNAL_LENGTH chosen? Do we have any information on
> whether this correlates well with the official Logitech software?
I didn't change that from the original version - looks like
'ten times more than we will ever need, just to make
On Sunday 07 September 2008, Phil Dibowitz wrote:
> Did you see my comments on the most recent version of your concordance
> patches? All but one of them were minor - and the other was more of a
> question.
Yes, I apologize for being lazy with my mails recently - besides holidays and
and business
OK, I'm a little late - at least, I'm planning to have a new
patch considering Phil's major points ready for tomorrow...
On Friday 19 September 2008, Stephen Warren wrote:
> If I press and hold the button on the original remote forever (I tested
> 60 seconds), the recording process takes forever.
Eventually, I hope that I got most of Phil's and Stephen's remarks into this
patch. Patch is against the current CVS status (as of 2008-10-10).
Changes are mostly editorial, as suggested by Phil, plus a few constant
definitions and minor adjustments in the processing flow.
Added missing timeout/b
Changes relative to first patch version (20080628):
===
libconcord.h
- added LC_ERROR_IR_OVERFLOW:
- fixed some spacings and line breaks
libconcord.cpp
- reverted #include back to #include
for some reason, original now works for both Windows and
Changes relative to first patch version (20080628):
===
concordance.1
- changed entry for IR learn option
concordance.c
Note: In contrast to Phil's belief, _getch will actually return only '\r'
when you hit return, as getchar wil return '\n', so the
On Tuesday 14 October 2008, Phil Dibowitz wrote:
> Stephen Warren wrote:
> > .. callback added ..
> Absolutely the right call.
> Applied, thanks.
Stephen and Phil, thanks also from me for that patch.
I have a few days off work next week, so I may find
some time to fill in code to make use of the
On Thursday 16 October 2008, Phil Dibowitz wrote:
> On Wed, Oct 15, 2008 at 09:49:38PM -0400, Stuart Doherty wrote:
> > The command I use is:
> > sudo concordance -v -C Connectivity.EZHex
>
> That's a *connectivity* file, not a config file. Don't specify a mode
> (it'll figure it out) or use -t for
Phil/Stuart,
Since I, from the lack of any follow-up messages, had the impression
that my proposed patch for Stuart's problem has failed to attract
public attention in this list (probably because I once again failed to
properly tag it as [patch]), here I try again:
On Thursday 16 October 2008, Ph
Hi,
Looks OK so far...
I will check as soon as I can borrow our 785 from the
rest of my family... Until then, could you try again with:
$ concordance -v LearnIr.EZTut
and post the result?
Regards, Andreas
On Sunday 19 April 2009, Michael Frase wrote:
> Hi,
>
> I'm trying to add new IR codes to m
OK, I tried, and it works for me.
I created a new dummy device in my config:
Home Appliance - zzz-testing Home Appliance (just4test)
and learned a few keys from some other remote (power, 1,2,3).
All these keys show up as 'your original remote' in light blue
in the keys table.
After I updated my 78
On Sunday 12 April 2009, Phil Dibowitz wrote:
> All,
>
> I'm fairly certain the work that made firmware updates work on the 5xx
> remotes should allow the 6xx (arch 7) and 748/768 (arch 3) to firmware
> update as well. But I never confirmed this.
>
> Can someone with these remotes test and report b
On Wednesday 22 April 2009, Phil Dibowitz wrote:
> Oh, sweet! Thanks. I've updated the website.
> Can you also send a concordance -vi ? Does it list the architecture? We
> don't have an arch number listed for the 720 and 785.
> Thanks again.
Concordance 0.21
Copyright 2007 Kevin Timmerman and Phil
; if not, write to the Free Software Foundation, Inc.,
* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
*
* (C) Copyright Andreas Schulz 2008
*
* Credits go to Evgueni Oulianov (eoulia...@hotbox.ru) for his
* excellent document about the different Pronto IR formats,
* which can be
On Sunday 26 April 2009, Michael Frase wrote:
> I have sent you a private mail with attached wireshark log. Did you
> receive it?
> Please have a look at it, if you have time.
> Michael
Yes, sorry for the delay, but I have been busy with other stuff
(plus, as I might have told already, I have alwa
Stephen,
After some time, I now had reason to check the Pronto code
conversion of congruity, just to discover an error in the code:
You just copy the burst/space times entered as Pronto hex code
to the IR signal that is passed to libconcord.
However, in Pronto codes burst/space times are given i
Just a minor update to my previous message:
The Pronto part of congruity was of course done by Stephen (with some minor
corrections by myself).
And, browsing once again through the old messages from 4 years ago, it seems
that the lack of ambition to push the pronto function into the mainline seem
- Nachricht von s...@techie.net -
> Never got your previous message...
..just when I thought that I had mastered that jet-lag after two weeks..
OK, next try, now hopefully with the correct From: and To:
--- snip --
Hi there,
yes it's me, still occasionally browsing the mails from the
(hope this one works on the first try...)
I have checked the archive at http://hellgi.funpic.de/concordance.zip
in between and it actually has the sources of concordance, including
HellG's hack for reading multiple codes from a file and my pronto
patch that I posted here a few years ago so he's not
64 matches
Mail list logo
