[INSTRM-591] Settle on temperature probe names for cryostat probes. Created: 11/Jan/19  Updated: 04/Oct/22  Resolved: 04/Oct/22

Status: Done
Project: Instrument control development
Component/s: ics_actorkeys
Affects Version/s: None
Fix Version/s: None

Type: Task Priority: Normal
Reporter: cloomis Assignee: cloomis
Resolution: Done Votes: 0
Labels: SM1, SPS
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Attachments: PNG File JHU-WS_temps_winchester.png    
Issue Links:
Relates
relates to INSTRM-839 Add visTemps key generation. Done
relates to INSTRM-1733 Add nirTemps actorkey Done
relates to INSTRM-1734 Generate nirTemps keyword Done
Story Points: 2

 Description   

We currently do not name the cryostat temperature probe keyword fields: they are just basically) 1..12. Various people have wondered: do we want to change that?

One complication is that the probes in the visible and NIR cryostats are not perfectly lined up. Just for the JHU probes:

Channel NIR Visible
1 Mangin Mangin
2 Mirror Cell 1 Spider Rod
3 Mirror Cell 2 Detector Box
4 SiC Spreader Thermal Spreader
5 Front Ring Barrel Front Ring Barrel
6 Spreader Pan Not Used
7 Not Used Not Used
8 Radiation Shield 1 Not Used
9 Radiation Shield 2 Not Used
10 Sidecar Not Used
11 Detector 1 Detector 1
12 Detector 2 Detector 2

[arnaud.lefur/fmadec – can you fill that in?]

 Comments   
Comment by fmadec [ 15/Jan/19 ]

 

One point, according to me we should put better names for Detector 1 and 2, because these 2 sensors are in the detector support on the nitrile plate (for the Visible if I remember well). These should not be confused with the temperature sensor of the chip itself

Comment by cloomis [ 15/Jan/19 ]

The current proposal is to replace temps with visTemps and nirTemps, each with the appropriate fields. nirTemps would have 11, and visTemps 7.

The xcu keyword dictionary would define both, but the actor would only generate the right one.

The FITS card generator will need some hackery.

Comment by arnaud.lefur [ 26/Feb/19 ]

Are you sure about the probe names ?

 

Channel Name
1 Detector Box
2 Mangin
3 Spider Rod C
4 Thermal Spreader
5 Front Ring
6 Not used
7 Not used
8 Not used
9 Not used
10 Not used
11 Detector Strap1
12 Detector Strap 2

It's what we currently have on the VISCU

Comment by Josh Peebles [ 26/Feb/19 ]

Here's what we have:

NIR

Ch1 Mangin
Ch2 Mirror Cell 1
Ch3 Mirror Cell 2
Ch4 SiC Spreader
Ch5 Front Ring Barrel
Ch6 Spreader Pan
Ch7 Not Used
Ch8 Radiation Shield 1
Ch9 Radiation Shield 2
Ch10 Sidecar
Ch11 Detector 1
Ch12 Detector 2

Vis

Ch1 Mangin
Ch2 Spider
Ch3 Detector Box
Ch4 Thermal Spreader
Ch5 Front Ring Barrel
Ch6 Not Used
Ch7 Not Used
Ch8 Not Used
Ch9 Not Used
Ch10 Not Used
Ch11 Detector 1
Ch12 Detector 2

Comment by fmadec [ 27/Feb/19 ]

This is describe in the ICD 344

https://pfs.ipmu.jp/internal/bts/show_bug.cgi?id=344

which corresponds to the table in the "Description" section.

Fabrice

Comment by Josh Peebles [ 22/Mar/19 ]

Updated ours to: (note I moved Mangin on NIR side too, so it is same channel for both)

NIR

Ch1 Mirror Cell 1
Ch2 Mangin
Ch3 Mirror Cell 2
Ch4 SiC Spreader
Ch5 Front Ring Barrel
Ch6 Spreader Pan
Ch7 Not Used
Ch8 Radiation Shield 1
Ch9 Radiation Shield 2
Ch10 Sidecar
Ch11 Detector 1
Ch12 Detector 2

Vis

Ch1 Detector Box
Ch2 Mangin
Ch3 Spider
Ch4 Thermal Spreader
Ch5 Front Ring Barrel
Ch6 Not Used
Ch7 Not Used
Ch8 Not Used
Ch9 Not Used
Ch10 Not Used
Ch11 Detector 1
Ch12 Detector 2

Comment by cloomis [ 04/Oct/22 ]

Settled a while back.

Generated at Sat Feb 10 16:26:36 JST 2024 using Jira 8.3.4#803005-sha1:1f96e09b3c60279a408a2ae47be3c745f571388b.