Re: 'My indexing' broken by 3.07
On 16/12/17 14:35, Ralph Corderoy wrote: But in this case, I think `^' is cheaper. Well done you! My instincts told me that it might be cheaper ;) ___ get_iplayer mailing list get_iplayer@lists.infradead.org http://lists.infradead.org/mailman/listinfo/get_iplayer
Re: 'My indexing' broken by 3.07
Hi Charles, > > Yes. `^' also suffices. > > Interesting. I wonder if 'match beginning of the line' is less > expensive internally? Perl's regexp engine is historically extremely good at spotting optimisations, and some of those details can be seen with its -D option if perl is compiled suitably, e.g. /x.*foo$/ might decide the minimum length is four and it must end with `foo' before attempting to find an `x'. But in this case, I think `^' is cheaper. $ for p in ^ '.*'; do > for n in 100 1000 1; do > seq $n | > perf stat -e instructions \ > perl -ne "/$p/" > done > done |& > grep instructions:u 2,588,069 instructions:u 4,947,485 instructions:u 28,715,945 instructions:u 2,600,183 instructions:u 5,089,466 instructions:u 30,189,787 instructions:u $ Re-arranged, that's n /^//.*//.*/ 100 2,588,069 2,600,183 +12,114 1,000 4,947,485 5,089,466+141,981 10,000 28,715,945 30,189,787 +1,473,842 -- Cheers, Ralph. https://plus.google.com/+RalphCorderoy ___ get_iplayer mailing list get_iplayer@lists.infradead.org http://lists.infradead.org/mailman/listinfo/get_iplayer
Re: 'My indexing' broken by 3.07
On 16/12/17 14:14, Ralph Corderoy wrote: Yes. `^' also suffices. Interesting. I wonder if 'match beginning of the line' is less expensive internally? ___ get_iplayer mailing list get_iplayer@lists.infradead.org http://lists.infradead.org/mailman/listinfo/get_iplayer
Re: 'My indexing' broken by 3.07
Hi Charles, > > ] wildcard search: get_iplayer ".*" - note the quotes. > > Thanks so much for that Mark. That looks like a regex. Is it, do you know? Yes. `^' also suffices. -- Cheers, Ralph. https://plus.google.com/+RalphCorderoy ___ get_iplayer mailing list get_iplayer@lists.infradead.org http://lists.infradead.org/mailman/listinfo/get_iplayer
Re: 'My indexing' broken by 3.07
On 16/12/17 13:39, Mark Carroll wrote: ] If you wish to list all programmes, you must now explicitly specify a ] wildcard search: get_iplayer ".*" - note the quotes. -- Mark Thanks so much for that Mark. That looks like a regex. Is it, do you know? ___ get_iplayer mailing list get_iplayer@lists.infradead.org http://lists.infradead.org/mailman/listinfo/get_iplayer
Re: 'My indexing' broken by 3.07
On 16 Dec 2017, Charles Johnson wrote: > get_iplayer --type=radio --refresh >$RADIO_FILE > > was the content of a script with which i built myself a text index of > programmes (there was possibly a more efficient way to derive the index > from the cache?). That no longer works in 3.07. (snip) > INFO: Indexing radio programmes (concurrent) > ... > INFO: Added 0 radio programmes to cache > > Is there some way i can do the same without sticking to 3.06? Are you running into the very first bullet point from https://github.com/get-iplayer/get_iplayer/wiki/release300to309#release307 ? ] If you wish to list all programmes, you must now explicitly specify a ] wildcard search: get_iplayer ".*" - note the quotes. -- Mark ___ get_iplayer mailing list get_iplayer@lists.infradead.org http://lists.infradead.org/mailman/listinfo/get_iplayer
'My indexing' broken by 3.07
get_iplayer --type=radio --refresh >$RADIO_FILE was the content of a script with which i built myself a text index of programmes (there was possibly a more efficient way to derive the index from the cache?). That no longer works in 3.07. This is what i get get_iplayer v3.07, Copyright (C) 2008-2010 Phil Lewis This program comes with ABSOLUTELY NO WARRANTY; for details use --warranty. This is free software, and you are welcome to redistribute it under certain conditions; use --conditions for details. INFO: Indexing radio programmes (concurrent) ... INFO: Added 0 radio programmes to cache Is there some way i can do the same without sticking to 3.06? ___ get_iplayer mailing list get_iplayer@lists.infradead.org http://lists.infradead.org/mailman/listinfo/get_iplayer
