{"action":"created","issue":{"url":"https://api.github.com/repos/Qiskit/qiskit-terra/issues/2314","repository_url":"https://api.github.com/repos/Qiskit/qiskit-terra","labels_url":"https://api.github.com/repos/Qiskit/qiskit-terra/issues/2314/labels{/name}","comments_url":"https://api.github.com/repos/Qiskit/qiskit-terra/issues/2314/comments","events_url":"https://api.github.com/repos/Qiskit/qiskit-terra/issues/2314/events","html_url":"https://github.com/Qiskit/qiskit-terra/issues/2314","id":440437637,"node_id":"MDU6SXNzdWU0NDA0Mzc2Mzc=","number":2314,"title":"CouplingMap.distance does not support \"partial\" coupling maps","user":{"login":"1ucian0","id":766693,"node_id":"MDQ6VXNlcjc2NjY5Mw==","avatar_url":"https://avatars1.githubusercontent.com/u/766693?v=4","gravatar_id":"","url":"https://api.github.com/users/1ucian0","html_url":"https://github.com/1ucian0","followers_url":"https://api.github.com/users/1ucian0/followers","following_url":"https://api.github.com/users/1ucian0/following{/other_user}","gists_url":"https://api.github.com/users/1ucian0/gists{/gist_id}","starred_url":"https://api.github.com/users/1ucian0/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/1ucian0/subscriptions","organizations_url":"https://api.github.com/users/1ucian0/orgs","repos_url":"https://api.github.com/users/1ucian0/repos","events_url":"https://api.github.com/users/1ucian0/events{/privacy}","received_events_url":"https://api.github.com/users/1ucian0/received_events","type":"User","site_admin":false},"labels":[{"id":933835148,"node_id":"MDU6TGFiZWw5MzM4MzUxNDg=","url":"https://api.github.com/repos/Qiskit/qiskit-terra/labels/type:%20discussion","name":"type: discussion","color":"d4c5f9","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2019-05-05T12:11:24Z","updated_at":"2019-05-21T20:35:20Z","closed_at":null,"author_association":"MEMBER","body":"The following code does not work:\r\n```\r\ncoupling_list = [(10, 11), (10, 12), (11, 12)]\r\ncoupling = CouplingMap(coupling_list) \r\ncoupling.distance(10, 11)\r\n```\r\n\r\nI think is because it was assumed that the coupling map was always \"complete\", with the practical consequence that starts in 0. But, from after https://github.com/Qiskit/qiskit-terra/pull/2225, my reading is that we want to move away from that assumption. Is my reading correct @ajavadia ?\r\n\r\nIt seems the issue was introduced in https://github.com/Qiskit/qiskit-terra/pull/1544"},"comment":{"url":"https://api.github.com/repos/Qiskit/qiskit-terra/issues/comments/494548144","html_url":"https://github.com/Qiskit/qiskit-terra/issues/2314#issuecomment-494548144","issue_url":"https://api.github.com/repos/Qiskit/qiskit-terra/issues/2314","id":494548144,"node_id":"MDEyOklzc3VlQ29tbWVudDQ5NDU0ODE0NA==","user":{"login":"nonhermitian","id":1249193,"node_id":"MDQ6VXNlcjEyNDkxOTM=","avatar_url":"https://avatars1.githubusercontent.com/u/1249193?v=4","gravatar_id":"","url":"https://api.github.com/users/nonhermitian","html_url":"https://github.com/nonhermitian","followers_url":"https://api.github.com/users/nonhermitian/followers","following_url":"https://api.github.com/users/nonhermitian/following{/other_user}","gists_url":"https://api.github.com/users/nonhermitian/gists{/gist_id}","starred_url":"https://api.github.com/users/nonhermitian/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nonhermitian/subscriptions","organizations_url":"https://api.github.com/users/nonhermitian/orgs","repos_url":"https://api.github.com/users/nonhermitian/repos","events_url":"https://api.github.com/users/nonhermitian/events{/privacy}","received_events_url":"https://api.github.com/users/nonhermitian/received_events","type":"User","site_admin":false},"created_at":"2019-05-21T20:35:20Z","updated_at":"2019-05-21T20:35:20Z","author_association":"CONTRIBUTOR","body":"If the transpiler does not expand circuits, then much of the use of reduce goes away. However, it still has some worth for returning subgraphs that the user would otherwise have to implement by hand. I still do not like your solution for the fact that it is more backend specific, and qasm code (which we still use to store and retrieve circuits) does not support this. "}}
{"actor":{"display_login":"nonhermitian"}}