Skip to content
Snippets Groups Projects
  1. Feb 22, 2021
    • Robert Rosca's avatar
      Add pre-commit configuration · aceded4b
      Robert Rosca authored
      Pretty standard setup
      
      Worth mentioning that flake8 is set to only run on the diff between current
      branch and merge target
      
      This will not work outside of a merge request
      aceded4b
    • Robert Rosca's avatar
      Update CI to work with pre-commit · c5b8d367
      Robert Rosca authored
      Check stage tries to find the commit hash being merged into
      
      Uses that hash to get  a list of changed files
      
      Passes this list of files to pre-commit so that checks only run on
      modified files
      c5b8d367
    • Karim Ahmed's avatar
      Merge branch 'fix/pnccd_gain_bugs' into 'master' · 233b8f7e
      Karim Ahmed authored
      [PNCCD][GAIN] Bug fix for sending gain constants to the database
      
      See merge request detectors/pycalibration!430
      233b8f7e
  2. Feb 21, 2021
  3. Feb 18, 2021
  4. Feb 16, 2021
  5. Feb 12, 2021
  6. Feb 11, 2021
  7. Feb 10, 2021
  8. Feb 09, 2021
    • David Hammer's avatar
      Merge branch 'feat/202' into 'master' · 507adb34
      David Hammer authored
      Feat/202 (save calibration pipeline parameters in YAML file)
      
      See merge request detectors/pycalibration!409
      507adb34
    • David Hammer's avatar
      Introduce "calibration_metadata.yml" · 2168cc6f
      David Hammer authored
      This metadata file subsumes the old retrieved constants YAML file.  The commit includes some refactoring centered around pathlib.
      2168cc6f
    • Robert Rosca's avatar
      Merge branch 'fix/unused-variables' into 'master' · 9b6879f7
      Robert Rosca authored
      Fix/Unused Variables
      
      Going through some of the warnings my IDE is throwing up so I'll probably end up doing a couple of these tiny pull requests.
      
      - `cal_tools/cal_tools/agipdlib.py`: `corr_bools={}` setting keyword arguments to mutables isn't a great idea since the state can persist across function calls, maybe in this case that was the intention but it usually isn't. Changed it to `None` and then set it to a dict if no argument is given.
      - `cal_tools/cal_tools/lpdlib.py`: `self.cnl = True` was set, but it looks like that should have taken its value from the `correct_non_linear` argument, set it to that.
      - `cal_tools/cal_tools/metrology.py` an `offset` is defined but never applied to the coordinates. Similar code appears throughout the file so I just did the same thing there as well, think it should be correct.
      
      See merge request detectors/pycalibration!415
      9b6879f7
    • Robert Rosca's avatar
      bda046f3
  9. Feb 08, 2021
Loading