Deliverable Review Form

Identification of the deliverable or milestone
Project: EMI Deliverable or milestone identifier: D4.3.3
Title: DSA2.3.3 - Periodic QA and QC Reports Doc. identifier: EMI-DXXX-CDSREF-Title-vx.x
Author(s): SA2* Due date: 30/04/2012

Identification of the reviewer
Name: J.K. Nilsen Affiliation: UIO EMI Activity/External project or Institute: JRA1

Review date 25/05/2012
Author(s) revision date 25/05/2012
Reviewer acceptance date 25/05/2012

Attach the reviewed document to the deliverable page, put here a link

General comments

Jon

It looks much better now, I believe all my comments were answered in v1.2. The deliverable is now acceptable for me.

Alberto

Hi, thanks for the comment, you are right about your points. The KPIs were wrong and I changed them, see in the document if is ok what I have put as comment.

For SLOC I removed then section as the KPI is not there anymore as a KPI.

25/05/12 Jon

I cannot accept this document in its current state. As far as I can see, none of my comments concerning SLOC and KPIs were taken into account.

Before I can accept this document, please provide answers on what is done to address each of my comments (both on the reviewer page and in the document), one by one, and if not addressed, give a justification on why.

25/05/12 Claudio

About the in-line comments: JN1: added reference to development tasks JN2: changed accordingly

About the KPI: I understand your point of view, but Quality Control cannot relies directly on DoW (where is the last version?), but it needs to comply with SA2 Quality Assurance policies and in particular the metric definition document (https://twiki.cern.ch/twiki/pub/EMI/QualityModel/EMI_SA2_MetricsSpecification_v1.4.doc), which does not reflect yet the changes you mentioned, so there was some communication problem between SA1/JRA1 and SA2. Anyway I have added some lines explaining that the SLOC metric will be revised for the next updates/release. By the way even the KPI wiki page is outdated: https://twiki.cern.ch/twiki/bin/view/EMI/EmiKPIs .

23/05/12 Jon

There seems to be a major confusion about JRA1 KPIs. These were changed in the revised DoW. The old KJRA1.1-4 are now replaced with the following:


KJRA1.1 - "Standard compliance KPI":

Name: Number of EMI service interfaces and libraries passing standard compliance tests

Description: The metric measures how many EMI service interfaces and libraries are successfully tested for standard compliance. Standard compliance is defined broadly and also includes compliance with EMI internal agreements.

How to measure: The number is taken by checking the available test reports generated during the quarter by the Product Teams.

Target: JRA1 aims to increase the number of successfull tests at least by two per reporting quarter.

KJRA1.2 - "Product integration KPI":

Name: Number of passed inter-product tests

Description: The metrics shows how EMI products can be used together by passing inter-product tests based on real-life use case scenarios.

How to measure: The number should be taken from the available test reports produced during the quarter. Each passed test should be assigned to a numeric value corresponding to the number of products involved in the test. The metric itself is the sum of the assigned test numbers.

Target: JRA1 aims to increase the metrics value at least by four per reporting quarter

KJRA1.3 - "Harmonization KPI":

Name: Number of EMI products implementing EMI agreements

Description: This metrics shows how the EMI harmonization is progressing by measuring the commitment and actual work to implement EMI agreements.

How to measure: The number of ongoing or completed development tasks of the EMI development tracker targeting EMI agreement implementation will be counted.

Target: Based on the currently known agreements, approximately 20 by the end of the project.


The last time the old KPIs were reported was halfway into the reporting period (in Q6), but this is not reflected in the report. I've tried to highlight the sections/parts were old KPIs are used, but not sure if I found all places that need change/rephrasing. I suggest you go through the document and revise the parts were the old KJRA1.1-4 are used, referenced or being a motivation.

Other than that, only minor comments, see attached tracked version.

Additional recommendations (not affecting the document content, e.g. recommendation for future work)

Detailed comments on the content

Note 1: The reviewers must list here any observation they want to track explicitly and that require interaction with the authors
Alternatively all changes must be listed in the document itself using Word change tracking features (if you use Word)
Note 2: These comments have to be explicitly addressed by the authors and the action taken must be clearly described

Page Section Observations and Replies Is Addressed?
1 xx x.y Sequence of comments and replies separated by twiki signature and date    
2          
3          
4          

Any other modification, spelling or grammatical corrections, etc must be done directly in the document using tracked changes or similar mechanisms that allows the authors to identify which correction is suggested.

-- FloridaEstrella - 24-May-2012

Edit | Attach | Watch | Print version | History: r5 < r4 < r3 < r2 < r1 | Backlinks | Raw View | WYSIWYG | More topic actions
Topic revision: r5 - 2012-05-26 - JonKerrNilsen
 
    • Cern Search Icon Cern Search
    • TWiki Search Icon TWiki Search
    • Google Search Icon Google Search

    EMI All webs login

This site is powered by the TWiki collaboration platform Powered by PerlCopyright &© 2008-2024 by the contributing authors. All material on this collaboration platform is the property of the contributing authors.
or Ideas, requests, problems regarding TWiki? use Discourse or Send feedback