Navigator-Suite Feedback, questions, bug-reports or just chatting |
![]() ![]() |
Navigator-Suite Feedback, questions, bug-reports or just chatting |
Feb 14 2005, 01:28
Post
#1
|
||
![]() Group: Members Posts: 3353 Joined: 6-July 03 From: Sachsen (DE) Member No.: 7609 |
Intro
This thread is for feedback, bug-reports or just chatting about "Navigator", a playlist design for Columns UI. The layout is very modular and can easily be adapted to suit your needs - here is a screenshot which shows a few possibilities of what you can do with it: Description
Download Current version for foobar 0.9.x:
Navigator_v1.4.3.zip ( 97.89K )
Number of downloads: 21802Old version for foobar 0.8.3:
Navigator_v1.3.2.zip ( 66.64K )
Number of downloads: 1676Important!: You need columns_ui for this to work. About feature-proposals I currently have no interest in feature-proposals. This is because i consider the 1.x.x line of Navigator finished. Bug-reports however are welcome. _______________________________________________________________ Changelog: 1.4.3 (foobar 0.9.x compatible) - fixed display of singles in albummode for tracks with no ARTIST - the "artist & album"-column now uses ARTIST instead of ALBUM for inline-metadata editing - added new colorscheme by 4nt1 1.4.2 - moved albummode STYLE-display into a seperate line - Tracks which have TOTALTRACKS=1 are now also considered singles - fixed inverted colors in albummode when displaying: performer, conductor, publisher 1.4.1 - fixed wrong upload (STYLE didn't work) 1.4.0 - compatible with foobar 0.9 and the coresponding ui-columns (no need for legacy mode) - improved performance by making use of new ui-columns style-features to reduce codesize - sorting by all kinds of playback-stats works correct now - added full support for the recently standardized STYLE-tag - added basic support for inline-metadata editing - redesigned hybrid-mode - fixed statusbar display-errors when using RTL-languages - when enqueueing audio-cds, albummode is now automatically activated - properly honors the new field-remappings - many bugfixes and code cleanup - misc changes to the how-to docs - probably some other stuff which i forgot 1.3.2 (foobar 0.8.3 compatible) - bugfix: "daily plays"-column did show wrong values. - last version for fb2k 0.8.3 1.3.1 - bugfix: in the "daily plays"-column, tracks which were played today the first time would show strange ratios - finally replaced the ancient screenshot with an up-to-date one 1.3.0 - full support for the new playcount-plugin - added "plays per day"-stats (new playcounter-plugin required for it to show up). You cannot sort by it yet. - added first_played-column (new playcounter-plugin required for it to work). - moved replaygain-indicators from the metadata-matrix into a seperate column - dropped configurability of the seperator-char, since almost nobody used it anyways - sorting-code polishing: you can now correctly sort by multiple columns (like for example first sorting by the album-column, and then sorting by rating) This post has been edited by Lyx: May 21 2006, 15:19 |
|
|
|
||
Feb 14 2005, 01:37
Post
#2
|
|
![]() Group: Members Posts: 71 Joined: 23-July 04 From: Australia Member No.: 15689 |
That looks awesome Lyx, will definetly give this a shot and thanks too.
-------------------- You're talking to my guy all wrong... It's the wrong tone. Say it again, and i'll stab you in the face with a soldering iron!
|
|
|
|
Feb 14 2005, 01:58
Post
#3
|
|
|
Columns UI developer Group: Developer Posts: 3034 Joined: 20-December 02 From: United Kingdom Member No.: 4177 |
Looks pretty (very) nice, probably first time ive said that at someone elses config as well. Slow as hell though, I dont know what you consider a low-end PC..
-------------------- .
|
|
|
|
Feb 14 2005, 02:22
Post
#4
|
|
![]() Group: Members Posts: 3353 Joined: 6-July 03 From: Sachsen (DE) Member No.: 7609 |
QUOTE (musicmusic @ Feb 14 2005, 02:58 AM) Looks pretty (very) nice, probably first time ive said that at someone elses config as well. Slow as hell though, I dont know what you consider a low-end PC.. Hmm, someone else who did lots of test-runs of it for me has a 700mhz box, and he says it runs acceptable - but i guess your bars are higher because you're a coder :-) Large parts of the code is non-trackspecific stuff *hinthint* To be more specific....... the "culprits" for the resource-hog are two: 1. non-trackspecific code - like calculation of secondary colors 2. the tag-guess code eats resources even though its inside a large if-loop which only gets executed when not all standard tags are there - but the parsing alone seems to be heavy already. Number 2 i will "fix" with a "light-version" (minus the tag-guess code) myself. Number one is out of my hands :-) Theoretically, given the truckload of code, this should run much slower than it actually does run. I doubt i can do any more except the above two things about it, because i've already tried to save execution-speed whereever i can by putting conditional stuff into if-blocks etc. --- Oh, btw: reports about scrolling-speed/responsiveness on a variety of systems are very welcome. Please also mention your CPU and OS when doing so. - Lyx This post has been edited by Lyx: Feb 14 2005, 02:54 -------------------- I am arrogant and I can afford it because I deliver.
|
|
|
|
Feb 14 2005, 12:19
Post
#5
|
|
![]() Group: Members Posts: 3353 Joined: 6-July 03 From: Sachsen (DE) Member No.: 7609 |
Updated final 1.00 version. The only change is that it now also includes a "Lite Version" without the tag-guess code, which runs a bit faster.
-------------------- I am arrogant and I can afford it because I deliver.
|
|
|
|
Feb 14 2005, 12:32
Post
#6
|
|
![]() Group: Members (Donating) Posts: 391 Joined: 2-March 04 Member No.: 12414 |
Lyx you can also ask Neksus for an account or use the upload for unregistered users.
|
|
|
|
Feb 14 2005, 15:27
Post
#7
|
|
![]() Group: Members Posts: 680 Joined: 11-July 03 From: Brno, Czech Rep. Member No.: 7705 |
thumbs up!
great work lyx, i like how easy it is to set it up and / or set custom colors (as you compute the rest; there are some limitations because of that's the pay-off for the ease of it). and i can see that it would greatly benefit from possibility of new non-track specific section in columns ui. btw, have you done any test with that color calculations disabled to see what impact it would have on speed of the string? because of your tag guessing it's one of few strings i can actually use (the other ones being old plisk's one with custom changes or mine, which is not actually available for columns ui). so once again, nice work! edit: dano - lyx is waiting for his account on that strings' site ... lyx - i'm with you on that issue with play_date. it would be better (for general configs) to have only one standard form This post has been edited by mazy: Feb 14 2005, 15:35 -------------------- info about my tag guesser script for foo_lua (preview version available):
http://www.hydrogenaudio.org/index.php?showtopic=16987 |
|
|
|
Feb 14 2005, 15:34
Post
#8
|
|
![]() Group: Members Posts: 73 Joined: 20-March 04 Member No.: 12874 |
congrats on the release
may it bring enlightenment to many |
|
|
|
Feb 14 2005, 16:09
Post
#9
|
|
![]() Group: Members Posts: 680 Joined: 11-July 03 From: Brno, Czech Rep. Member No.: 7705 |
lyx: it seems the string doesn't recognize my va albums.
most of them start with 'VA-', like this one: 'G:\Sorted\Samba, Bossa Nova, Brazil & Latin\VA-Bossa_Nova_For_Lovers-2003-ego\'. i'm using your string with album mode as default and tag guessing. i've quickly checked your string and it seems to me you do look for '\va-' in the path (after stripping off whitespace and separators), so it should work i guess ... any idea? it's this line: CODE $strstr($lower($replace(%_path%,'.',$char(),' ',$char()),'_',$char()),'\va-'), it seems to me it's somehow broken (not that line, the rest of string in regards to va mode) ... btw nice way of doing it, i'm mean the way you strip off separators to make it more bullet-proof. also, i have a feature request: could you add switch like 'treat_ost_like_va', which would cause ost albums to be treated like va albums. ost album would be one with soundtrack genre or '\ost-' (after your stripping) or '(ost)' in its path. '(ost)' is what i use in folder's name for soundtracks, '\ost-' is being used by some scene groups instead of 'va-...(ost)...', 'v.a.-...(ost)...', 'va_-_...(ost)...' etc., you know what i mean, don't you? This post has been edited by mazy: Feb 14 2005, 16:10 -------------------- info about my tag guesser script for foo_lua (preview version available):
http://www.hydrogenaudio.org/index.php?showtopic=16987 |
|
|
|
Feb 14 2005, 16:46
Post
#10
|
|
![]() Group: Members Posts: 3353 Joined: 6-July 03 From: Sachsen (DE) Member No.: 7609 |
mazy, i will look into the va-issue later today to check if its intentional, or a bug (i intentionally don't support some va-patterns because searching for them would trigger too many false-alarms)
The OST-thingie is a nice idea - i'll probably implement it someway. - Lyx -------------------- I am arrogant and I can afford it because I deliver.
|
|
|
|
Feb 14 2005, 16:48
Post
#11
|
|
![]() Group: Members Posts: 680 Joined: 11-July 03 From: Brno, Czech Rep. Member No.: 7705 |
great, thanks ...
-------------------- info about my tag guesser script for foo_lua (preview version available):
http://www.hydrogenaudio.org/index.php?showtopic=16987 |
|
|
|
Feb 14 2005, 17:24
Post
#12
|
|
![]() Group: Members Posts: 3353 Joined: 6-July 03 From: Sachsen (DE) Member No.: 7609 |
Hmm, i just had an idea how to speedup the code a bit more. Currently, the V.A. check is done for every track, even when albummode is not active. I could change this - but this would make the detection core not redundant anymore, because it would then check for a variable which is not inside of it. Hmm, or i could make it so that it only disables the va-check when albummode is explicitely marked disabled. That way, it would still work when copy'n pasted into another string - the cost would just be wasting one if+strcmp for checking against something which doesn't exist.
I'll probably do that, because extending the VA-code to include stuff like OSTs is just not viable when its done for every track even in singlemode. So, unless some big sideeffect arises, 1.01 will probably contain a slight speedup in singlemode, support for OSTs and (if its a bug, but i guess it is) a fix for the VA-problem mentioned by mazy. - Lyx -------------------- I am arrogant and I can afford it because I deliver.
|
|
|
|
Feb 14 2005, 17:34
Post
#13
|
|
![]() Group: Members (Donating) Posts: 391 Joined: 2-March 04 Member No.: 12414 |
Does the OST thing stay optional? Not all my OST's are by various artists.
|
|
|
|
Feb 14 2005, 17:38
Post
#14
|
|
![]() Group: Members Posts: 3353 Joined: 6-July 03 From: Sachsen (DE) Member No.: 7609 |
Yes, i'll make an option in the config "Treat OSTs like Various Artists albums?"
- Lyx edit: mazy, there is a brackets-typo in the code-line you mentioned which causes the va-bug. Codeline in 1.00: CODE $strstr($lower($replace(%_path%,'.',$char(),' ',$char()),'_',$char()),'\va-'), Fixed Codeline in upcoming 1.01: CODE $strstr($lower($replace(%_path%,'.',$char(),' ',$char(),'_',$char())),'\va-'), Late answer about possible speedup via non-trackspecific global string support in columns ui: Well, it's difficult to test it, but let me put it this way: With the light-version (no tag-guessing) singlemode is still laggy on my 400mhz box. The thing is, the columns which i had enabled to test it didn't contain much code - so it has to be the global-string. With the tag-guessing removed, 50% of the global string is non-trackspecific code and 15% is exporting vars. So, i'd say its _very_ probable that the two resource-hogs are tag-guessing (fixed via the lite-version) and non-trackspecific code. This post has been edited by Lyx: Feb 14 2005, 19:11 -------------------- I am arrogant and I can afford it because I deliver.
|
|
|
|
Feb 14 2005, 20:00
Post
#15
|
|
![]() Group: Members Posts: 1099 Joined: 18-March 03 From: Oslo, Norway Member No.: 5569 |
QUOTE (mazy @ Feb 14 2005, 03:27 PM) lyx - i'm with you on that issue with play_date. it would be better (for general configs) to have only one standard form Maybe we should start a thread to discuss and agree upon, some "standard" tags for different purposes? E.g. the format of a time/date tag itself, doesn't really make any difference, as its contents can be easily customized for display. If we agree on one format, we could just provide the code needed to display it in different ways.I think this community could benefit from it in the long run, as changing from one formatting to another would be less confusing to new users. |
|
|
|
Feb 14 2005, 20:22
Post
#16
|
|
![]() Group: Members Posts: 3353 Joined: 6-July 03 From: Sachsen (DE) Member No.: 7609 |
1.01 uploaded - changelog is in the first post of this thread.
@upnorth: Agreed. Although imho a balance between "popularity" and "reasonable to support for devs" has to be found. Requiring users to set unpopular weird non-standard tags isn't newbie-friendly as well :-) But thats just details, in general, i completely agree with you. -------------------- I am arrogant and I can afford it because I deliver.
|
|
|
|
Feb 14 2005, 20:47
Post
#17
|
|
![]() Group: Members Posts: 1099 Joined: 18-March 03 From: Oslo, Norway Member No.: 5569 |
QUOTE (Lyx @ Feb 14 2005, 08:22 PM) Yeah, just something that could be used as a guideline.
|
|
|
|
Feb 14 2005, 20:58
Post
#18
|
|
|
Group: Members Posts: 525 Joined: 1-January 05 From: Boston Member No.: 18762 |
QUOTE (upNorth @ Feb 14 2005, 02:47 PM) QUOTE (Lyx @ Feb 14 2005, 08:22 PM) @upnorth: Yeah, just something that could be used as a guideline.Agreed. Although imho a balance between "popularity" and "reasonable to support for devs" has to be found. Requiring users to set unpopular weird non-standard tags isn't newbie-friendly as well :-) But thats just details, in general, i completely agree with you. I've created an "Accepted Tag Standards" section in the wiki, so please add any standards for which you come to a conclusion. I've already included what I expect will be the PLAY_DATE standard (YYYYMMDD), but please correct me if I'm wrong in that. http://wiki.hydrogenaudio.org/index.php?ti...d_Tag_Standards |
|
|
|
Feb 14 2005, 21:09
Post
#19
|
|
![]() Group: Members Posts: 3353 Joined: 6-July 03 From: Sachsen (DE) Member No.: 7609 |
QUOTE (topdownjimmy @ Feb 14 2005, 09:58 PM) on in the wiki, so please add any standards for which you come to a conclusion. I've already included what I expect will be the PLAY_DATE standard (YYYYMMDD), but please correct me if I'm wrong in that. In general, i'm all for the ISO-version (YYYYMMDD). But my proposal would be to make it "YYYY-MM-DD", because that way, it can be verified with TAGZ (by checking the position of the seperators). As a bonus, it also makes it easily readable even without processing. - Lyx -------------------- I am arrogant and I can afford it because I deliver.
|
|
|
|
Feb 14 2005, 21:19
Post
#20
|
|
![]() Group: Members Posts: 1099 Joined: 18-March 03 From: Oslo, Norway Member No.: 5569 |
I don't remember the contents of the default PLAY_DATE tag, but wouldn't it be nice to append time info to it ($H$M$S or something)? No need to add (waste) another tag just for info that can be stored in the first. I don't really use time info myself, but I'm considering adding it when I change format (something I'm about to do).
Maybe there isn't enough people that cares about this, but at least I want to think it through before I abandon my current format. |
|
|
|
Feb 14 2005, 21:26
Post
#21
|
|
![]() Group: Members Posts: 3353 Joined: 6-July 03 From: Sachsen (DE) Member No.: 7609 |
the default content is DDMMYY if i remember right. The european-version... just to make absolutely sure that the majority of users will change it to something else anyways(thats the main cause for the problem - if it would have been ISO by default, then it would be easy to say "i only support the default" - but since the default is geared only towards europeans, its understandable that users will change it to an unknown format).
(this is not to say anything against europeans - i'm from germany - but picking this dateformat for an international app which supports other "plugins" which make use of it, is just..... weird) - Lyx -------------------- I am arrogant and I can afford it because I deliver.
|
|
|
|
Feb 14 2005, 21:51
Post
#22
|
|
|
Group: Members Posts: 525 Joined: 1-January 05 From: Boston Member No.: 18762 |
QUOTE (Lyx @ Feb 14 2005, 03:09 PM) QUOTE (topdownjimmy @ Feb 14 2005, 09:58 PM) on in the wiki, so please add any standards for which you come to a conclusion. I've already included what I expect will be the PLAY_DATE standard (YYYYMMDD), but please correct me if I'm wrong in that. In general, i'm all for the ISO-version (YYYYMMDD). But my proposal would be to make it "YYYY-MM-DD", because that way, it can be verified with TAGZ (by checking the position of the seperators). As a bonus, it also makes it easily readable even without processing. - Lyx Can quantitative comparisons be made on strings containing hyphens? I'm at work, so can't test :/ upNorth, if the time were to be included in the PLAY_DATE field, how do you suggest the entire field be formatted? I think this would be best because then if someone enjoys having a timestamp but wants to make their second playcount field available for some custom tag, they won't be forced to violate the standard (by appending $H$M$S to the standard YYYY-MM-DD or whatever). ...and because the field would then contain both date and time, would PLAY_DATE still be an appropriate field name? Perhaps with a new standard a new field name is in order (e.g. LAST_PLAYED). |
|
|
|
Feb 14 2005, 21:58
Post
#23
|
|
![]() Group: Members Posts: 3353 Joined: 6-July 03 From: Sachsen (DE) Member No.: 7609 |
Sidenote: When deciding about a standard, it should also be considered how easy it is for users to switch to it. Thus, the more easy, less disruptive and more attractive the transition, the better the chances of widespread use.
Creating a completely new tag-field may be a bit hefty to enforce. Thats what i meant with "a balance between popularity and reasonable-to-support for devs". BTW: i just created a thread to discuss this topic. -------------------- I am arrogant and I can afford it because I deliver.
|
|
|
|
Feb 14 2005, 22:11
Post
#24
|
|
![]() Group: Members Posts: 680 Joined: 11-July 03 From: Brno, Czech Rep. Member No.: 7705 |
i'm too for YYYY-MM-DD, it's better to have more significant part first for sorting and other stuff. and with separators you could do some safe-check, as Lyx said.
as for appending time - it's probably a good idea, but i'm not sure other users would agree on that. edit: Lyx, thanx for implementing that ost feature i've requested. there's a typo in your code though: CODE // OST-SUBCHECK $if($strcmp($get(enable_ost),1), $if($or( $strstr($lower($replace(%_path%,'.',$char(),'_',' ','-',' ','\',' ')),' OST '), $strstr($lower($replace(%album%,'.',$char(),'-',' ')),' OST ') ),$puts(albumartist,'Various Artists') )) ' OST ' should be in lowercase. thank you for your work! This post has been edited by mazy: Feb 14 2005, 23:02 -------------------- info about my tag guesser script for foo_lua (preview version available):
http://www.hydrogenaudio.org/index.php?showtopic=16987 |
|
|
|
Feb 14 2005, 23:05
Post
#25
|
|
![]() Group: Members Posts: 3353 Joined: 6-July 03 From: Sachsen (DE) Member No.: 7609 |
Thats not a typo, thats intentional. :-)
Otherwise, it could cause all kinds of false-alarms, because contrary to VA, it can be written anywhere in the foldername or albumname - its not usually positioned at the beginning. "ost" for example means "east" in german - so, a string like "ost-something" would trigger it. But when its all in caps, then its quite probably that it indeed means "soundtrack". - Lyx This post has been edited by Lyx: Feb 14 2005, 23:08 -------------------- I am arrogant and I can afford it because I deliver.
|
|
|
|
![]() ![]() |
|
Lo-Fi Version | Time is now: 24th May 2013 - 09:56 |