xpdAcq / mission-control

Releases, Installers, Specs, and more!
0 stars 4 forks source link

Verbose name of detector object #153

Closed chiahaoliu closed 5 years ago

chiahaoliu commented 5 years ago

We might need more verbose names for the detector objects, since now both far and near detector are connected simultaneously. User could trigger the one that's not in the beam easily when they are coding their own plan...

attn: @sghose @dooryhee

Expected Behavior

Simple alias will be helpful, like

near_det = pe1c
far_det = pe2c

Current Behavior

User can trigger pe1c when far detector is in the beam

Possible solution

Assign alias (approved by the beamline staff) during startup?

CJ-Wright commented 5 years ago

@xpdAcq/bls Thoughts?

DanOlds commented 5 years ago

I'm not sure what the purpose of this is. Aren't we able to do this at the ipython level?

On Thu, Jan 17, 2019, 3:09 PM Christopher J. Wright < notifications@github.com wrote:

@xpdAcq/bls https://github.com/orgs/xpdAcq/teams/bls Thoughts?

— You are receiving this because you are on a team that was mentioned. Reply to this email directly, view it on GitHub https://github.com/xpdAcq/mission-control/issues/153#issuecomment-455314590, or mute the thread https://github.com/notifications/unsubscribe-auth/AMQJYYlP6CkdXhVzADUS7Z3VEHRkF-ABks5vENhdgaJpZM4YMCfn .

chiahaoliu commented 5 years ago

Yes. we are able to do it at ipython level easily. The hope of this discussion is to come up with persistent names (approved by beamline) that will live in the start-up sequence.

CJ-Wright commented 5 years ago

I think we can close this, right @DanOlds?

DanOlds commented 5 years ago

Super close!

On Tue, Feb 5, 2019, 10:31 AM Christopher J. Wright < notifications@github.com wrote:

I think we can close this, right @DanOlds https://github.com/DanOlds?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/xpdAcq/mission-control/issues/153#issuecomment-460681620, or mute the thread https://github.com/notifications/unsubscribe-auth/AMQJYR9YM_jLErillJke7N00u83cV37zks5vKaPYgaJpZM4YMCfn .