IUPAC / WFChemCookbook

The IUPAC WorldFAIR Cookbook for FAIR chemical data
https://iupac.github.io/WFChemCookbook
Creative Commons Attribution 4.0 International
14 stars 4 forks source link

Review of 'Structured UV/Vis (meta)data for small molecule characterisation' #15

Closed stuchalk closed 7 months ago

stuchalk commented 1 year ago

Which page are you reviewing? https://iupac.github.io/WFChemCookbook/samples/uvvis_metadata.html

What is your ORCID https://orcid.org/0000-0002-0703-7776

Review This recipe outlines metadata to be included in the reporting of UVVIS spectra and includes a Python script for processing of a spectrum and generation of characteristic metadata. The recipe is comprehensive in its description and I like the organization of the different metadata levels that helps readers think about the different stages of the process. I particularly like the 'Doc string' text in the function which is both educational about Beer's law and also exemplifies best practice in documenting Python code.

Suggestions:

S-Bloodworth commented 1 year ago

Corrections have been made. All - please add any further comments. We'll close this issue in due course if there are no further suggestions.

stuchalk commented 1 year ago

Have the changes been pushed to GitHub? I am not seeing a commit on this... (hopefully I am not blind...).

Smund27 commented 1 year ago

All changes have been pushed now

stuchalk commented 1 year ago

Thanks, Sam. I pushed the changes this morning.