-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PD_INSTR
and PD_INSTR_DETECTOR
extension
#157
Comments
As far as I can tell, multi-pixel detector geometry is completely covered by the imgCIF dictionary, including laying out all of the relevant axes. Once our raw powder data working group starts up as part of an IUCr journals project, we'll have a better idea of what is missing and we can feed back that information here. Mirrors are an excellent idea. We should be able to recreate a geometrical layout of the instrument suitable for simulation calculations. |
Moved from #162
That's not going to work, as . First pass at something (all names are probably terrible, and there is at least one thing @jamesrhester doesn't like*): The inner details could also be hidden, a la how
Dataitems:
The enumeration values of
A similar thing would apply for Examples:
|
OK. So some comments.
But actually it's not that far off being pretty fit for purpose. The only danger is that we'll end up defining a whole separate dictionary that is useful for general beamline description that "everyone" will want to use. |
This is the bit I couldn't figure out how to do without having essentially a diagonal loop, and why I went with the evil solution. Given a component ordering:
How to assign a value to the component and associate it with its order? One way is to have one big loop
But it is nearly diagonal, wastes a lot of space and is hard to write and read. The second way is to have many categories:
which has it's own problems in terms of managing the number of loops. |
The many categories approach is preferable, because it is robust to extension. Conceivably these categories could expand in the future with geometric descriptions, manufacturing details, whatever. There is no harm in having many categories, and that just reflects the fact that these are all different things with different needs. We could consider placing all slits into a single category, though. |
This could very quickly lead down the path of a thing for a separate dictionary! |
Yep. At which point one asks oneself what is most important to get done first. |
Yeah, nah. |
There needs at least parallel and elliptical focussing mirrors, as well as more dataitems to describe the detector
mirrors
PD_INSTR
has_pd_instr.2theta_monochr_pre
andPD_INSTR_DETECTOR
has_pd_instr.2theta_monochr_post
which have the value of the mono angle.There could be
_pd_instr.2theta_mirror_pre
and_pd_instr.2theta_mirror_post
, which takes the value 'parallel' or 'focussing', where it is assumed that the focus is on the detector (or source for a diffracted beam focussing mirror, whatever that means...).An alternate value is to give the distance of the focal point from the optic; parallel == 0, and focussing == some finite value. This allows for mirrors that don't focus on the detector.
Detectors
There needs to be more dataitems to describe strip detectors. These are small, linear position-sensitive detectors which have ~100-300 channels. The channels are normally opaque to the user, and you just get one, pre-made diffraction pattern, rather than a bunch you need to merge.
Suggestions:
_pd_instr.ax_size_detc
and_pd_instr.eq_size_detc
_pd_instr.dist_spec_detc
, so it should be possible to interconvert. There is potential perceived overlap with_pd_instr.slit_ax|eq_spec_detc
: this data item is the slit width in mm defining the collimation in the axial/equatorial direction between the specimen and detector. In the event of a linear detector oriented normally (ie strips running up in 2Th),_pd_instr.eq_size_detc
is exactly analagous with_pd_instr.slit_eq_spec_detc
_pd_instr.ax_pixels_detc
and_pd_instr.eq_pixels_detc
(or maybechannels
instead ofpixels
)_pd_instr.ax_pixels_detc 200ish
and_pd_instr.eq_pixels_detc 1
_pd_instr.detc_type
point
,linear
,curved_linear
, andarea
.point
is_pd_instr.ax_pixels_detc 1
and_pd_instr.eq_pixels_detc 1
linear
has one of_pd_instr.ax|eq_pixels_detc 1
and the other_pd_instr.eq|ax_pixels_detc >1
and is physically straight.curved_linear
has one of_pd_instr.ax|eq_pixels_detc 1
and the other_pd_instr.eq|ax_pixels_detc >1
and is curved in the direction of the larger_pd_instr.ax|eq_size_detc
dimension such that the spec-detc distance is constant.area
has both_pd_instr.ax_pixels_detc >1
and_pd_instr.eq_pixels_detc >1
and is physically flat.The text was updated successfully, but these errors were encountered: