cms-gem-daq-project / gem-plotting-tools

Repository for GEM commissioning plotting tools

GNU General Public License v3.0

1 stars 26 forks source link

Bug Fix: Addresses #236 #237

Closed bdorney closed 5 years ago

bdorney commented 5 years ago

Description

Fixes #236

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Motivation and Context

236

How Has This Been Tested?

Change was trivial.

Screenshots (if appropriate):

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

jsturdy commented 5 years ago

In future, please consider the impact on other PRs awaiting merge of opening an issue, PR, and subsequently merging all within 5 minutes (especially without review/signoff)