Merge pull request #290 from XENONnT/add_coveralls

Add coveralls to travis

.travis.yml

@@ -1,16 +1,24 @@
 language: python
 dist: xenial
-python:
-- '3.6'
-- '3.7'
-- '3.8'
+jobs:
+  include:
+    - name: "Python 3.7"
+      env: PYTHON=3.7 DEPLOY_ME=true
+    - name: "Python 3.7 numbaless (for coverage)"
+      env: PYTHON=3.7 NUMBA_DISABLE_JIT=1
+    - name: "Python 3.6 (legacy)"
+      env: PYTHON=3.6
+    - name: "Python 3.8 (beta)"
+      env: PYTHON=3.8
 install:
 - pip install tensorflow>=2.3
 - git clone https://github.com/AxFoundation/strax.git ../strax
 - pip install ../strax
 - pip install -r requirements.txt
 script:
 - python setup.py test
+- pip install coveralls
+- coverage run --source=straxen setup.py test
 deploy:
   provider: pypi
   skip_existing: true
@@ -19,3 +27,8 @@ deploy:
     secure: sZFPmLKtCKvyN/HrPk2gTh8PEx28g9ksv4aBi15WGg14PUWFg53v4gEMH/DrhBc3MDrIA3ib/smMx0airn+XMBDQixwzHg1y+X1KsljOQzfk5MSC+Uv+wB+GkJW2802r00aKL851X0VU2XdjZIxTDd1DcS2QwoQjKSknRrsOtXUUAipH7JJtwKV3WcLGBUn51yygeuFHMOmtca3zlDAFEdk4W29IyAnVBj38PT8ucEWj64OIWSVJ1xf3WWdamFpNz+z6JopHUygXgyIuj8maAeSISWf8n9i2GuX8MzyumFuExklqEjcn1gcKVjmqxWdAHwvf6ul3sfjUsTbXXoPxPx1Oiz8r09H3p8lDCDwexOoh1UrQbsRGCj9oQH8e3gp8deHyosXFmIGm1DeW9efG/S/abDLQ6L9e7S5owYYFw2Muf3blJ2c6nS43k4drsS3ioP1wBsV/Z7HRI1f9NTsKpH4JwAsvIi7cr4BlTyjNip05Narq6eOp8XHSTbt/u7gfN1gfZdxwhPryRAjI+M2ck6sk6eq6nPA5DXd1B4Ekti7LwVaIBgcPCgbn2R0xaL+FrWU++6IYwhNqmnHOq9Hv3iLenGF+Mp3aTJCiyw8hvAbxqbe2vUSO6VwBQW8W8ZM3H8diU5+UaWlj+bZ0nyjxlB/arQle/6Y/WX8CHc9vBYg=
   on:
     tags: true
+
+# Upload code coverage information
+# Only do this if not jitting, otherwise we miss those tests.
+after_success:
+  - if [ $NUMBA_DISABLE_JIT ]; then coveralls; fi

README.md

@@ -5,6 +5,7 @@ Streaming analysis for XENON(nT)
 [![Readthedocs Badge](https://readthedocs.org/projects/straxen/badge/?version=latest)](https://straxen.readthedocs.io/en/latest/?badge=latest)
 [![Join the chat at https://gitter.im/AxFoundation/strax](https://badges.gitter.im/Join%20Chat.svg)](https://gitter.im/AxFoundation/strax?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge)
 [![CodeFactor](https://www.codefactor.io/repository/github/xenonnt/straxen/badge)](https://www.codefactor.io/repository/github/xenonnt/straxen)
+[![Coverage Status](https://coveralls.io/repos/github/XENONnT/straxen/badge.svg)](https://coveralls.io/github/XENONnT/straxen)
 
 Straxen is the analysis framework for XENONnT, built on top of the generic [strax framework](https://github.com/AxFoundation/strax). Currently it is configured for analyzing XENON1T data.
 

straxen/plugins/nveto_recorder.py

@@ -63,8 +63,6 @@ def infer_dtype(self):
         return {k: v for k, v in zip(self.provides, dtypes)}
 
     def compute(self, raw_records_nv, start, end):
-        strax.zero_out_of_bounds(raw_records_nv)
-
         # First we have to split rr into records and lone records:
         # Please note that we consider everything as a lone record which
         # does not satisfy the coincidence requirement

tests/test_plugins.py

@@ -55,7 +55,6 @@ def compute(self, chunk_i):
                            straxen.daqreader.DAQReader.provides
                            if p not in DummyRawRecords.provides])
 
-
 def _run_plugins(st,
                  make_all=False,
                  run_id=run_id,