1 - SF028A_AssiHiFrq

document1

2 - SF028A_AssiHiFrq_ind

Page 1
Page 2
Page 3
Page 4

3 - SF028A_AssiHiFrq_Peer Review Checklist Design

Nexteer_Template_V1.0

Overview

Peer Review Instructions
Technical Review Checklist
Template Change Log


Sheet 1: Peer Review Instructions

Instructions for Functional Design Package Peer Review




PRE-MEETING


Function OwnerConfirm that requirements are reviewed and approved PRIOR to the FDP peer review

Function OwnerStart with latest version of the template for any "first reviews" - Continue to use existing temmplate for re-reviews

Function OwnerProvide the functional design package (changed documents) to the invited attendees 1-2 working days in advance of review

Function OwnerNotify the assigned peer reviewer and make sure they are prepared to do their function in the meeting

Function OwnerIdentify necessary attendance and invite to meeting

Function OwnerComplete the "Author" column information for sections 1 through 3 prior to the review

Function OwnerComplete the attendance invitation list in section 5

Function OwnerFor Re-reviews only: Complete the column "remarks by author" to identify actions taken to address items found in earlier reviews.



DURING MEETING


Function OwnerPresent document changes to the review team

Peer ReviewerCapture attendance of the review

Peer ReviewerCapture actions and issues in section 4. Identify issue summary, Document type, Reference (Requirement ID, section number, etc), Defect Type and indicate status as "OPEN"



POST MEETING


Function OwnerFollow up on all "open" items. Update "Summary of Resolution" to indicate what was done or decided.

Function OwnerSchedule follow up review OR review open items with peer reviewer and obtain agreement to close

Peer ReviewerClose change request in system and confirm all associated tasks are complete. Upload peer review checklist (this document) with any FDP updates

Sheet 2: Technical Review Checklist

Technical Review Checklist - Template Version 02.00.00







Product NameElectric Power SteeringElectrical Arch.4Review ScopeDefect TypeNumbers




YesClosedFR
Function IDSF028A_AssiHiFrq

Initial Baseline in EA4Requirement0




NoRejectedFDD
Long NameHigh Frequency Assist

Interface0




NAOpenModel
Version that you started from. NOT the version you hope to release. If this will be v1.0.0, enter NA. Starting BaselineNAEffortDesign7






FMEA
AuthorShrenik ShahReview Effort(Hrs.)2.50Standards0






*.m File


Corr+Verf effort(Hrs.)5.00Documentation0






Cal Process


Total Effort (Hrs.)7.50Others0













Total7







Checklist No.Description of CheckAuthor: This column is for Self review. Author shall fill Yes/No/NA against each point in checklist. AuthorAuthor: This column is for reviewer. Reviewer shall fill Yes/No/NA against each point in checklist. ReviewerAuthor: Detailed Description of the finding shall be provided by the reviewer. Description of finding by reviewerAuthor: Defect type to be selected. Defect TypeAuthor: What action is taken to fix the comment & other remarks need to be filled by author. Remarks By AuthorAuthor: Data in this column shall be filled by reviewer after checking whether the rework is completed. Status







1Section 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?Yes



Closed








Does FDD Long Name, Short Name, and Description match requirements?Yes



Closed








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?NA
No Clients











Do client definitions match the corresponding server runnable?No













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

Closed








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




























2Section 2: ModelAuthor: This column is for Self review. Author shall fill Yes/No/NA against each point in checklist. AuthorAuthor: This column is for reviewer. Reviewer shall fill Yes/No/NA against each point in checklist. ReviewerAuthor: Detailed Description of the finding shall be provided by the reviewer. Description of finding by reviewerAuthor: Defect type to be selected. Defect TypeAuthor: What action is taken to fix the comment & other remarks need to be filled by author. Remarks By AuthorAuthor: Data in this column shall be filled by reviewer after checking whether the rework is completed. Status








Is filename of model in correct format?Yes













Is Top level of model annotated with Requirements Baseline?Yes



Closed








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>"?No
Model not upto 2015 standards











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
No NVM defined











Does model execute without errors/warnings after loading NxtrMBDConfig configuration set?No
Model not upto 2015 standards


























3Section 3: Requirements LinkingAuthor: This column is for Self review. Author shall fill Yes/No/NA against each point in checklist. AuthorAuthor: This column is for reviewer. Reviewer shall fill Yes/No/NA against each point in checklist. ReviewerAuthor: Detailed Description of the finding shall be provided by the reviewer. Description of finding by reviewerAuthor: Defect type to be selected. Defect TypeAuthor: What action is taken to fix the comment & other remarks need to be filled by author. Remarks By AuthorAuthor: Data in this column shall be filled by reviewer after checking whether the rework is completed. Status








Are all requirements links of the format <FDDNumber>_<ObjectID>?NA
No baseline requirements available

Closed








Does requirements HTML report reference only the DOORS module of this component for all links in the design?NA
No baseline requirements available

Closed








Are linked blocks linked to the correct requirements(s)? (watch for problems due to copy/pasted blocks)NA
No baseline requirements available

Closed








Is the list of unlinked blocks acceptable?NA
No baseline requirements available

Closed























4Section 4: Model AdvisorAuthor: This column is for Self review. Author shall fill Yes/No/NA against each point in checklist. AuthorAuthor: This column is for reviewer. Reviewer shall fill Yes/No/NA against each point in checklist. ReviewerAuthor: Detailed Description of the finding shall be provided by the reviewer. Description of finding by reviewerAuthor: Defect type to be selected. Defect TypeAuthor: What action is taken to fix the comment & other remarks need to be filled by author. Remarks By AuthorAuthor: Data in this column shall be filled by reviewer after checking whether the rework is completed. Status








Was Model Advisor run with the correct configuration settings?No
Model not upto 2015 standards











Is the Model Advisor rerport free from "Fails".No
Model not upto 2015 standards











Are Model Advisor report "Warnings" acceptable?No
Model not upto 2015 standards


























5Section 5: Delivery PackageAuthor: This column is for Self review. Author shall fill Yes/No/NA against each point in checklist. AuthorAuthor: This column is for reviewer. Reviewer shall fill Yes/No/NA against each point in checklist. ReviewerAuthor: Detailed Description of the finding shall be provided by the reviewer. Description of finding by reviewerAuthor: Defect type to be selected. Defect TypeAuthor: What action is taken to fix the comment & other remarks need to be filled by author. Remarks By AuthorAuthor: Data in this column shall be filled by reviewer after checking whether the rework is completed. Status








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

Closed























6Section 6: Other Issus/Actions IdentifiedDocumentReferenceSummary of resolutionAuthor: Defect type to be selected. Defect TypeAuthor: What action is taken to fix the comment & other remarks need to be filled by author. Remarks By AuthorAuthor: Data in this column shall be filled by reviewer after checking whether the rework is completed. Status







6.1Add system vehicle speed table as imported calibration*.m Fileimported calibrationsAdded the system vehicle speed table as imported calibrationsDesignupdatedclosed







6.2Add PIM variable definitions*.m FilePIM variablesAdded PIM variable definitionsDesignupdatedclosed







6.3Update variable properties like cardinality, online, customer visible*.m Filevarriable propertiesUpdated variable properties like cardinality, online, customer visible after talking with system's engineersDesignupdatedclosed







6.4Long names should be related to the short names*.m Filevarriable propertiesUpdated long names so they are related to the short namesDesignupdatedclosed







6.5Change input output names in simulink to match the data dictionaryModelinput/output namesChanged input output names in simulink to match the data dictionaryDesignupdatedclosed







6.6Add change log and update version number and descrpition in SimulinkModelsimulinkAdded change log and update version number and descrpition in SimulinkDesignupdatedclosed







6.7Add memory blocks for display variablesModeldisplay variablesAdded memory blocks for display variablesDesignupdatedclosed







































































































































































































































































































7Section 7: APPROVALS













RoleFirst ReviewDateAttendanceApproval?










Function Owner*Shrenik Shah6/11/2015Yes











Peer Reviewer*Kevin DerryYes











Safety<Name - if invited>












SoftwareSpandana BalaniYes











ESG / Systems<Name - if invited>












EPDT / CSE<Name - if invited>












Hardware<Name - if invited>












Test<Name - if invited>












RoleSecond Review (if required)DateAttendanceApproval?










Function Owner*Shrenik Shah6/24/2015Yes











Peer Reviewer*Kevin Derry, Sudeep ShankarYes











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>












RoleThird Review (if required)DateAttendanceApproval?










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>












RoleFourth Review (if required)DateAttendanceApproval?










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>












RoleAdd more if necessaryDateAttendanceApproval?










































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














Sheet 3: Template Change Log

RevChangeAuthor


























































































4 - SF028A_AssiHiFrqs