{"action":"created","issue":{"url":"https://api.github.com/repos/cheton/cnc/issues/53","repository_url":"https://api.github.com/repos/cheton/cnc","labels_url":"https://api.github.com/repos/cheton/cnc/issues/53/labels{/name}","comments_url":"https://api.github.com/repos/cheton/cnc/issues/53/comments","events_url":"https://api.github.com/repos/cheton/cnc/issues/53/events","html_url":"https://github.com/cheton/cnc/issues/53","id":163320240,"number":53,"title":"Probing without applying tool offset","user":{"login":"iceboundflame","id":933687,"avatar_url":"https://avatars.githubusercontent.com/u/933687?v=3","gravatar_id":"","url":"https://api.github.com/users/iceboundflame","html_url":"https://github.com/iceboundflame","followers_url":"https://api.github.com/users/iceboundflame/followers","following_url":"https://api.github.com/users/iceboundflame/following{/other_user}","gists_url":"https://api.github.com/users/iceboundflame/gists{/gist_id}","starred_url":"https://api.github.com/users/iceboundflame/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/iceboundflame/subscriptions","organizations_url":"https://api.github.com/users/iceboundflame/orgs","repos_url":"https://api.github.com/users/iceboundflame/repos","events_url":"https://api.github.com/users/iceboundflame/events{/privacy}","received_events_url":"https://api.github.com/users/iceboundflame/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/cheton/cnc/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2016-07-01T05:06:35Z","updated_at":"2016-07-07T03:38:40Z","closed_at":"2016-07-07T03:38:40Z","body":"Hi Cheton,\r\n\r\nI use a Z probe is to set the Z zero of the WCS; the current implementation appears to:\r\n- set G54 Z=0 upon contact\r\n- apply a G43.1 tool length offset equal to the touch plate height.\r\n\r\nSince G54 coordinates are persistent across power cycles and soft resets, while G43.1 settings are lost, do you think it would be better to just issue a single command `G10 L20 P1 Z<touch plate height>`? This would be especially handy because I like to soft reset before starting jobs. It also preserves coordinates if I have to E-stop, power cycling the controller; then I don't have to re-zero or re-apply the tool length offset.\r\n\r\nDo you see any downsides to this approach?\r\n\r\nThanks!\r\nDavid"},"comment":{"url":"https://api.github.com/repos/cheton/cnc/issues/comments/230970869","html_url":"https://github.com/cheton/cnc/issues/53#issuecomment-230970869","issue_url":"https://api.github.com/repos/cheton/cnc/issues/53","id":230970869,"user":{"login":"iceboundflame","id":933687,"avatar_url":"https://avatars.githubusercontent.com/u/933687?v=3","gravatar_id":"","url":"https://api.github.com/users/iceboundflame","html_url":"https://github.com/iceboundflame","followers_url":"https://api.github.com/users/iceboundflame/followers","following_url":"https://api.github.com/users/iceboundflame/following{/other_user}","gists_url":"https://api.github.com/users/iceboundflame/gists{/gist_id}","starred_url":"https://api.github.com/users/iceboundflame/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/iceboundflame/subscriptions","organizations_url":"https://api.github.com/users/iceboundflame/orgs","repos_url":"https://api.github.com/users/iceboundflame/repos","events_url":"https://api.github.com/users/iceboundflame/events{/privacy}","received_events_url":"https://api.github.com/users/iceboundflame/received_events","type":"User","site_admin":false},"created_at":"2016-07-07T03:38:40Z","updated_at":"2016-07-07T03:38:40Z","body":"Hmmm -- I tried the above and it seems to work now. Strange. Going to close this issue since the probing function now works great. Thanks for the change!\r\n\r\nBTW - looking back, it appears I only ran the following commands the first time:\r\n```\r\nnpm install --production\r\n./bin/cnc\r\n```\r\nI didn't run `npm run prepublish`. Is that necessary? From the instructions at https://github.com/cheton/cnc/wiki/Installation#git-installation it seems like  `npm install --production` is all that is needed."}}
{"actor":{"display_login":"iceboundflame"}}