COMCIFS / Powder_Dictionary

CIF definitions for powder diffraction
4 stars 4 forks source link

adding phase_id and/or diffractogram_id to categories as keys #81

Closed rowlesmr closed 1 year ago

rowlesmr commented 1 year ago

as mentioned in #79

Adding phase_id and/or diffractogram_id to categories as keys to allow multiple appearances of the various categories in difference blocks in the same data collection.

rowlesmr commented 1 year ago
rowlesmr commented 1 year ago

The following already have a _pd*.detector_id as a category key (although I still need to work out how that ends up pointing to a diffractogram_id)

Maybe I can also give _*.diffractogram/phase_id to these categories, and the detector_id may be able to be found through them (just not added as category keys).

rowlesmr commented 1 year ago

When adding keys, I'm thinking

If I put this Set/Loop information in a data block all by itself, how do I get the information back into the diffractogram and/or phase of interest.

Is that the right frame of mind?

jamesrhester commented 1 year ago

When adding keys, I'm thinking

If I put this Set/Loop information in a data block all by itself, how do I get the information back into the diffractogram and/or phase of interest.

Is that the right frame of mind?

Yes, sounds OK. You can also think "would these values necessarily change if the phase/diffractogram/detector/X was different, and is that already captured via the key data names I already have".

rowlesmr commented 1 year ago

These are a little different to the other categories:

They're all Set categories, and form a hierachy (in the above order), with only PD_SPEC actually having any formal link to a diffractogram. I don't think any of them have a link to a specific phase.

In a formal sense, I think that

That seems rather complicated.

In a pragmatic sense, I think that

is possibly a way to go.

How could you then deal with an in situ experiment with many diffractograms with the same specimen (and hence, the same sample and material)?

rowlesmr commented 1 year ago

I think all the easy ones are done. The remainder, I think, need some thought.

eg PD_PEAKS: to whom do they belong? phase? diffractogram? both? what about _pd_background.peak_id?

jamesrhester commented 1 year ago

I think (after those two comments of mine are resolved) we can merge these changes, and then tackle the other categories you've raised as separate small pull requests.