Skip to main content

Topic: foo_discogs (Read 742586 times) previous topic - next topic

Grand Dizzy and 2 Guests are viewing this topic.
  • bubbleguuum
  • [*][*][*][*][*]
  • Developer
foo_discogs
Reply #75

your save path make it crash...$replace(%path%) evaluate to "", that's not what you want.
Put "$replace(%path%,%filename_ext%,)folder" instead.
I'll to have to prevent from malformed titleformat script though...


That works, but it names the file folder_1.jpg. How do I get it just to name it folder.jpg?


for now you can't because there might be several files...folder_2.jpg, folder_3.jpg etc if you enable fetch of all art. Just add "-$replace(%path%,%filename_ext%,)folder_1.jpg" in your foo_albumart config and that should do the trick.
  • Last Edit: 09 December, 2006, 07:36:58 PM by bubbleguuum

  • bubbleguuum
  • [*][*][*][*][*]
  • Developer
foo_discogs
Reply #76
Just released v0.26 which fixes a few bugs of 0.25

  • iOsiris
  • [*][*][*]
foo_discogs
Reply #77
Shows promise, this database even has some asian songs (very limited). But I'd just like to say thanks for making it we appreciate it!

  • hiscores
  • [*][*]
foo_discogs
Reply #78
for now you can't because there might be several files...folder_2.jpg, folder_3.jpg etc if you enable fetch of all art. Just add "-$replace(%path%,%filename_ext%,)folder_1.jpg" in your foo_albumart config and that should do the trick.


To be honest, that's kind of a pain, and not very elegant. It'd be nice to see a fix for this in an upcoming version. Thanks for the update though -- foodiscogs is amazing.

  • drbeachboy
  • [*][*][*][*][*]
foo_discogs
Reply #79
I'm getting this error message in foobar2000_v0.9.4.2:

Failed to load DLL: foo_discogs.dll
Reason: Invalid ordinal.

Any ideas on what may be causing this?
Surf's Up!
"Columnated Ruins Domino"

  • c0utta
  • [*][*]
foo_discogs
Reply #80
drbeachboy,

As per the first post, make sure that you use libcurl and zlib1 that are in the archive.  Even though I had a later version of zlib1 I had to use it, otherwise, I received "Invalid ordinal".

Cheers,

c0utta

  • mquander
  • [*][*]
foo_discogs
Reply #81
Hah, it still does it!  Tagging this release gives me:

TRACKNUMBER:  1, 2, 3, 4, 5, 6, 7
TOTALTRACKS:  "CD"


EDIT:  DISREGARD THAT, I SUCK COCKS
  • Last Edit: 10 December, 2006, 05:40:12 AM by mquander

  • orchid
  • [*][*][*]
foo_discogs
Reply #82
Thanks for the fast update

Working smoothly except in a case of multi-artist release like this http://www.discogs.com/release/630496, 'album artist' tag is not written. Didn't test this with earlier version so not sure if its intended or a bug.

Shows promise, this database even has some asian songs (very limited).


Discogs is as large as its users make it so go ahead and contribute some releases.


EDIT: Found a bug. Every time I restart foobar some mappings reset to defaults (maybe all). Tag mapping editing panel still shows the old settings
  • Last Edit: 10 December, 2006, 07:24:39 AM by orchid

  • stampgevaar
  • [*][*][*]
foo_discogs
Reply #83
Is it safe to start tagging my whole collection with your plugin or are there going to be some major changes? I like to know this because I'm not planning to retag it again when your plugin changes again cause this will take a lot of time.

  • mquander
  • [*][*]
foo_discogs
Reply #84
EDIT: Found a bug. Every time I restart foobar some mappings reset to defaults (maybe all). Tag mapping editing panel still shows the old settings


I second this bug.  It looks like you forgot to read the functional settings from the saved settings at startup.

  • bubbleguuum
  • [*][*][*][*][*]
  • Developer
foo_discogs
Reply #85
EDIT: Found a bug. Every time I restart foobar some mappings reset to defaults (maybe all). Tag mapping editing panel still shows the old settings


I second this bug.  It looks like you forgot to read the functional settings from the saved settings at startup.


Strange, redid some tests here, everything work fine. Are you able to reproduce lost settings consistently ? for example changing a setting, hit OK (or apply), quit fb2k, relaunch and you losed the setting ?

Is it safe to start tagging my whole collection with your plugin or are there going to be some major changes? I like to know this because I'm not planning to retag it again when your plugin changes again cause this will take a lot of time.


It's quite safe and there won't be incompatible changes now. It can still be considered as beta for the moment so if you're afraid of that you can wait some more.

Thanks for the fast update

Working smoothly except in a case of multi-artist release like this http://www.discogs.com/release/630496, 'album artist' tag is not written. Didn't test this with earlier version so not sure if its intended or a bug.


Yes, for now compilation is detected to 1 only if Artist = "Various" which is not the case in this release.
As there is not deterministic way to determine if a release is compilation (except the obvious "Various" case which is used for 95% of compilations), in next version I'll put a checkbox for it so you can force it.

edit: didn't see you talked about "album artist", not "compilation". For "Album Artist" this can be determined automatically
  • Last Edit: 10 December, 2006, 09:50:10 AM by bubbleguuum

  • orchid
  • [*][*][*]
foo_discogs
Reply #86
Strange, redid some tests here, everything work fine. Are you able to reproduce lost settings consistently ? for example changing a setting, hit OK (or apply), quit fb2k, relaunch and you losed the setting ?


The settings reset every time i restart foobar. However its easy to restore my own settings by opening the mapping panel and hitting OK, since my old settings are still stored there.

Could you make the matching system optional please. It messes up albums if track lengths differ from discogs

  • mquander
  • [*][*]
foo_discogs
Reply #87

Strange, redid some tests here, everything work fine. Are you able to reproduce lost settings consistently ? for example changing a setting, hit OK (or apply), quit fb2k, relaunch and you losed the setting ?


The settings reset every time i restart foobar. However its easy to restore my own settings by opening the mapping panel and hitting OK, since my old settings are still stored there.


This matches my experience.  I will try to document it in more detail later.

  • drbeachboy
  • [*][*][*][*][*]
foo_discogs
Reply #88
drbeachboy,

As per the first post, make sure that you use libcurl and zlib1 that are in the archive.  Even though I had a later version of zlib1 I had to use it, otherwise, I received "Invalid ordinal".

Cheers,

c0utta

c0utta,

Thank you! I read about libcurl, but didn't realize that it pertained to zlib1 too. I must of missed that while reading the 4 pages of posts.
Surf's Up!
"Columnated Ruins Domino"

  • bubbleguuum
  • [*][*][*][*][*]
  • Developer
foo_discogs
Reply #89
Could you make the matching system optional please. It messes up albums if track lengths differ from discogs


The matching based on track length is foobar'ed in 0.26, ie buggy, hence the messed mapping. I just corrected it and know it works as it should . Probably a good idea to be able to disable auto-matching though.

Will try to make 0.27 available tonight. Thanks to people who report bugs!

The settings reset every time i restart foobar. However its easy to restore my own settings by opening the mapping panel and hitting OK, since my old settings are still stored there.


Found it! will be corrected in 0.27
  • Last Edit: 10 December, 2006, 11:39:57 AM by bubbleguuum

  • bubbleguuum
  • [*][*][*][*][*]
  • Developer
foo_discogs
Reply #90
V0.27 is out, see first post as usual for changes. Hopefully this version will be good until next week-end!
With this version you don't need libcurl.dll/zlib1.dll anymore, if it's not used by another plugin you can delete them (be careful).

  • orchid
  • [*][*][*]
foo_discogs
Reply #91
Bugs:
- 'album artist' still not written on every release with multiple artists. (working ones 250323 338567, not working 238793 600329 576739)
- Track lengths were written to end of tracks on release 7237

Feature requests:
- Make the fields editable
- Custom button support through context menu

Thanks for your work with this. Really appreciate it

  • mquander
  • [*][*]
foo_discogs
Reply #92
Bugs:
- 'album artist' still not written on every release with multiple artists. (working ones 250323 338567, not working 238793 600329 576739)
- Track lengths were written to end of tracks on release 7237

Feature requests:
- Make the fields editable
- Custom button support through context menu

Thanks for your work with this. Really appreciate it


Discogs has got to standardize their shit, that's ridiculous.  Track lengths with periods?

  • bubbleguuum
  • [*][*][*][*][*]
  • Developer
foo_discogs
Reply #93
Bugs:
- 'album artist' still not written on every release with multiple artists. (working ones 250323 338567, not working 238793 600329 576739)


fixed. I like bugs that easy to fix

Quote
- Track lengths were written to end of tracks on release 7237

will fix it. As mquander noticed points in track length should not happen

Quote
- Make the fields editable


Any concrete example of situation you'd like to edit discogs data ?

Quote
- Custom button support through context menu


Can you explain more, I did not understand
  • Last Edit: 11 December, 2006, 07:34:42 AM by bubbleguuum

  • orchid
  • [*][*][*]
foo_discogs
Reply #94
Quote
Any concrete example of situation you'd like to edit discogs data ?

Main reason would be this new ANV system (good example http://www.discogs.com/release/338567). A variety of names for same artist messes up searches, foo_browse etc. Also Discogs is full of typos but they don't bother me that much, might someone else.

Quote
Can you explain more, I did not understand

Sorry never mind. Didn't notice adding discogs menu commands to buttons is already possible

  • mquander
  • [*][*]
foo_discogs
Reply #95
Your parser chokes on this, but I don't blame it.

  • bubbleguuum
  • [*][*][*][*][*]
  • Developer
foo_discogs
Reply #96
Quote
Main reason would be this new ANV system (good example http://www.discogs.com/release/338567). A variety of names for same artist messes up searches, foo_browse etc. Also Discogs is full of typos but they don't bother me that much, might someone else.


The parser alerady detects ANV to discard "*" appended to them, although I don't make use of this info for anything else. What I thought some time ago and would be could in the case of the artist being an ANV, let the user choose between the ANV or the Non-ANV name of the artist. Even a configuration settings to automaticcaly use the non-ANV artist name could be done. Anyway there's no harm to make the fields editable (except track names) and i'll think about it.

Now I have a question for you people. If foo_discogs could use foo_custominfo to store tags (customizable per tag) would you use that ? for which use scenario and which tags ?
I see more iconvenient to storing using foo_custominfo, especially because it's usable only with foobar and if you loose the database you loose the tags

Your parser chokes on this, but I don't blame it.


nice find! no wonder the parser goes berserk!
  • Last Edit: 11 December, 2006, 01:55:46 PM by bubbleguuum

  • mquander
  • [*][*]
foo_discogs
Reply #97
This doesn't seem to get an "Album Artist" tag.  Shouldn't it?  (Nevermind that the artists are all fake aliases of Bogdan anyway.)

  • bubbleguuum
  • [*][*][*][*][*]
  • Developer
foo_discogs
Reply #98
This doesn't seem to get an "Album Artist" tag.  Shouldn't it?  (Nevermind that the artists are all fake aliases of Bogdan anyway.)


Fixed in upcoming version. Actually you have Album Artist written if the main artist is made of several artists, or if Album="Various" or if in the tracklisting there's at least a track with two artists.  This is not the case for this release so the last test is more like "has a least a track with *one* artist"
  • Last Edit: 11 December, 2006, 03:52:50 PM by bubbleguuum

  • mquander
  • [*][*]
foo_discogs
Reply #99

This doesn't seem to get an "Album Artist" tag.  Shouldn't it?  (Nevermind that the artists are all fake aliases of Bogdan anyway.)


Fixed in upcoming version. Actually you have Album Artist written if the main artist is made of several artists, or if Album="Various" or if in the tracklisting there's at least a track with two artists.  This is not the case for this release so the last test is more like "has a least a track with *one* artist"


Aha, I understand.  Thanks for the clarification.