{"action":"created","issue":{"url":"https://api.github.com/repos/MarlinFirmware/Marlin/issues/9617","repository_url":"https://api.github.com/repos/MarlinFirmware/Marlin","labels_url":"https://api.github.com/repos/MarlinFirmware/Marlin/issues/9617/labels{/name}","comments_url":"https://api.github.com/repos/MarlinFirmware/Marlin/issues/9617/comments","events_url":"https://api.github.com/repos/MarlinFirmware/Marlin/issues/9617/events","html_url":"https://github.com/MarlinFirmware/Marlin/issues/9617","id":296644128,"number":9617,"title":"WHO IS THE GUY THAT HAVE MADE THE PR FOR THE ACTUAL PAUSE SYSTEM","user":{"login":"studiodyne","id":24499680,"avatar_url":"https://avatars0.githubusercontent.com/u/24499680?v=4","gravatar_id":"","url":"https://api.github.com/users/studiodyne","html_url":"https://github.com/studiodyne","followers_url":"https://api.github.com/users/studiodyne/followers","following_url":"https://api.github.com/users/studiodyne/following{/other_user}","gists_url":"https://api.github.com/users/studiodyne/gists{/gist_id}","starred_url":"https://api.github.com/users/studiodyne/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/studiodyne/subscriptions","organizations_url":"https://api.github.com/users/studiodyne/orgs","repos_url":"https://api.github.com/users/studiodyne/repos","events_url":"https://api.github.com/users/studiodyne/events{/privacy}","received_events_url":"https://api.github.com/users/studiodyne/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2018-02-13T07:59:13Z","updated_at":"2018-02-14T14:08:00Z","closed_at":"2018-02-14T12:43:23Z","author_association":"CONTRIBUTOR","body":"**PLEASE STOP YOUR ETERNAL MODIFICATION I'M REALLY TIRED !!!!!!!!!\r\n\r\nNot tested , not finished ,  and MADE ONLY FOR YOUR MACHINE\r\n\r\nEvery day something is modified  , and now pause system IS SHIT\r\n\r\nYOU CAN NOT MODIFY ETERNALLY , because we built our machine and we need A FIX PSYCHOLOGY \r\n\r\nYou change the names every weeks , You change all \r\n\r\nAND NOT TESTED NOT VERIFIED**\r\n\r\n@thinkyhead   please stop to merge pause system , before asking me to check  , looks the tons of shit , i don't know who is the guy that write this but , since one year , every week i have modifications , eternally modifications , and it's enough now , i need progression , no regression \r\n\r\nFirst : \r\n```\r\n/**\r\n * Unload filament from the hotend\r\n *\r\n * - Fail if the a safe temperature was not reached\r\n * - Show \"wait for unload\" placard\r\n * - Retract, pause, then unload filament\r\n * - Disable E stepper (on most machines)\r\n *\r\n * Returns 'true' if unload was completed, 'false' for abort\r\n */\r\n```\r\nThis code says '  retract , pause to cool and unload ' ,   HERE I UNDERSTAND  the filament is cooled to avoid to dirty the gear when go out and look the bad syntax in the description \r\nBut later in the code \r\n```\r\n  // Filament Unload does a Retract, Delay, and Purge first:\r\n  #define FILAMENT_UNLOAD_RETRACT_LENGTH 13   // (mm) Unload initial retract length.\r\n  #define FILAMENT_UNLOAD_DELAY 5000          // (ms) Delay for the filament to cool after retract.\r\n  #define FILAMENT_UNLOAD_PURGE_LENGTH 8      // (mm) An unretract is done, then this length is purged.\r\n```\r\n Now it's impossible to understand because NOW  WE PURGE after RETRACT , and we don't UNLOAD, and we have purge length value ' why '\r\n\r\nand later in the code \r\n\r\n```\r\n// Retract filament\r\n  do_pause_e_move(-FILAMENT_UNLOAD_RETRACT_LENGTH, PAUSE_PARK_RETRACT_FEEDRATE);\r\n\r\n  // Wait for filament to cool\r\n  safe_delay(FILAMENT_UNLOAD_DELAY);\r\n\r\n  // Quickly purge\r\n  do_pause_e_move(FILAMENT_UNLOAD_RETRACT_LENGTH + FILAMENT_UNLOAD_PURGE_LENGTH, planner.max_feedrate_mm_s[E_AXIS]);\r\n\r\n  // Unload filament\r\n  do_pause_e_move(unload_length, FILAMENT_CHANGE_UNLOAD_FEEDRATE);\r\n```\r\nHere we have  PURGE  + UNLOAD , I DON T UNDERSTANT , and the worst is the MAX FEEDRATE that can explodes the filament or  sand it , in the pause definition  we have defined a FEEDRATE for unload , why not used here , this CODE IS SHITTTTTTTTTTTTTTTTTTTTTTTTT ! We retract , we cool , and we return in the hotend , and we unload ,  it's the fifth dimensions here !\r\n\r\nIT S NOT SERIOUS , i'm tired  to see eternal VERY BAD modification OF OUR PRECIOUS PAUSING PROCESS\r\n\r\nAnd stop to change the name  ETERNALLY , because , just for one or two words , we can not drop our config and configadv to test new release , i'm ok for deep modification , but  just for one new modified word , the unit is dead\r\n\r\nPLEASE DOWNGRADE !\r\nThe thinkyHead Punition requiered !\r\nSome guys here are too much happy to contribute and forget to give priority to the quality instead of to be merged the more quickly as possible to feel good and feel ' great engineer'"},"comment":{"url":"https://api.github.com/repos/MarlinFirmware/Marlin/issues/comments/365617383","html_url":"https://github.com/MarlinFirmware/Marlin/issues/9617#issuecomment-365617383","issue_url":"https://api.github.com/repos/MarlinFirmware/Marlin/issues/9617","id":365617383,"user":{"login":"studiodyne","id":24499680,"avatar_url":"https://avatars0.githubusercontent.com/u/24499680?v=4","gravatar_id":"","url":"https://api.github.com/users/studiodyne","html_url":"https://github.com/studiodyne","followers_url":"https://api.github.com/users/studiodyne/followers","following_url":"https://api.github.com/users/studiodyne/following{/other_user}","gists_url":"https://api.github.com/users/studiodyne/gists{/gist_id}","starred_url":"https://api.github.com/users/studiodyne/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/studiodyne/subscriptions","organizations_url":"https://api.github.com/users/studiodyne/orgs","repos_url":"https://api.github.com/users/studiodyne/repos","events_url":"https://api.github.com/users/studiodyne/events{/privacy}","received_events_url":"https://api.github.com/users/studiodyne/received_events","type":"User","site_admin":false},"created_at":"2018-02-14T14:08:00Z","updated_at":"2018-02-14T14:08:00Z","author_association":"CONTRIBUTOR","body":"@AnHardt  I totally don't care what you always says , i hate your style of communication and i never read all you say\r\n@Thinkyhead , i love your style , your brain , this extraordinary brain , and every day when i use marlin , this magic firmware that is truely a great advance firmware , babystep/ubl/Fw , i think about you , and the chance to have you as boss here, and the time you spent everyday FOR ME , and all people of this world , and all the companies that love marlin. \r\nBut i'm not engineer like you , but , i know when SHIT IS DONE !!!!!!!!!!!! and i know the last PR is not thinkyheadstyle , you trust too quickly some guys and you merge them . The last pause system is modified everyweek , it's too much ! Names are changed all the time ! Impossible to try a new release just because of a little name. What a surprise when i would help roxy to find the fix for ubl strange behavior , to see what is became the config adv ... changed again , and again , and always THE PAUSE SYSTEM !\r\nSTOP !!! too much ! And i was afraid to try it. I know it's not you , because , you don't have this writting style , but you trust too much some bad writters.\r\nIn all i have said about this topic , you will never the target of my words , only the guys you trust too much !\r\n@AnHardt  : do not answer me , i will never read\r\n"}}
{"actor":{"display_login":"studiodyne"}}