{"action":"created","issue":{"url":"https://api.github.com/repos/trilinos/Trilinos/issues/613","repository_url":"https://api.github.com/repos/trilinos/Trilinos","labels_url":"https://api.github.com/repos/trilinos/Trilinos/issues/613/labels{/name}","comments_url":"https://api.github.com/repos/trilinos/Trilinos/issues/613/comments","events_url":"https://api.github.com/repos/trilinos/Trilinos/issues/613/events","html_url":"https://github.com/trilinos/Trilinos/issues/613","id":177008134,"number":613,"title":"Epetra_BlockMap: preserve indexBase in ReplaceCommWithSubset","user":{"login":"mhoemmen","id":926747,"avatar_url":"https://avatars.githubusercontent.com/u/926747?v=3","gravatar_id":"","url":"https://api.github.com/users/mhoemmen","html_url":"https://github.com/mhoemmen","followers_url":"https://api.github.com/users/mhoemmen/followers","following_url":"https://api.github.com/users/mhoemmen/following{/other_user}","gists_url":"https://api.github.com/users/mhoemmen/gists{/gist_id}","starred_url":"https://api.github.com/users/mhoemmen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mhoemmen/subscriptions","organizations_url":"https://api.github.com/users/mhoemmen/orgs","repos_url":"https://api.github.com/users/mhoemmen/repos","events_url":"https://api.github.com/users/mhoemmen/events{/privacy}","received_events_url":"https://api.github.com/users/mhoemmen/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/trilinos/Trilinos/labels/Epetra","name":"Epetra","color":"eb6420"},{"url":"https://api.github.com/repos/trilinos/Trilinos/labels/in%20progress","name":"in progress","color":"ededed"}],"state":"open","locked":false,"assignee":{"login":"tawiesn","id":12903075,"avatar_url":"https://avatars.githubusercontent.com/u/12903075?v=3","gravatar_id":"","url":"https://api.github.com/users/tawiesn","html_url":"https://github.com/tawiesn","followers_url":"https://api.github.com/users/tawiesn/followers","following_url":"https://api.github.com/users/tawiesn/following{/other_user}","gists_url":"https://api.github.com/users/tawiesn/gists{/gist_id}","starred_url":"https://api.github.com/users/tawiesn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tawiesn/subscriptions","organizations_url":"https://api.github.com/users/tawiesn/orgs","repos_url":"https://api.github.com/users/tawiesn/repos","events_url":"https://api.github.com/users/tawiesn/events{/privacy}","received_events_url":"https://api.github.com/users/tawiesn/received_events","type":"User","site_admin":false},"assignees":[{"login":"tawiesn","id":12903075,"avatar_url":"https://avatars.githubusercontent.com/u/12903075?v=3","gravatar_id":"","url":"https://api.github.com/users/tawiesn","html_url":"https://github.com/tawiesn","followers_url":"https://api.github.com/users/tawiesn/followers","following_url":"https://api.github.com/users/tawiesn/following{/other_user}","gists_url":"https://api.github.com/users/tawiesn/gists{/gist_id}","starred_url":"https://api.github.com/users/tawiesn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tawiesn/subscriptions","organizations_url":"https://api.github.com/users/tawiesn/orgs","repos_url":"https://api.github.com/users/tawiesn/repos","events_url":"https://api.github.com/users/tawiesn/events{/privacy}","received_events_url":"https://api.github.com/users/tawiesn/received_events","type":"User","site_admin":false}],"milestone":null,"comments":1,"created_at":"2016-09-14T20:12:19Z","updated_at":"2016-09-14T22:04:09Z","closed_at":null,"body":"@trilinos/xpetra @trilinos/tpetra @tawiesn @csiefer2 \r\n\r\nThe index base is redundant; users can get this information from the min or min overall global index.  Furthermore, it's confusing that Epetra requires the index base to equal the min overall global index.  This causes problems with implementing subset and strided Maps.\r\n\r\nChanging this requirement on the index base might break backwards compatibility.  The easiest way to work around for now would be for SameAs to ignore the index base."},"comment":{"url":"https://api.github.com/repos/trilinos/Trilinos/issues/comments/247170484","html_url":"https://github.com/trilinos/Trilinos/issues/613#issuecomment-247170484","issue_url":"https://api.github.com/repos/trilinos/Trilinos/issues/613","id":247170484,"user":{"login":"tawiesn","id":12903075,"avatar_url":"https://avatars.githubusercontent.com/u/12903075?v=3","gravatar_id":"","url":"https://api.github.com/users/tawiesn","html_url":"https://github.com/tawiesn","followers_url":"https://api.github.com/users/tawiesn/followers","following_url":"https://api.github.com/users/tawiesn/following{/other_user}","gists_url":"https://api.github.com/users/tawiesn/gists{/gist_id}","starred_url":"https://api.github.com/users/tawiesn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tawiesn/subscriptions","organizations_url":"https://api.github.com/users/tawiesn/orgs","repos_url":"https://api.github.com/users/tawiesn/repos","events_url":"https://api.github.com/users/tawiesn/events{/privacy}","received_events_url":"https://api.github.com/users/tawiesn/received_events","type":"User","site_admin":false},"created_at":"2016-09-14T22:04:09Z","updated_at":"2016-09-14T22:04:09Z","body":"For your information: it's the same for Tpetra.\r\n@mhoemmen,@csiefer2: i propse to replace\r\n```\r\n const GO myMinGid = (this->getNodeNumElements () == 0) ?\r\n        this->getMaxAllGlobalIndex () : this->getMinGlobalIndex ();\r\n GO newIndexBase = OrdinalTraits<GO>::invalid ();\r\n reduceAll<int, GO> (*newComm, REDUCE_MIN, myMinGid, outArg (newIndexBase));\r\n return rcp (new map_type (globalNumElts, myElts, newIndexBase, newComm, node));\r\n\r\n```\r\nby\r\n`      return rcp (new map_type (globalNumElts, myElts, getIndexBase(), newComm, node));\r\n`\r\nin Tpetra::Map::replaceCommWithSubset\r\n@mhoemmen: could you test this for me? There's still this issue with my configuration which makes the Tpetra tests crash."}}
{"actor":{"display_login":"tawiesn"}}