{"action":"created","issue":{"url":"https://api.github.com/repos/beetbox/beets/issues/2336","repository_url":"https://api.github.com/repos/beetbox/beets","labels_url":"https://api.github.com/repos/beetbox/beets/issues/2336/labels{/name}","comments_url":"https://api.github.com/repos/beetbox/beets/issues/2336/comments","events_url":"https://api.github.com/repos/beetbox/beets/issues/2336/events","html_url":"https://github.com/beetbox/beets/issues/2336","id":197092137,"number":2336,"title":"discogs track ids are not written after successful import","user":{"login":"welch","id":1497207,"avatar_url":"https://avatars.githubusercontent.com/u/1497207?v=3","gravatar_id":"","url":"https://api.github.com/users/welch","html_url":"https://github.com/welch","followers_url":"https://api.github.com/users/welch/followers","following_url":"https://api.github.com/users/welch/following{/other_user}","gists_url":"https://api.github.com/users/welch/gists{/gist_id}","starred_url":"https://api.github.com/users/welch/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/welch/subscriptions","organizations_url":"https://api.github.com/users/welch/orgs","repos_url":"https://api.github.com/users/welch/repos","events_url":"https://api.github.com/users/welch/events{/privacy}","received_events_url":"https://api.github.com/users/welch/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-12-22T04:26:27Z","updated_at":"2016-12-22T10:51:46Z","closed_at":null,"body":"### Problem\r\n\r\nWhile importing tracks correctly identified as \r\nhttps://www.discogs.com/The-Larry-Goldings-Trio-Moonbird/release/8913670\r\nthe import appears to succeed, yet track id's are not stored.\r\n`beet info moonbird` gives this same set of id's for each track:\r\n\r\n```\r\n    mb_albumartistid: 354487\r\n    mb_albumid: 8913670\r\n    mb_artistid: 354487\r\n    mb_releasegroupid:\r\n    mb_trackid:\r\n```\r\nthose are discogs id's, and no track has an mb_trackid, although the tracks\r\nare given their proper titles.\r\n\r\nThis came to my attention because the `duplicates` command sees all these\r\ntracks as duplicates of each other. I've done the drill of `beet rm moonbird`\r\nand reimporting their directory, behavior is consistent.\r\n\r\n### Setup\r\n\r\n* OS: ubuntu 16.04\r\n* Python version: 2.7.12\r\n* beets version: 1.4.2\r\n* Turning off plugins made problem go away (yes/no): uuuuuuhhhhh\r\n\r\nMy configuration (output of `beet config`) is:\r\n\r\n```yaml\r\n\r\nformat_item: $artist - $album - $title .$format-$bitrate\r\nduplicates:\r\n    tiebreak:\r\n        items: [added, bitrate, country]\r\n        albums: [added]\r\n    format: $artist - $album - $title .$format-$bitrate $path\r\n    count: no\r\n    full: no\r\n    keys: []\r\n    move: ''\r\n    tag: ''\r\n    path: no\r\n    copy: ''\r\n    album: no\r\n    strict: no\r\n    checksum: ''\r\n    merge: no\r\n    delete: no\r\nlibrary: /music/beets/musiclibrary.blb\r\n\r\nui:\r\n    color: yes\r\n    colors:\r\n        text_success: green\r\n        text_warning: purple\r\n        text_error: red\r\n        text_highlight: red\r\n        text_highlight_minor: lightgray\r\n        action_default: turquoise\r\n        action: blue\r\n\r\nplugins: acousticbrainz convert duplicates edit fetchart fromfilename info lastgenre play discogs\r\ndirectory: /archive/music/\r\n\r\nimport:\r\n    move: yes\r\n    duplicate_action: remove\r\n    log: /music/beets/log/flacs-2016-12-09.log\r\n    ignore_hidden: no\r\n\r\nmatch:\r\n    strong_rec_thresh: 0.15\r\nedit:\r\n    itemfields: track title artist album\r\n    albumfields: album albumartist\r\n    ignore_fields: id path\r\nacousticbrainz:\r\n    auto: yes\r\ndiscogs:\r\n    tokenfile: discogs_token.json\r\n    apikey: REDACTED\r\n    apisecret: REDACTED\r\n    source_weight: 0.5\r\nlastgenre:\r\n    count: 1\r\n    source: album\r\n    prefer_specific: no\r\n    force: yes\r\n    min_weight: 10\r\n    auto: yes\r\n    whitelist: yes\r\n    separator: ', '\r\n    fallback:\r\n    canonical: no\r\nplay:\r\n    warning_threshold: -2\r\n    use_folders: no\r\n    warning_treshold: 100\r\n    raw: no\r\n    command:\r\n    relative_to:\r\nfetchart:\r\n    auto: yes\r\n    minwidth: 0\r\n    sources:\r\n    - filesystem\r\n    - coverart\r\n    - itunes\r\n    - amazon\r\n    - albumart\r\n    google_engine: 001442825323518660753:hrh5ch1gjzm\r\n    enforce_ratio: no\r\n    cautious: no\r\n    maxwidth: 0\r\n    store_source: no\r\n    google_key: REDACTED\r\n    fanarttv_key: REDACTED\r\n    cover_names:\r\n    - cover\r\n    - front\r\n    - art\r\n    - album\r\n    - folder\r\nconvert:\r\n    never_convert_lossy_files: no\r\n    album_art_maxwidth: 0\r\n    format: mp3\r\n    dest:\r\n    auto: no\r\n    threads: 4\r\n    copy_album_art: no\r\n    tmpdir:\r\n    max_bitrate: 500\r\n\r\n    paths: {}\r\n    pretend: no\r\n    quiet: no\r\n    formats:\r\n        alac:\r\n            command: ffmpeg -i $source -y -vn -acodec alac $dest\r\n            extension: m4a\r\n        aac:\r\n            command: ffmpeg -i $source -y -vn -acodec libfaac -aq 100 $dest\r\n            extension: m4a\r\n        opus: ffmpeg -i $source -y -vn -acodec libopus -ab 96k $dest\r\n        mp3: ffmpeg -i $source -y -vn -aq 2 $dest\r\n        flac: ffmpeg -i $source -y -vn -acodec flac $dest\r\n        ogg: ffmpeg -i $source -y -vn -acodec libvorbis -aq 3 $dest\r\n        wma: ffmpeg -i $source -y -vn -acodec wmav2 -vn $dest\r\n    embed: yes\r\n```\r\n"},"comment":{"url":"https://api.github.com/repos/beetbox/beets/issues/comments/268775133","html_url":"https://github.com/beetbox/beets/issues/2336#issuecomment-268775133","issue_url":"https://api.github.com/repos/beetbox/beets/issues/2336","id":268775133,"user":{"login":"Freso","id":177659,"avatar_url":"https://avatars.githubusercontent.com/u/177659?v=3","gravatar_id":"","url":"https://api.github.com/users/Freso","html_url":"https://github.com/Freso","followers_url":"https://api.github.com/users/Freso/followers","following_url":"https://api.github.com/users/Freso/following{/other_user}","gists_url":"https://api.github.com/users/Freso/gists{/gist_id}","starred_url":"https://api.github.com/users/Freso/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Freso/subscriptions","organizations_url":"https://api.github.com/users/Freso/orgs","repos_url":"https://api.github.com/users/Freso/repos","events_url":"https://api.github.com/users/Freso/events{/privacy}","received_events_url":"https://api.github.com/users/Freso/received_events","type":"User","site_admin":false},"created_at":"2016-12-22T10:51:46Z","updated_at":"2016-12-22T10:51:46Z","body":"Having any `mb_*id` set to a Discogs id is wrong. `mb_*id` are for [MusicBrainz ids](https://musicbrainz.org/doc/MusicBrainz_Identifier). A number of applications use those ids to get additional data or for other features, so having a non-MBID in the field will break any lookups in those.\r\n\r\nIf Discogs ids are wanted, they should go into their own tags. (Also Discogs has no concept of Recordings or Tracks (or Works or …) that MusicBrainz have, so a `discogs_trackid` tag is not applicable.)"}}
{"actor":{"display_login":"Freso"}}