GitHub PushEvent at 2014-09-22 09:15:24
Repository: pyEudetAnalysis
Actor: mathieubenoit
Payload: {"shas":[["7b2e5f6d6bb9d1f0a7c1984645a79dd7550be477","cd8c7793eb4e079d3d776af26375edc58603925e@cern.ch","Merge pull request #15 from mathieubenoit/master\n\nA few improvements and modification for compatibility with Timepix3 data","sarfaoui",true],["1c47ab31ce1af03f78da85fd7daddcd65546c082","e9320f87cc29d9b05054d3694be9541578065a1d@gmail.com","Removed unnecessary parameter from function definition.\n\nIn ToolBox.py:\n- removed \"step\" parameter from arguements, not used.","Sophie Redford",true],["fc260c3375e1131fb71d445913ff60a2fe1c338d","e9320f87cc29d9b05054d3694be9541578065a1d@gmail.com","Altered ClusterEvent to not depend on scaler.\n\nThis makes sense as we should no longer use scaler (it's not a constant).\n\nAlso modified the call to ClusterEvent in:\n- AugustAnalysis.py\n- ComputeAlignment.py\n- pyEudetAnalysis.py\n- pyEudetReconstructionOnly.py","Sophie Redford",true],["621be7dae8458b18eabd5cb40b06f091855a61ed","e9320f87cc29d9b05054d3694be9541578065a1d@gmail.com","Removed scaler from pyEudetReconstructionOnly.py.","Sophie Redford",true],["fe5b53234b61353094bd139b133ac30ed1c00fe8","e9320f87cc29d9b05054d3694be9541578065a1d@gmail.com","Added check to pyEudetReconstructionOnly.py to not recluster repeated pixel maps.\n\nInstead it just adds the clusters it found last time.\nThis makes the event loop run about twice as fast, for identical results.","Sophie Redford",true],["b1320ed8feef30dcf40b4605999dc6ad8ea7ff3a","439cb31ebe1acf39c9d77228e82f0f4b67c6e39e@cern.ch","Merge pull request #16 from sredford/master\n\nGeneral tidy of code and some speed improvements.","Mathieu Benoit",true],["65a14ce55d0618250ad440c0c5e19d0b3b3c6047","e9320f87cc29d9b05054d3694be9541578065a1d@gmail.com","Simplification and improvements to resolution plots.\n\nIn pyEudetAnalysisOnly.py:\n- added res plots for s1x1y1 and s3x2y2, previously unplotted\n- used h1_style for all res plots\n- removed the _cs series of res plots, they seem unnecessary given the others\n- moved stuff around so it comes in a more logical order\n\n- also removed unneeded scaler definition","Sophie Redford",true],["b15db23e3c3b675afdaa648e5e237e2693ad73c4","439cb31ebe1acf39c9d77228e82f0f4b67c6e39e@cern.ch","Merge pull request #17 from sredford/master\n\nSimplification and improvements to resolution plots.","Mathieu Benoit",true],["9a24b55dd52e35690cf5ba37b061d82a500d4fd0","e9320f87cc29d9b05054d3694be9541578065a1d@gmail.com","Added EtaCorrection method for 3 hit clusters.\n\nIn Cluster.py:\n- added eta correction method for L shaped 3 hit clusters\n- this improves the 3 hit cluster resolution by about a factor 2 (see plots)\n\nTo do in the future:\n- optimise the added fourth pixel tot (currently 1) to minimise the resolution.\nMay need to be a different value for different thicknesses. May need to be a\ndifferent value depending on tot of three other pixels.","Sophie Redford",true],["cbdc5eb08fe199133df982b57b434fc5c9cf0e93","e9320f87cc29d9b05054d3694be9541578065a1d@gmail.com","Added fit info to res plots.\n\nIn ComputeAlignment.py:\n- removed unneeded scaler definition\n\nIn pyEudetAnalysisOnly.py:\n- added fit information to res plots","Sophie Redford",true],["c8226238a86fa076fc34f6bbd9622fba346249ab","e9320f87cc29d9b05054d3694be9541578065a1d@gmail.com","Final version of Anne-Laure's Cluster.py code.\n\nSmall changes, comments and tidy to the existing Cluster.py code written by Anne-Laure.\nI plan to make some changes to the EtaCorrection method, and wanted to document a final\nversion of 'how it used to be'.","Sophie Redford",true],["d6bed0b1e8084863d6cae83bc8fcbeed524dac65","439cb31ebe1acf39c9d77228e82f0f4b67c6e39e@cern.ch","Merge pull request #18 from sredford/master\n\nAdded EtaCorrection method for 3 hit clusters.","Mathieu Benoit",true],["2c6c137b7cab539108905a30047cd74eb9345793","e9320f87cc29d9b05054d3694be9541578065a1d@gmail.com","Rewrite of EtaCorrection method in Cluster.py.\n\nIn Cluster.py:\n- use the possibility to call ErfInverse with a Qrel less than 0.5. The result will\nbe negative, which takes care of the direction of the shift.\n- this results in much shorter code! And easier to follow I think.\n- results in cluster centroid position are usually exactly the same. Rarely (13 times in\n14k events) there is a difference in the ~15th decimal place of the computed centroid, due\nto calling ErfInverse with a Qrel of eg 0.3 instead of 0.7. This makes no difference to\nmeasured residuals.\n- plots are the same as before\n- time the code takes to run is the same as before\n\nAlso:\n- removed duplicate code by calling existing functions\n- added a check that the centroid of the cluster is always computed","Sophie Redford",true],["b37b9c030dc11c90e6e3ef8166e0d3bc02a90495","e9320f87cc29d9b05054d3694be9541578065a1d@gmail.com","Replaced EtaCorrection method for s2x2y2 clusters with maxTOT method.\n\nIn Cluster.py:\n- replaced the EtaCorrection method with the maxTOT method for s2x2y2 clusters.\nThese are two hit clusters in which the pixels are corner to corner. I think\nmost of them are not caused by charge sharing, but rather noise / cross talk /\ndelta rays.\n\nTaking the hottest pixel as the centre improves the fitted resolution of these\nevents, see plots. However, there are more outliers, so the RMS remains about\nthe same. Nevertheless, the overall resolution with all cluster sizes does decrease\nslightly with this change.\n\nTo do in the future:\n- it might be possible to improve further by considering the Qrel of the cluster,\nand performing maxTOT for some range and EtaCorrection for the rest.","Sophie Redford",true],["9b8139aa971d684182abd063c8437144a5f2990c","439cb31ebe1acf39c9d77228e82f0f4b67c6e39e@cern.ch","Merge pull request #19 from sredford/master\n\nRewrite of EtaCorrection method, and improved resolution for s2x2y2 clusters.","Mathieu Benoit",true],["0130e4dd26379bf06bb9387810eeb21ffeeef0db","e9320f87cc29d9b05054d3694be9541578065a1d@gmail.com","Added event skip to ComputeAlignment.py, so alignment considers whole run.\n\nIn ComputeAlignment.py:\n- defined a skip to ensure that 10000 events from the requested total are considered\n- strongly recommended that the total considered is -1 (all events)\n- changed how the name of the alignment file is defined, it now contains the parameters\nused to create it\n- changed default gtol from 1e-3 to 1e-5\n- improved help text\n- tried to replace alignement with alignment everywhere\n\nIn ToolBox.py:\n- tried to tidy up comments in functions used for alignment procedure","Sophie Redford",true],["62d12c716e11eeceddbb71bce6e6178831a3756d","439cb31ebe1acf39c9d77228e82f0f4b67c6e39e@cern.ch","Merge pull request #20 from sredford/master\n\nAdded event skip to ComputeAlignment.py, so alignment considers whole ru...","Mathieu Benoit",true],["129aa3f70b41681256469401d81dedb684b84d97","ecd1f14f7c1c6dc7a40210bdcc3810e0107ecbc8@pclcd11.cern.ch","Merge with latest master","Mathieu Benoit",true]],"size":18,"ref":"refs/heads/master","head":"129aa3f70b41681256469401d81dedb684b84d97"}