{"action":"created","issue":{"url":"https://api.github.com/repos/adafruit/Adafruit_HMC5883_Unified/issues/2","repository_url":"https://api.github.com/repos/adafruit/Adafruit_HMC5883_Unified","labels_url":"https://api.github.com/repos/adafruit/Adafruit_HMC5883_Unified/issues/2/labels{/name}","comments_url":"https://api.github.com/repos/adafruit/Adafruit_HMC5883_Unified/issues/2/comments","events_url":"https://api.github.com/repos/adafruit/Adafruit_HMC5883_Unified/issues/2/events","html_url":"https://github.com/adafruit/Adafruit_HMC5883_Unified/issues/2","id":124688382,"number":2,"title":"The \"begin\" function always returns TRUE","user":{"login":"budryerson","id":7094018,"avatar_url":"https://avatars.githubusercontent.com/u/7094018?v=3","gravatar_id":"","url":"https://api.github.com/users/budryerson","html_url":"https://github.com/budryerson","followers_url":"https://api.github.com/users/budryerson/followers","following_url":"https://api.github.com/users/budryerson/following{/other_user}","gists_url":"https://api.github.com/users/budryerson/gists{/gist_id}","starred_url":"https://api.github.com/users/budryerson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/budryerson/subscriptions","organizations_url":"https://api.github.com/users/budryerson/orgs","repos_url":"https://api.github.com/users/budryerson/repos","events_url":"https://api.github.com/users/budryerson/events{/privacy}","received_events_url":"https://api.github.com/users/budryerson/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-01-04T02:52:01Z","updated_at":"2016-11-03T18:01:00Z","closed_at":null,"body":"This function apparently means to detect whether an HMC5883 device is detected, and if no device is detected, to trigger the \"Ooops, no HMC5883 detected ... Check your wiring!\" message in the Adafruit sensor initialization example code. It does not.  As written, the function must always returns TRUE.\n\n```\nbool Adafruit_HMC5883_Unified::begin()\n{\n  // Enable I2C\n  Wire.begin();\n\n  // Enable the magnetometer\n  write8(HMC5883_ADDRESS_MAG, HMC5883_REGISTER_MAG_MR_REG_M, 0x00);\n\n  // Set the gain to a known level\n  setMagGain(HMC5883_MAGGAIN_1_3);\n\n  return true;\n}\n```\n"},"comment":{"url":"https://api.github.com/repos/adafruit/Adafruit_HMC5883_Unified/issues/comments/258224399","html_url":"https://github.com/adafruit/Adafruit_HMC5883_Unified/issues/2#issuecomment-258224399","issue_url":"https://api.github.com/repos/adafruit/Adafruit_HMC5883_Unified/issues/2","id":258224399,"user":{"login":"budryerson","id":7094018,"avatar_url":"https://avatars.githubusercontent.com/u/7094018?v=3","gravatar_id":"","url":"https://api.github.com/users/budryerson","html_url":"https://github.com/budryerson","followers_url":"https://api.github.com/users/budryerson/followers","following_url":"https://api.github.com/users/budryerson/following{/other_user}","gists_url":"https://api.github.com/users/budryerson/gists{/gist_id}","starred_url":"https://api.github.com/users/budryerson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/budryerson/subscriptions","organizations_url":"https://api.github.com/users/budryerson/orgs","repos_url":"https://api.github.com/users/budryerson/repos","events_url":"https://api.github.com/users/budryerson/events{/privacy}","received_events_url":"https://api.github.com/users/budryerson/received_events","type":"User","site_admin":false},"created_at":"2016-11-03T18:01:00Z","updated_at":"2016-11-03T18:01:00Z","body":"Thanks, Oliver.  Adafruit should add that correction to their library.\r\n\r\nI cobbled together my own library from bits and pieces of this and the Honeywell version.  In my setup, I request the ID from the device and fail if I get other than \"H43\".\r\n\r\nBut these days, everyone uses an IMU of some kind with the mag compass built in.  I don't think anyone but a masochist would use this device anymore."}}
{"actor":{"display_login":"budryerson"}}