{"action":"created","issue":{"url":"https://api.github.com/repos/centrofermi/e3pipe/issues/86","labels_url":"https://api.github.com/repos/centrofermi/e3pipe/issues/86/labels{/name}","comments_url":"https://api.github.com/repos/centrofermi/e3pipe/issues/86/comments","events_url":"https://api.github.com/repos/centrofermi/e3pipe/issues/86/events","html_url":"https://github.com/centrofermi/e3pipe/issues/86","id":110658943,"number":86,"title":"Update the pipeline to the new analyzer","user":{"login":"lucabaldini","id":8972541,"avatar_url":"https://avatars.githubusercontent.com/u/8972541?v=3","gravatar_id":"","url":"https://api.github.com/users/lucabaldini","html_url":"https://github.com/lucabaldini","followers_url":"https://api.github.com/users/lucabaldini/followers","following_url":"https://api.github.com/users/lucabaldini/following{/other_user}","gists_url":"https://api.github.com/users/lucabaldini/gists{/gist_id}","starred_url":"https://api.github.com/users/lucabaldini/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lucabaldini/subscriptions","organizations_url":"https://api.github.com/users/lucabaldini/orgs","repos_url":"https://api.github.com/users/lucabaldini/repos","events_url":"https://api.github.com/users/lucabaldini/events{/privacy}","received_events_url":"https://api.github.com/users/lucabaldini/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-10-09T13:26:24Z","updated_at":"2015-12-01T22:43:34Z","closed_at":null,"body":"The relevant part of the code is in tasks/e3recon.py, particularly the lines\r\n\r\n    baseFilePath = e3analyzer(rawFilePath, suffix)\r\n    dstFilePath = e3dst(baseFilePath)\r\n\r\nAssuming that the root output of the new analyzer is compatible (from the prospective of the DQM) with the old one, we should just replace this with a one-line call to the new analyzer, and then proceed as before.\r\n\r\nSome questions/items for discussion:\r\n- Is the output of the new analyzer compatible with the old in the part in common?\r\n- Does it need to be run twice as the old one?\r\n- Is the new format documented somewhere? (If not we should probably replace the old page)\r\n- Do we have edit codes and are these in line with the old ones?\r\n\r\nAs we do this, it'd be nice to fix a few other things that have had issues open for a long time. I will do a list here.\r\n\r\nThe new analyzer produces an additional output file that's useful for comparison with the old analyzer. In this initial stage we want to keep it.\r\n\r\nWe want to maintain the old analyzer for the time being."},"comment":{"url":"https://api.github.com/repos/centrofermi/e3pipe/issues/comments/161120559","html_url":"https://github.com/centrofermi/e3pipe/issues/86#issuecomment-161120559","issue_url":"https://api.github.com/repos/centrofermi/e3pipe/issues/86","id":161120559,"user":{"login":"lucabaldini","id":8972541,"avatar_url":"https://avatars.githubusercontent.com/u/8972541?v=3","gravatar_id":"","url":"https://api.github.com/users/lucabaldini","html_url":"https://github.com/lucabaldini","followers_url":"https://api.github.com/users/lucabaldini/followers","following_url":"https://api.github.com/users/lucabaldini/following{/other_user}","gists_url":"https://api.github.com/users/lucabaldini/gists{/gist_id}","starred_url":"https://api.github.com/users/lucabaldini/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lucabaldini/subscriptions","organizations_url":"https://api.github.com/users/lucabaldini/orgs","repos_url":"https://api.github.com/users/lucabaldini/repos","events_url":"https://api.github.com/users/lucabaldini/events{/privacy}","received_events_url":"https://api.github.com/users/lucabaldini/received_events","type":"User","site_admin":false},"created_at":"2015-12-01T22:43:34Z","updated_at":"2015-12-01T22:43:34Z","body":"And starting with the dst:\r\n\r\n* tasks/e3recon2.py: created new task for the new analyzer + dqm\r\n* apps/e3recon2.py: created new app for the new analyzer + dqm\r\n\r\nThis fails, but at this point I can't tell whether it's because I am using an old version of the analyzer.\r\n."}}