{"action":"created","issue":{"url":"https://api.github.com/repos/VLSIDA/OpenRAM/issues/43","repository_url":"https://api.github.com/repos/VLSIDA/OpenRAM","labels_url":"https://api.github.com/repos/VLSIDA/OpenRAM/issues/43/labels{/name}","comments_url":"https://api.github.com/repos/VLSIDA/OpenRAM/issues/43/comments","events_url":"https://api.github.com/repos/VLSIDA/OpenRAM/issues/43/events","html_url":"https://github.com/VLSIDA/OpenRAM/issues/43","id":385582325,"node_id":"MDU6SXNzdWUzODU1ODIzMjU=","number":43,"title":"Logical Error in Hierarchical Decoder","user":{"login":"lekez2005","id":4502461,"node_id":"MDQ6VXNlcjQ1MDI0NjE=","avatar_url":"https://avatars2.githubusercontent.com/u/4502461?v=4","gravatar_id":"","url":"https://api.github.com/users/lekez2005","html_url":"https://github.com/lekez2005","followers_url":"https://api.github.com/users/lekez2005/followers","following_url":"https://api.github.com/users/lekez2005/following{/other_user}","gists_url":"https://api.github.com/users/lekez2005/gists{/gist_id}","starred_url":"https://api.github.com/users/lekez2005/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lekez2005/subscriptions","organizations_url":"https://api.github.com/users/lekez2005/orgs","repos_url":"https://api.github.com/users/lekez2005/repos","events_url":"https://api.github.com/users/lekez2005/events{/privacy}","received_events_url":"https://api.github.com/users/lekez2005/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2018-11-29T06:00:17Z","updated_at":"2018-11-29T17:23:18Z","closed_at":"2018-11-29T17:23:18Z","author_association":"NONE","body":"The hierarchical decoder is incorrect in terms of mapping an input address to the correct row. For example, for a 64 row decoder, address 1 mapped to row 16 and address 32 mapped to row 2. Unfortunately, my current decoder has widely diverged from the existing implementation so I can't generate a good enough clean patch.\r\n [This gist](https://gist.github.com/lekez2005/2a1f61a6fd873fd4d14a2c744a90c7a3) shows my patch for fixing the issue. At the very least, whatever fix for this issue should pass the included tests. [This gist](https://gist.github.com/lekez2005/2b40bd5445e0e735a3c80a7f7c94b083) shows the full decoder for reference. "},"comment":{"url":"https://api.github.com/repos/VLSIDA/OpenRAM/issues/comments/442919934","html_url":"https://github.com/VLSIDA/OpenRAM/issues/43#issuecomment-442919934","issue_url":"https://api.github.com/repos/VLSIDA/OpenRAM/issues/43","id":442919934,"node_id":"MDEyOklzc3VlQ29tbWVudDQ0MjkxOTkzNA==","user":{"login":"mguthaus","id":12450095,"node_id":"MDQ6VXNlcjEyNDUwMDk1","avatar_url":"https://avatars3.githubusercontent.com/u/12450095?v=4","gravatar_id":"","url":"https://api.github.com/users/mguthaus","html_url":"https://github.com/mguthaus","followers_url":"https://api.github.com/users/mguthaus/followers","following_url":"https://api.github.com/users/mguthaus/following{/other_user}","gists_url":"https://api.github.com/users/mguthaus/gists{/gist_id}","starred_url":"https://api.github.com/users/mguthaus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mguthaus/subscriptions","organizations_url":"https://api.github.com/users/mguthaus/orgs","repos_url":"https://api.github.com/users/mguthaus/repos","events_url":"https://api.github.com/users/mguthaus/events{/privacy}","received_events_url":"https://api.github.com/users/mguthaus/received_events","type":"User","site_admin":false},"created_at":"2018-11-29T17:23:18Z","updated_at":"2018-11-29T17:23:18Z","author_association":"COLLABORATOR","body":"Resolved in upcoming release."}}
{"actor":{"display_login":"mguthaus"}}