{"action":"created","issue":{"url":"https://api.github.com/repos/sympy/sympy/issues/9476","labels_url":"https://api.github.com/repos/sympy/sympy/issues/9476/labels{/name}","comments_url":"https://api.github.com/repos/sympy/sympy/issues/9476/comments","events_url":"https://api.github.com/repos/sympy/sympy/issues/9476/events","html_url":"https://github.com/sympy/sympy/issues/9476","id":85596500,"number":9476,"title":"`M.row_insert(index)` and `M.col_insert(index)` should operate correctly for all indices","user":{"login":"gxyd","id":7712241,"avatar_url":"https://avatars.githubusercontent.com/u/7712241?v=3","gravatar_id":"","url":"https://api.github.com/users/gxyd","html_url":"https://github.com/gxyd","followers_url":"https://api.github.com/users/gxyd/followers","following_url":"https://api.github.com/users/gxyd/following{/other_user}","gists_url":"https://api.github.com/users/gxyd/gists{/gist_id}","starred_url":"https://api.github.com/users/gxyd/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gxyd/subscriptions","organizations_url":"https://api.github.com/users/gxyd/orgs","repos_url":"https://api.github.com/users/gxyd/repos","events_url":"https://api.github.com/users/gxyd/events{/privacy}","received_events_url":"https://api.github.com/users/gxyd/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-06-05T18:13:45Z","updated_at":"2015-06-06T08:23:43Z","closed_at":null,"body":"`M.row_insert(index)` should operate correctly for all indices\r\n```\r\n>>> N = Matrix([ [1, 2, 3], [2, 3, 4] ])\r\n>>> N.row_insert(-5.5, V)                                 # operates even for non-integers\r\nMatrix([ [1, 1, 1], [1, 2, 3], [2, 3, 4] ])\r\n```\r\nAccording to documentation `row_insert: Insert one or more rows at the given row position`\r\nThis operations violates it.\r\nPerhaps if it is Ok we should also look into making `row_insert` and `col_insert` in-place operation rather that returning new Matrices, #6992\r\n@toolforger perhaps the issue #6992 is the one you have been referring to."},"comment":{"url":"https://api.github.com/repos/sympy/sympy/issues/comments/109552482","html_url":"https://github.com/sympy/sympy/issues/9476#issuecomment-109552482","issue_url":"https://api.github.com/repos/sympy/sympy/issues/9476","id":109552482,"user":{"login":"toolforger","id":915906,"avatar_url":"https://avatars.githubusercontent.com/u/915906?v=3","gravatar_id":"","url":"https://api.github.com/users/toolforger","html_url":"https://github.com/toolforger","followers_url":"https://api.github.com/users/toolforger/followers","following_url":"https://api.github.com/users/toolforger/following{/other_user}","gists_url":"https://api.github.com/users/toolforger/gists{/gist_id}","starred_url":"https://api.github.com/users/toolforger/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/toolforger/subscriptions","organizations_url":"https://api.github.com/users/toolforger/orgs","repos_url":"https://api.github.com/users/toolforger/repos","events_url":"https://api.github.com/users/toolforger/events{/privacy}","received_events_url":"https://api.github.com/users/toolforger/received_events","type":"User","site_admin":false},"created_at":"2015-06-06T08:23:43Z","updated_at":"2015-06-06T08:23:43Z","body":"That would be excellent.\r\nFeel free to add it in this PR, or open a new PR for that.\r\n(The latter would be easier for us because we could sign this off as done, and wouldn't have to review it again. But it's not a biggie.)"}}