Technical Review Checklist - Template Version 02.00.00 |
|
|
|
|
|
|
|
|
Product Name | Electric Power Steering | Electrical Arch. | 4 | Review Scope | Defect Type | Numbers |
|
|
|
|
| Yes | Closed | FR |
Function ID | SF012A_HysCmp |
|
| Initial Baseline in EA4 | Requirement | 0 |
|
|
|
|
| No | Rejected | FDD |
Long Name | Hysteresis Compensation |
|
| Interface | 0 |
|
|
|
|
| NA | Open | Model |
Version that you started from.
NOT the version you hope to release.
If this will be v1.0.0, enter NA.
Starting Baseline | NA | Effort | Design | 0 |
|
|
|
|
|
|
| FMEA |
Author | KPIT Offshore | Review Effort(Hrs.) | 0.20 | Standards | 0 |
|
|
|
|
|
|
| *.m File |
|
| Corr+Verf effort(Hrs.) | 0.10 | Documentation | 0 |
|
|
|
|
|
|
| Cal Process |
|
| Total Effort (Hrs.) |
| Others | 0 |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Total | 0 |
|
|
|
|
|
|
|
|
Checklist No. | Description of Check | Author:
This column is for Self review. Author shall fill Yes/No/NA against each point in checklist.
Author | Author:
This column is for reviewer. Reviewer shall fill Yes/No/NA against each point in checklist.
Reviewer | Author:
Detailed Description of the finding shall be provided by the reviewer.
Description of finding by reviewer | Author:
Defect type to be selected.
Defect Type | Author:
What action is taken to fix the comment & other remarks need to be filled by author. Remarks By Author | Author:
Data in this column shall be filled by reviewer after checking whether the rework is completed. Status |
|
|
|
|
|
|
|
|
1 | Section 1: Data Dictionary |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| Is Filename of Data Dictionary in correct format? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Is the FDD.Version property correctly updated? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Is the Data Dictionary Verification report error free? | No |
| Errors are due to missing signal names in the nexteer signal list and known issues in the tool |
|
|
|
|
|
|
|
|
|
|
|
| Does FDD Long Name, Short Name, and Description match requirements? | NA |
| No Baseline requirements available |
|
|
|
|
|
|
|
|
|
|
|
| Are all runnables defined? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Do runnables have the correct time step? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Do server runnables correctly define arguments? | NA |
| No Server runnables |
|
|
|
|
|
|
|
|
|
|
|
| Are all clients defined? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Do client definitions match the corresponding server runnable? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Does name and metadata of every signal match its corresponding interface signal? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Do output signal ranges match requirements (check DOOR min/max attributes too)? | NA |
| No Baseline requirements available |
|
|
|
|
|
|
|
|
|
|
|
| Are calibration tables named correctly (e.g. AssiX and AssiY)? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Do all calibrations have correct values for all metadata? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Are calibration tables named correctly (e.g. AssiX and AssiY)? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Is NVM defined in the appropriate number of blocks? | NA |
| No NVM defined |
|
|
|
|
|
|
|
|
|
|
|
| Are constants defined with proper scope (local vs global)? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Are all dependent constants and calibrations included in one file? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2 | Section 2: Model | Author:
This column is for Self review. Author shall fill Yes/No/NA against each point in checklist.
Author | Author:
This column is for reviewer. Reviewer shall fill Yes/No/NA against each point in checklist.
Reviewer | Author:
Detailed Description of the finding shall be provided by the reviewer.
Description of finding by reviewer | Author:
Defect type to be selected.
Defect Type | Author:
What action is taken to fix the comment & other remarks need to be filled by author. Remarks By Author | Author:
Data in this column shall be filled by reviewer after checking whether the rework is completed.
|
|
|
|
|
|
|
|
|
| Is filename of model in correct format? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Is Top level of model annotated with Requirements Baseline? | NA |
| No Baseline requirements available |
|
|
|
|
|
|
|
|
|
|
|
| Is the Top level of the model annotated with Tool Dependencies? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Is Top level of model annotated with Change Log or History? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Does the Component shortname match data dictionary FDD metadata? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Is the 2nd level of model free from subsystems that are not Function-Call Subsystems? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Is the 2nd level of model free from arithmetic and logic operations? | Yes |
| |
|
|
|
|
|
|
|
|
|
|
|
| Are the Runnable trigger signals named as "call_<Runnable>"? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Does 2nd level of model have a properly updated annotation with name, description, and intended baseline number? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Are all data flow layers free of Function-Call Subsystems and Memory Store blocks? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Does the Model have the confidentiality and copyright information inside all its Subsystems? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Are all the Memory Store blocks for PIM and Display Variables located on the 2nd level of model? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Do all Memory Store blocks for PIM and Display Variables have the "Data store name must resolve to Simulink signal object" setting checked to true? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Is each diagnostic (NTC) capable of being set to "PASS"? | NA |
| No NTCs |
|
|
|
|
|
|
|
|
|
|
|
| Does non-zero intialization of PIM occur in the function's Init runnable? | NA |
| No non-zero intialization of PIMs in the model |
|
|
|
|
|
|
|
|
|
|
|
| Does design properly include Set Ram Block Status when NVM RAM values change? | NA |
| No NVM defined |
|
|
|
|
|
|
|
|
|
|
|
| Does model include appropriate logic for dealing with missing or corrupted NVM data? | NA |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Does model execute without errors/warnings after loading NxtrMBDConfig configuration set? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
3 | Section 3: Requirements Linking | Author:
This column is for Self review. Author shall fill Yes/No/NA against each point in checklist.
Author | Author:
This column is for reviewer. Reviewer shall fill Yes/No/NA against each point in checklist.
Reviewer | Author:
Detailed Description of the finding shall be provided by the reviewer.
Description of finding by reviewer | Author:
Defect type to be selected.
Defect Type | Author:
What action is taken to fix the comment & other remarks need to be filled by author. Remarks By Author | Author:
Data in this column shall be filled by reviewer after checking whether the rework is completed.
|
|
|
|
|
|
|
|
|
| Are all requirements links of the format <FDDNumber>_<ObjectID>? | NA |
| No baseline requirements available |
|
|
|
|
|
|
|
|
|
|
|
| Does requirements HTML report reference only the DOORS module of this component for all links in the design? | NA |
| No baseline requirements available |
|
|
|
|
|
|
|
|
|
|
|
| Are linked blocks linked to the correct requirements(s)? (watch for problems due to copy/pasted blocks) | NA |
| No baseline requirements available |
|
|
|
|
|
|
|
|
|
|
|
| Is the list of unlinked blocks acceptable? | NA |
| No baseline requirements available |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
4 | Section 4: Model Advisor | Author:
This column is for Self review. Author shall fill Yes/No/NA against each point in checklist.
Author | Author:
This column is for reviewer. Reviewer shall fill Yes/No/NA against each point in checklist.
Reviewer | Author:
Detailed Description of the finding shall be provided by the reviewer.
Description of finding by reviewer | Author:
Defect type to be selected.
Defect Type | Author:
What action is taken to fix the comment & other remarks need to be filled by author. Remarks By Author | Author:
Data in this column shall be filled by reviewer after checking whether the rework is completed.
|
|
|
|
|
|
|
|
|
| Was Model Advisor run with the correct configuration settings? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Is the Model Advisor rerport free from "Fails". | No |
| Fail' due to Simscape licensce error. This fail can be ignored as we don’t use Simscape |
|
|
|
|
|
|
|
|
|
|
|
| Are Model Advisor report "Warnings" acceptable? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
5 | Section 5: Delivery Package | Author:
This column is for Self review. Author shall fill Yes/No/NA against each point in checklist.
Author | Author:
This column is for reviewer. Reviewer shall fill Yes/No/NA against each point in checklist.
Reviewer | Author:
Detailed Description of the finding shall be provided by the reviewer.
Description of finding by reviewer | Author:
Defect type to be selected.
Defect Type | Author:
What action is taken to fix the comment & other remarks need to be filled by author. Remarks By Author | Author:
Data in this column shall be filled by reviewer after checking whether the rework is completed.
|
|
|
|
|
|
|
|
|
| Does Design folder contain only the model, data dictionary, and (optionally) a simulation setup script? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Does Doc folder contain a zipped HTML webview model? | Yes |
|
|
|
|
|
|
|
|
|
|
|
|
|
| Does Reports folder contain only the data dictionary verification report, zipped Model Advisor report, and zipped requirements traceability report? | No |
| Requirement traceability report does not exists as requirement linking is not created due to non availability of baseline requirements |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
6 | Section 6: Other Issus/Actions Identified | Document | Reference | Summary of resolution | Author:
Defect type to be selected.
Defect Type | Author:
What action is taken to fix the comment & other remarks need to be filled by author. Remarks By Author | Author:
Data in this column shall be filled by reviewer after checking whether the rework is completed.
|
|
|
|
|
|
|
|
|
6.1 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
6.2 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
6.3 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
6.4 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
6.5 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
6.6 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
6.7 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
6.8 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
6.9 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
6.10 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
6.11 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
6.12 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
6.13 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
6.14 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
6.15 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
6.16 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
6.17 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
6.18 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
6.19 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
6.20 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
6.21 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
6.22 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
6.23 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
6.24 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
6.25 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
7 | Section 7: APPROVALS |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Role | First Review | Date | Attendance | Approval? |
|
|
|
|
|
|
|
|
|
|
|
Function Owner* | <Name - if invited> | 1/6/2016 |
|
|
|
|
|
|
|
|
|
|
|
|
|
Peer Reviewer* | Kevin Derry | Yes | Yes |
|
|
|
|
|
|
|
|
|
|
|
Safety | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
Software | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
ESG / Systems | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
EPDT / CSE | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
Hardware | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
Test | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
Role | Second Review (if required) | Date | Attendance | Approval? |
|
|
|
|
|
|
|
|
|
|
|
Function Owner* | <Owner Name> | |
|
|
|
|
|
|
|
|
|
|
|
|
|
Peer Reviewer* | <Name> |
|
|
|
|
|
|
|
|
|
|
|
|
|
Safety | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
Software | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
ESG / Systems | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
EPDT / CSE | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
Hardware | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
Test | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
Role | Third Review (if required) | Date | Attendance | Approval? |
|
|
|
|
|
|
|
|
|
|
|
Function Owner* | <Owner Name> | |
|
|
|
|
|
|
|
|
|
|
|
|
|
Peer Reviewer* | <Name> |
|
|
|
|
|
|
|
|
|
|
|
|
|
Safety | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
Software | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
ESG / Systems | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
EPDT / CSE | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
Hardware | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
Test | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
Role | Fourth Review (if required) | Date | Attendance | Approval? |
|
|
|
|
|
|
|
|
|
|
|
Function Owner* | <Owner Name> |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Peer Reviewer* | <Name> |
|
|
|
|
|
|
|
|
|
|
|
|
|
Safety | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
Software | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
ESG / Systems | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
EPDT / CSE | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
Hardware | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
Test | <Name - if invited> |
|
|
|
|
|
|
|
|
|
|
|
|
|
Role | Add more if necessary | Date | Attendance | Approval? |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
P.S.: | Yes indicates adherence |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| No indicates non-adherence, reviewer shall provide suitable comments at the end of this document for each point. |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| NA indicates not applicable |
|
|
|
|
|
|
|
|
|
|
|
|
|
|