NSLS-II-CSX / profile_collection

BSD 3-Clause "New" or "Revised" License
0 stars 5 forks source link

FIX: these methods would have name errors #74

Closed tacaswell closed 11 months ago

ambarb commented 1 year ago

@tacaswell these are pretty recent. do we need to test or should we merge?

tacaswell commented 1 year ago

If I recall I found these via a static analysis linter and I think the code I changed can not be currently running properly so I suspect that means in is not actually used on the beamline.

It is probably safe to merge this PR, but then either tracking down how to make these code paths run on the beamline or just removing the classes (as they are not used!).