search

NSLS-II / nslsii

NSLS-II related devices
BSD 3-Clause "New" or "Revised" License
10 stars 22 forks source link

Add transforms #78

Closed danielballan closed 4 years ago

danielballan commented 4 years ago

This is a work in progress, will be expanded as we find more transforms that we need.

codecov-io commented 4 years ago

Codecov Report

Merging #78 into master will increase coverage by 0.55%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #78      +/-   ##
==========================================
+ Coverage   52.34%   52.90%   +0.55%     
==========================================
  Files          13       13              
  Lines         724      809      +85     
==========================================
+ Hits          379      428      +49     
- Misses        345      381      +36
Impacted Files Coverage Δ
nslsii/temperature_controllers.py 55.10% <0.00%> (-36.74%) :arrow_down:
nslsii/tests/temperature_controllers_test.py 66.66% <0.00%> (-33.34%) :arrow_down:
nslsii/common/ipynb/logutils.py 92.85% <0.00%> (-7.15%) :arrow_down:
nslsii/tests/test_logutils.py 100.00% <0.00%> (ø)
nslsii/__init__.py 24.71% <0.00%> (+17.94%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update ee11542...92d0945. Read the comment docs.

gwbischof commented 4 years ago

What if we put the beamline name first? And no beamline name if it can apply to other beamlines?

danielballan commented 4 years ago

That works for me. Change pushed.