{"action":"created","issue":{"url":"https://api.github.com/repos/hyperspy/hyperspy/issues/1357","repository_url":"https://api.github.com/repos/hyperspy/hyperspy","labels_url":"https://api.github.com/repos/hyperspy/hyperspy/issues/1357/labels{/name}","comments_url":"https://api.github.com/repos/hyperspy/hyperspy/issues/1357/comments","events_url":"https://api.github.com/repos/hyperspy/hyperspy/issues/1357/events","html_url":"https://github.com/hyperspy/hyperspy/pull/1357","id":195659520,"number":1357,"title":"fix-doc: bcf warnings and add missing mapping of original_metadata","user":{"login":"sem-geologist","id":12324068,"avatar_url":"https://avatars.githubusercontent.com/u/12324068?v=3","gravatar_id":"","url":"https://api.github.com/users/sem-geologist","html_url":"https://github.com/sem-geologist","followers_url":"https://api.github.com/users/sem-geologist/followers","following_url":"https://api.github.com/users/sem-geologist/following{/other_user}","gists_url":"https://api.github.com/users/sem-geologist/gists{/gist_id}","starred_url":"https://api.github.com/users/sem-geologist/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sem-geologist/subscriptions","organizations_url":"https://api.github.com/users/sem-geologist/orgs","repos_url":"https://api.github.com/users/sem-geologist/repos","events_url":"https://api.github.com/users/sem-geologist/events{/privacy}","received_events_url":"https://api.github.com/users/sem-geologist/received_events","type":"User","site_admin":false},"labels":[{"id":262046960,"url":"https://api.github.com/repos/hyperspy/hyperspy/labels/bug-fix","name":"bug-fix","color":"e11d21","default":false},{"id":410127191,"url":"https://api.github.com/repos/hyperspy/hyperspy/labels/RELEASE_next_patch","name":"RELEASE_next_patch","color":"fef2c0","default":false},{"id":239616072,"url":"https://api.github.com/repos/hyperspy/hyperspy/labels/waiting%20for%20author","name":"waiting for author","color":"5319e7","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":27,"created_at":"2016-12-14T22:07:21Z","updated_at":"2016-12-28T12:41:50Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/hyperspy/hyperspy/pulls/1357","html_url":"https://github.com/hyperspy/hyperspy/pull/1357","diff_url":"https://github.com/hyperspy/hyperspy/pull/1357.diff","patch_url":"https://github.com/hyperspy/hyperspy/pull/1357.patch"},"body":"bcf can't save contain dead/live/real time\r\nas also fwhm. Previous behavior was that\r\nit was warning about missing dead time with every load of file.\r\nThis fix gets rid of repeated warning, and warns the user about shortcomings of\r\nbcf format only once at the first loading of bcf\r\n\r\nTasks: \r\n- [x] add note to doc about missing dead/live/real times and fwhm\r\n- [x] remove completely annoying warning from code, do not try to import dead time as it is never saved in bcf (there is logical explanation why, any producer which provide such value actually does great miss-service to his customers. The problem can be not actual for TEM, but for SEM it is very much important: Different mineral of radically diferent density will cause different counting rates so and different dead time, and there cant be universal dead time for whole image on SEM working on large scale heterogenoues samples)\r\n- [x] add most of bruker saved parameters to be available as original_metadata\r\n- [x] add some of parameters saved at the beginning of xml header (~~the problem: they can be missguiding, as it is possible to have completely different scanning averaging for imaging and mapping~~ the situation were investigated and it cleared up)\r\n- [x] add tests\r\n- [x] add calculated real_time to the metadata"},"comment":{"url":"https://api.github.com/repos/hyperspy/hyperspy/issues/comments/269472057","html_url":"https://github.com/hyperspy/hyperspy/pull/1357#issuecomment-269472057","issue_url":"https://api.github.com/repos/hyperspy/hyperspy/issues/1357","id":269472057,"user":{"login":"sem-geologist","id":12324068,"avatar_url":"https://avatars.githubusercontent.com/u/12324068?v=3","gravatar_id":"","url":"https://api.github.com/users/sem-geologist","html_url":"https://github.com/sem-geologist","followers_url":"https://api.github.com/users/sem-geologist/followers","following_url":"https://api.github.com/users/sem-geologist/following{/other_user}","gists_url":"https://api.github.com/users/sem-geologist/gists{/gist_id}","starred_url":"https://api.github.com/users/sem-geologist/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sem-geologist/subscriptions","organizations_url":"https://api.github.com/users/sem-geologist/orgs","repos_url":"https://api.github.com/users/sem-geologist/repos","events_url":"https://api.github.com/users/sem-geologist/events{/privacy}","received_events_url":"https://api.github.com/users/sem-geologist/received_events","type":"User","site_admin":false},"created_at":"2016-12-28T12:41:50Z","updated_at":"2016-12-28T12:41:50Z","body":"The issue is already known: [link_of_coverall_on_bitbucket_issues](https://bitbucket.org/ned/coveragepy/issues/541/coverage-43-breaks-nosetest-with-coverage)"}}
{"actor":{"display_login":"sem-geologist"}}