{"action":"created","issue":{"url":"https://api.github.com/repos/MarlinFirmware/Marlin/issues/2378","labels_url":"https://api.github.com/repos/MarlinFirmware/Marlin/issues/2378/labels{/name}","comments_url":"https://api.github.com/repos/MarlinFirmware/Marlin/issues/2378/comments","events_url":"https://api.github.com/repos/MarlinFirmware/Marlin/issues/2378/events","html_url":"https://github.com/MarlinFirmware/Marlin/pull/2378","id":93019310,"number":2378,"title":"QUICKHOME for COREXY","user":{"login":"darkjavi","id":1727123,"avatar_url":"https://avatars.githubusercontent.com/u/1727123?v=3","gravatar_id":"","url":"https://api.github.com/users/darkjavi","html_url":"https://github.com/darkjavi","followers_url":"https://api.github.com/users/darkjavi/followers","following_url":"https://api.github.com/users/darkjavi/following{/other_user}","gists_url":"https://api.github.com/users/darkjavi/gists{/gist_id}","starred_url":"https://api.github.com/users/darkjavi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/darkjavi/subscriptions","organizations_url":"https://api.github.com/users/darkjavi/orgs","repos_url":"https://api.github.com/users/darkjavi/repos","events_url":"https://api.github.com/users/darkjavi/events{/privacy}","received_events_url":"https://api.github.com/users/darkjavi/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-07-04T15:01:35Z","updated_at":"2015-07-06T04:00:01Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/MarlinFirmware/Marlin/pulls/2378","html_url":"https://github.com/MarlinFirmware/Marlin/pull/2378","diff_url":"https://github.com/MarlinFirmware/Marlin/pull/2378.diff","patch_url":"https://github.com/MarlinFirmware/Marlin/pull/2378.patch"},"body":"It comes from #2255 , updated fix, rebased and branched.\r\n\r\nTrying to sum it up: On the sttepper.cpp there is an UPDATE_ENDSTOPS routine which is called in the homing process, that routine has a condition to check if the endstop is hit && the axis is mounted on is moving. That check used to be done by the step count of the stteper, which is not correct for COREXY because moving only one motor in corexy causes the head to move diagonally thus can hit either X endstop or Y endstop.\r\n\r\nFrom the tree owners of COREXY participating the thread, only 2 where exposing the problem(@darkjavi and @rugbb) , @Salandora 's  machine is apparently quickhoming correctly though.\r\n\r\nThis pr exposes dx and dy(actual axis movement) on the block struct, in the same fashion of the steps array so in the UPDATE_ENDSTOP routine can replace :  \r\n```C\r\ncurrent_block->steps[_AXIS(AXIS)]\r\n```\r\nwith\r\n```C\r\ncurrent_block->axis_movement[_AXIS(AXIS)] \r\n```\r\nin case COREXY is enabled.\r\n\r\nI tried this fix(not this particular pr yet,which is only compiled) last week and so far is working.It will be great if @ruggb, @Salandora, or any other COREXY owner could try it and give feedback about it."},"comment":{"url":"https://api.github.com/repos/MarlinFirmware/Marlin/issues/comments/118707722","html_url":"https://github.com/MarlinFirmware/Marlin/pull/2378#issuecomment-118707722","issue_url":"https://api.github.com/repos/MarlinFirmware/Marlin/issues/2378","id":118707722,"user":{"login":"Wurstnase","id":8709415,"avatar_url":"https://avatars.githubusercontent.com/u/8709415?v=3","gravatar_id":"","url":"https://api.github.com/users/Wurstnase","html_url":"https://github.com/Wurstnase","followers_url":"https://api.github.com/users/Wurstnase/followers","following_url":"https://api.github.com/users/Wurstnase/following{/other_user}","gists_url":"https://api.github.com/users/Wurstnase/gists{/gist_id}","starred_url":"https://api.github.com/users/Wurstnase/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Wurstnase/subscriptions","organizations_url":"https://api.github.com/users/Wurstnase/orgs","repos_url":"https://api.github.com/users/Wurstnase/repos","events_url":"https://api.github.com/users/Wurstnase/events{/privacy}","received_events_url":"https://api.github.com/users/Wurstnase/received_events","type":"User","site_admin":false},"created_at":"2015-07-06T04:00:01Z","updated_at":"2015-07-06T04:00:01Z","body":"The polarity isn't changed. Thisnpatch don't do anything with the endstop logics."}}