{"action":"created","issue":{"url":"https://api.github.com/repos/boris-kz/CogAlg/issues/14","repository_url":"https://api.github.com/repos/boris-kz/CogAlg","labels_url":"https://api.github.com/repos/boris-kz/CogAlg/issues/14/labels{/name}","comments_url":"https://api.github.com/repos/boris-kz/CogAlg/issues/14/comments","events_url":"https://api.github.com/repos/boris-kz/CogAlg/issues/14/events","html_url":"https://github.com/boris-kz/CogAlg/issues/14","id":368035045,"node_id":"MDU6SXNzdWUzNjgwMzUwNDU=","number":14,"title":"Assignment by reference may cause overwritten values from aliased copies","user":{"login":"Twenkid","id":23367640,"node_id":"MDQ6VXNlcjIzMzY3NjQw","avatar_url":"https://avatars2.githubusercontent.com/u/23367640?v=4","gravatar_id":"","url":"https://api.github.com/users/Twenkid","html_url":"https://github.com/Twenkid","followers_url":"https://api.github.com/users/Twenkid/followers","following_url":"https://api.github.com/users/Twenkid/following{/other_user}","gists_url":"https://api.github.com/users/Twenkid/gists{/gist_id}","starred_url":"https://api.github.com/users/Twenkid/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Twenkid/subscriptions","organizations_url":"https://api.github.com/users/Twenkid/orgs","repos_url":"https://api.github.com/users/Twenkid/repos","events_url":"https://api.github.com/users/Twenkid/events{/privacy}","received_events_url":"https://api.github.com/users/Twenkid/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2018-10-09T04:19:44Z","updated_at":"2018-10-11T13:19:01Z","closed_at":null,"author_association":"COLLABORATOR","body":"https://github.com/boris-kz/CogAlg/blob/527a66491b03067cc194795d8e476ed33f417b3c/frame_dblobs.py#L146\r\n\r\nI can't run or track it now and don't know if that's the case but that's a common source of  bugs. The initial value that is correct may be overwritten by an aliased  copy of the same reference by  reassignments. \r\n"},"comment":{"url":"https://api.github.com/repos/boris-kz/CogAlg/issues/comments/428951924","html_url":"https://github.com/boris-kz/CogAlg/issues/14#issuecomment-428951924","issue_url":"https://api.github.com/repos/boris-kz/CogAlg/issues/14","id":428951924,"node_id":"MDEyOklzc3VlQ29tbWVudDQyODk1MTkyNA==","user":{"login":"boris-kz","id":20977433,"node_id":"MDQ6VXNlcjIwOTc3NDMz","avatar_url":"https://avatars0.githubusercontent.com/u/20977433?v=4","gravatar_id":"","url":"https://api.github.com/users/boris-kz","html_url":"https://github.com/boris-kz","followers_url":"https://api.github.com/users/boris-kz/followers","following_url":"https://api.github.com/users/boris-kz/following{/other_user}","gists_url":"https://api.github.com/users/boris-kz/gists{/gist_id}","starred_url":"https://api.github.com/users/boris-kz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/boris-kz/subscriptions","organizations_url":"https://api.github.com/users/boris-kz/orgs","repos_url":"https://api.github.com/users/boris-kz/repos","events_url":"https://api.github.com/users/boris-kz/events{/privacy}","received_events_url":"https://api.github.com/users/boris-kz/received_events","type":"User","site_admin":false},"created_at":"2018-10-11T13:19:01Z","updated_at":"2018-10-11T13:19:01Z","author_association":"OWNER","body":"Thanks, I thought \"if len_(hP) == 6\" would prevent these marginal cases,\nbut the len mismatch was actually in the first tuple of hP.\nI fixed it in new update.\nStill, there is a bug that seems to increment roots and fork_ somewhere\noutside of scan_P_.\nI have cases with roots == 1 at y==5 but not greater. Which is weird\nbecause y shouldn't matter, roots are initialized at 0 on each line, in\nln188.\n\nI will look into tracking, but right now I am focused on basic principles,\nmainly how to define projected match through its ratios.\n\n\nOn Thu, Oct 11, 2018 at 7:37 AM Todor Arnaudov <notifications@github.com>\nwrote:\n\n> I wrote some code to track ids and see these forks.\n>\n> https://github.com/Twenkid/CogAlg/blob/master/frame_dblobs_track.py\n>\n> You said it runs, but in my setup it breaks at:\n>\n>\n> After about 6800 logged forks (different branches hit):\n>\n> python f3.py\n> too many values to unpack (expected 7)\n> <class 'ValueError'>\n> too many values to unpack (expected 7)\n>   File \"f3.py\", line 264, in scan_P_\n>     hP, frame = form_blob(hP, frame)  # blob (all connected blob segments) += blob segment at hP\n>   File \"f3.py\", line 309, in form_blob\n>     (s, Ls, Is, Ds, Dys, Vs, Vys), Py_, x, xd, root, fork_ = seg  # s, root are ignored\n> ForkLog 6768\n>\n> The last logged line is that branch:\n>\n> 6767: if roots == 0 id=2109825995208,x=1024,_x=1019, y=410 *fork*=#>0#\n>\n> That's out of bounds (1023)?\n>\n> ...\n>\n> Two logs:\n>\n> http://twenkid.com/cog/\n>\n> Of the _fork_at the respective branches in scan_P_, one file with the\n> content of the forks, one which only shows if they're empty [] or #>0#.\n>\n> The simplest way to track how given fork grows is by a code editor such as\n> Notepad++, also a browser, select a number or Ctrl+F and it will mark the\n> other lines with it.\n>\n> As can be seen from the log with the contents (big file), the fork content\n> grows rapidly after the period of [].\n>\n> I may write something that shows all occasions of each unique id.\n>\n> [image: f1]\n> <https://user-images.githubusercontent.com/23367640/46800644-02f60700-cd61-11e8-9c04-e9dcfc9dc2e6.png>\n> [image: f2]\n> <https://user-images.githubusercontent.com/23367640/46800646-02f60700-cd61-11e8-9950-c03da66215a7.png>\n> [image: f3]\n> <https://user-images.githubusercontent.com/23367640/46800812-844d9980-cd61-11e8-812f-128f9178bf14.png>\n>\n> —\n> You are receiving this because you commented.\n> Reply to this email directly, view it on GitHub\n> <https://github.com/boris-kz/CogAlg/issues/14#issuecomment-428922633>, or mute\n> the thread\n> <https://github.com/notifications/unsubscribe-auth/AUAXGRGwvDzCJ4SsX4Q3wrwqAc6bzszgks5ujy1tgaJpZM4XQktA>\n> .\n>\n"}}
{"actor":{"display_login":"boris-kz"}}