Skip to content

Conversation

@rowlesmr
Copy link
Collaborator

will close #129

PD_AMORPHOUS was insufficient to describe the same amorphous phase being present in more that one diffractogram; it could only link to peaks and diffractograms. The new category gives amorphous peak positions and intensities, and links them to actual instansiations by PD_PEAK in diffractograms.

Essentially, an amorphous phase has peak positions and intensities, widths are an implementation detail, which is the same as how crystalline phases work.

@rowlesmr
Copy link
Collaborator Author

rowlesmr commented Feb 5, 2024

Is it possible to add a method to give point-wise contributions of an amorphous phase to a diffractogram?

@rowlesmr
Copy link
Collaborator Author

rowlesmr commented Feb 5, 2024

Is it possible to add a method to give point-wise contributions of an amorphous phase to a diffractogram?

Ah! Forgot... already possible through PD_CALC_COMPONENT.

rowlesmr and others added 4 commits April 23, 2025 12:52
Co-authored-by: Antanas Vaitkus <[email protected]>
Co-authored-by: Antanas Vaitkus <[email protected]>
Co-authored-by: Antanas Vaitkus <[email protected]>
…ybe it would make sense to change this to Code as well
@rowlesmr rowlesmr requested a review from vaitkus April 23, 2025 08:12
@rowlesmr
Copy link
Collaborator Author

rowlesmr commented Jun 2, 2025

widths are an implementation detail

and also, already defined in PD_PEAK, and so could use that.

@jamesrhester
Copy link
Contributor

See my very belated comment on #129. Sorry for taking so long.

extracted from the measured or, if present, the processed diffractogram.
Each peak in this table will have a unique label
(see _pd_amorphous.peak_id).
Each peak in will have a unique label (see _pd_amorphous.id).
Copy link
Collaborator

@vaitkus vaitkus Jul 10, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a word missing after "in". Also, the _pd_amorphous.id does not seem to be defined anywhere.


The intensities given in this category must only be correct relative to each
other. In the most correct case, the intensities will represent structure
factor amplitudes, but this is not necessary
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
factor amplitudes, but this is not necessary
factor amplitudes, but this is not necessary.

loop_
_pd_amorphous_peak.id
_pd_amorphous_peak.d_spacing
_pd_amorphous_peak.rel_intensity
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This data name does not seem to be defined.

_description.text
;
Description of amorphous peak details that cannot otherwise be recorded
using other PD_AMORPHOUS data items.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
using other PD_AMORPHOUS data items.
using other PD_AMORPHOUS_PEAK data items.

_definition.update 2023-06-18
_description.text
;
List of peak_overall id values (_pd_peak_overall.id) associated with this
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
List of peak_overall id values (_pd_peak_overall.id) associated with this
List of PD_PEAK_OVERALL id values (_pd_peak_overall.id) associated with this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PD_PEAK needs diffractogram_id? and some musings on it's use in PD_AMORPHOUS

3 participants