{"action":"created","issue":{"url":"https://api.github.com/repos/MarlinFirmware/Marlin/issues/11683","repository_url":"https://api.github.com/repos/MarlinFirmware/Marlin","labels_url":"https://api.github.com/repos/MarlinFirmware/Marlin/issues/11683/labels{/name}","comments_url":"https://api.github.com/repos/MarlinFirmware/Marlin/issues/11683/comments","events_url":"https://api.github.com/repos/MarlinFirmware/Marlin/issues/11683/events","html_url":"https://github.com/MarlinFirmware/Marlin/pull/11683","id":355987726,"node_id":"MDExOlB1bGxSZXF1ZXN0MjEyMzg4OTY1","number":11683,"title":"Fix SWITCHING_NOZZLE zdiff","user":{"login":"scott0122","id":39105828,"node_id":"MDQ6VXNlcjM5MTA1ODI4","avatar_url":"https://avatars2.githubusercontent.com/u/39105828?v=4","gravatar_id":"","url":"https://api.github.com/users/scott0122","html_url":"https://github.com/scott0122","followers_url":"https://api.github.com/users/scott0122/followers","following_url":"https://api.github.com/users/scott0122/following{/other_user}","gists_url":"https://api.github.com/users/scott0122/gists{/gist_id}","starred_url":"https://api.github.com/users/scott0122/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/scott0122/subscriptions","organizations_url":"https://api.github.com/users/scott0122/orgs","repos_url":"https://api.github.com/users/scott0122/repos","events_url":"https://api.github.com/users/scott0122/events{/privacy}","received_events_url":"https://api.github.com/users/scott0122/received_events","type":"User","site_admin":false},"labels":[{"id":196934261,"node_id":"MDU6TGFiZWwxOTY5MzQyNjE=","url":"https://api.github.com/repos/MarlinFirmware/Marlin/labels/PR:%20Bug%20Fix","name":"PR: Bug Fix","color":"bfe5bf","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2018-08-31T13:47:13Z","updated_at":"2018-09-03T05:31:48Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/MarlinFirmware/Marlin/pulls/11683","html_url":"https://github.com/MarlinFirmware/Marlin/pull/11683","diff_url":"https://github.com/MarlinFirmware/Marlin/pull/11683.diff","patch_url":"https://github.com/MarlinFirmware/Marlin/pull/11683.patch"},"body":"line 436 to 439 should delete since line 429 already add zdiff\r\n\r\nold zdiff code\r\n```cpp\r\nconst float zdiff = hotend_offset[Z_AXIS][active_extruder] - hotend_offset[Z_AXIS][tmp_extruder];\r\n```\r\nnew zdiff code\r\n```cpp\r\nzdiff = hotend_offset[Z_AXIS][tmp_extruder] - hotend_offset[Z_AXIS][active_extruder];\r\n```\r\n\r\nso `zdiff` is reversed\r\n\r\nline 429\r\n```cpp\r\ncurrent_position[Z_AXIS] += zdiff;\r\n```\r\n\r\nline 436 to 439\r\n```cpp\r\n#if ENABLED(SWITCHING_NOZZLE)\r\n  // The newly-selected extruder Z is actually at...\r\n  current_position[Z_AXIS] -= zdiff;\r\n#endif\r\n```"},"comment":{"url":"https://api.github.com/repos/MarlinFirmware/Marlin/issues/comments/418004357","html_url":"https://github.com/MarlinFirmware/Marlin/pull/11683#issuecomment-418004357","issue_url":"https://api.github.com/repos/MarlinFirmware/Marlin/issues/11683","id":418004357,"node_id":"MDEyOklzc3VlQ29tbWVudDQxODAwNDM1Nw==","user":{"login":"thinkyhead","id":698003,"node_id":"MDQ6VXNlcjY5ODAwMw==","avatar_url":"https://avatars2.githubusercontent.com/u/698003?v=4","gravatar_id":"","url":"https://api.github.com/users/thinkyhead","html_url":"https://github.com/thinkyhead","followers_url":"https://api.github.com/users/thinkyhead/followers","following_url":"https://api.github.com/users/thinkyhead/following{/other_user}","gists_url":"https://api.github.com/users/thinkyhead/gists{/gist_id}","starred_url":"https://api.github.com/users/thinkyhead/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thinkyhead/subscriptions","organizations_url":"https://api.github.com/users/thinkyhead/orgs","repos_url":"https://api.github.com/users/thinkyhead/repos","events_url":"https://api.github.com/users/thinkyhead/events{/privacy}","received_events_url":"https://api.github.com/users/thinkyhead/received_events","type":"User","site_admin":false},"created_at":"2018-09-03T05:31:48Z","updated_at":"2018-09-03T05:31:48Z","author_association":"MEMBER","body":"Have you tested it with a Z-offset applied to one or more your switching nozzles?"}}
{"actor":{"display_login":"thinkyhead"}}