{"action":"created","issue":{"url":"https://api.github.com/repos/openPMD/openPMD-api/issues/368","repository_url":"https://api.github.com/repos/openPMD/openPMD-api","labels_url":"https://api.github.com/repos/openPMD/openPMD-api/issues/368/labels{/name}","comments_url":"https://api.github.com/repos/openPMD/openPMD-api/issues/368/comments","events_url":"https://api.github.com/repos/openPMD/openPMD-api/issues/368/events","html_url":"https://github.com/openPMD/openPMD-api/issues/368","id":362561424,"node_id":"MDU6SXNzdWUzNjI1NjE0MjQ=","number":368,"title":"storeChunk argument order","user":{"login":"ax3l","id":1353258,"node_id":"MDQ6VXNlcjEzNTMyNTg=","avatar_url":"https://avatars3.githubusercontent.com/u/1353258?v=4","gravatar_id":"","url":"https://api.github.com/users/ax3l","html_url":"https://github.com/ax3l","followers_url":"https://api.github.com/users/ax3l/followers","following_url":"https://api.github.com/users/ax3l/following{/other_user}","gists_url":"https://api.github.com/users/ax3l/gists{/gist_id}","starred_url":"https://api.github.com/users/ax3l/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ax3l/subscriptions","organizations_url":"https://api.github.com/users/ax3l/orgs","repos_url":"https://api.github.com/users/ax3l/repos","events_url":"https://api.github.com/users/ax3l/events{/privacy}","received_events_url":"https://api.github.com/users/ax3l/received_events","type":"User","site_admin":false},"labels":[{"id":805165679,"node_id":"MDU6TGFiZWw4MDUxNjU2Nzk=","url":"https://api.github.com/repos/openPMD/openPMD-api/labels/api:%20breaking","name":"api: breaking","color":"1d76db","default":false},{"id":805221066,"node_id":"MDU6TGFiZWw4MDUyMjEwNjY=","url":"https://api.github.com/repos/openPMD/openPMD-api/labels/frontend:%20C++11","name":"frontend: C++11","color":"c5def5","default":false},{"id":805221139,"node_id":"MDU6TGFiZWw4MDUyMjExMzk=","url":"https://api.github.com/repos/openPMD/openPMD-api/labels/frontend:%20Python3","name":"frontend: Python3","color":"c5def5","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-09-21T10:51:23Z","updated_at":"2018-09-25T09:03:05Z","closed_at":null,"author_association":"MEMBER","body":"The order of `storeChunk`/`store_chunk` should be changed to allow useful defaults to:\r\n- `data`\r\n- `offset`\r\n- `extent`\r\n\r\nor\r\n- `data`\r\n- `extent`\r\n- `offset` = n-D {0}\r\n\r\nFor example `extent` can be derived from `data` if a container is given (needs more overloads in C++, e.g. if we add xtensor later on but also for `std::vector` useful) but also in the Python interface where we naturally work with the buffer protocol.\r\n\r\nNot sure if `extent` or `offset` should be last. My thoughts are: in serial I/O where one most of the time only writes a single chunk, this does not matter much since both can be defaulted. In parallel I/O, when using the overloads described in the future, one could only default for extent, so the order is `data`, `offset`, `extent` = default is probably best."},"comment":{"url":"https://api.github.com/repos/openPMD/openPMD-api/issues/comments/424262817","html_url":"https://github.com/openPMD/openPMD-api/issues/368#issuecomment-424262817","issue_url":"https://api.github.com/repos/openPMD/openPMD-api/issues/368","id":424262817,"node_id":"MDEyOklzc3VlQ29tbWVudDQyNDI2MjgxNw==","user":{"login":"ax3l","id":1353258,"node_id":"MDQ6VXNlcjEzNTMyNTg=","avatar_url":"https://avatars3.githubusercontent.com/u/1353258?v=4","gravatar_id":"","url":"https://api.github.com/users/ax3l","html_url":"https://github.com/ax3l","followers_url":"https://api.github.com/users/ax3l/followers","following_url":"https://api.github.com/users/ax3l/following{/other_user}","gists_url":"https://api.github.com/users/ax3l/gists{/gist_id}","starred_url":"https://api.github.com/users/ax3l/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ax3l/subscriptions","organizations_url":"https://api.github.com/users/ax3l/orgs","repos_url":"https://api.github.com/users/ax3l/repos","events_url":"https://api.github.com/users/ax3l/events{/privacy}","received_events_url":"https://api.github.com/users/ax3l/received_events","type":"User","site_admin":false},"created_at":"2018-09-25T09:03:05Z","updated_at":"2018-09-25T09:03:05Z","author_association":"MEMBER","body":"Since this is a breaking change that can fit the signature in some cases with unwanted side-effects, we could also think about renaming the methods. Since the `store` methods just \"register\" arrays until flush is called... dunno, we had the same naming discussion a few months back with ADIOS2 devs."}}
{"actor":{"display_login":"ax3l"}}