Option to show Interim fields on COA


#1

Hello everyone, since interim fields are not shown on the COA(Certificate of Analysis). I would to ask if it is ok to add an option for them to be shown on the COA. @lemoene suggested we add a boolean Report field on the interim fields as shown on the images below

or should this functionality be in an add-on?


#2

Hi Lunga,

I would like to have interim fields as well in the report to overcome the shortcoming of having only one result.
So I think having that checkbox is a good idea and gives the user the possibility to select the fields he/she wants to display in the report.
You can also add the functionality into the default reports, but keep in mind that we will soon remove the whole publishing engine from senaite.core to senaite.impress.

Maybe some things to consider:

  • order and sorting of the records field might be challenging
  • no specification/out of range possible if these values are used to add multiple results w/o a calculation
  • format/layout of the report might be challenging (especially when you show results of multiple ARs next to each other in columns)

#3

Hi Ramon,
Thank you. I’ve started working on the default reports and once the layout is accepted will move to senaite.impress.

What should be the order of the interim fields(alphabetically or display as they were captured)?
Do we use an indicator like we do on the worksheet when the results are out of range?


#4

Thanks Lunga

There is an issue for the order of the Interim fields on WSs and ARs, COA gets the same order but omitting the Interims not selected for Reporting

Interims should be sorted and displayed

*firstly per AS, so they are next to one another

  • secondly in the order they are on the AS method tab (inherited from the Calc)

#5

https://github.com/senaite/senaite.core/issues/888. Fixed by @xispa earlier, maybe ‘small’-)


#6

messy screenshot of what we are aiming for

Edit, actual formula for combining DM and Unit conversion to be built…

image


#7

I’ve create PR but have marked as not ready because I’ve made a draft on how the interim fields would be displayed on the COA.
Will wait from the community about the way forward of the layout/design.


#8

Draft looks like this

and I also thought maybe this


#9

I’d prefer the last example for standard reports, because it is more flexible regarding the number of interims and space.


#10

I am testing this now. In 3rd party commercial labs, they don’t readily publish Interim values, probably different for production labs

Our first prize is Feeds chemistry, probably 30 interims dispersed throughout their ASs, and we only want to show results before and after drying (DM), also handy in Cannabis. And then use the same machinery for Unit Conversion

Here’s a simplified COA, lacking Analysis Specs in the config …

image

I think @lunga has the display order for them sorted too, https://github.com/senaite/senaite.core/issues/888

image

Am continuing testing with more than one Calc per view, but in the Feeds case, it becomes spaghetti very quickly. We are considering a setup option to disable Interim displays for ARs

Data capturing goes well on WSs with only one or two calcs used per WS for yay Samples. There the tab order on the WS, say with interfaced balances, is important, testing that too, https://github.com/senaite/senaite.core/issues/952

decimal issues remain…


#11

That said, fixing up your own COA is easier now