MAVEN Particles and Fields Flight Software Peer Review

MAVEN Particles and Fields Flight Software Peer Review RFAs and Recommendations Peter R. Harvey Pre-PDR Peer Review 1 UCB

Recommendation Sw. PDR-1 Title : Requirement 251 - Comm Loss Response Reviewer: S. Harris (Chair) Recommendation: Currently the time out is set to 3 seconds. Instrument needs an override on the response to deal with test scenarios. Response: The FSW response to a Communication Timeout is planned to be a Real-Time-Sequence action. Since all RTS’s can be disabled, it would be straightforward to disable that RTS in test scenarios when S/C messages are turned off. Pre-PDR Peer Review 2 UCB

Recommendation Sw. PDR-2 Title : Requirement 310 – Similar HV requirements Reviewer: S. Harris (Chair) Recommendation: This requirement, as well as similar ones for several instruments, looks identical to the generic rule for HV control (Req 243). Why have separate requirement for Static and each instrument? Response: Agreed. Below is a comparison of the HV requirements. We plan to move common elements to one general requirement, and leave unique elements with each instrument HV system. Pre-PDR Peer Review 3 UCB

Recommendation Sw. PDR-3 Title : Requirement 516 - Typo Reviewer: S. Harris (Chair) Recommendation: “Sun” should be “Sum” Response: Corrected in the MAVEN_PF_SYS_010_FSWRequirements revision F And will be corrected in MAVEN_PF_FSW_002_SRS revision D Pre-PDR Peer Review 4 UCB

Recommendation Sw. PDR-4 Title : S/C Command Timing Reviewer: S. Harris (Chair) Recommendation: The gap time between commands from the S/C needs clarification in the ICD. Response: Will clarify the timing in the S/C ICD has no TBDs and is consistent with the FPGA and FSW expectations. Answer: Pre-PDR Peer Review 5 UCB

Recommendation Sw. PDR-5 Title : New Platform, New FSW Risk Reviewer: S. Harris (Chair) Recommendation: The team should consider adding a project risk that using a new processor (Coldfire) and developing the FSW in a different language (C), presents a risk to the schedule. The FSW is based on the same architecture as the heritage code, but is being entirely re-written in C and assembly, for a different processor, using a new development environment Response: We have added this to the risk list as a schedule risk (not technical since we have plenty of technical margins). The plan would be to watch the development to determine what troubles we run into. If the conversion process is much more difficult than planned, we should review the approach before we run out of time. Pre-PDR Peer Review 6 UCB

Recommendation Sw. PDR-6 Title : Document Due Dates Reviewer: T. Jackson Recommendation: Re-evaluate your project requested document list and due dates. Your list is not correct. Response: MAVEN-SYS-PLAN-0020 page 47 is the source of columns 1 and 2 of the table below. The STP and SMP are due in 2011 and 2012 and are not due this year. The confusion stems from pages 41 and 42 where these documents are listed as necessary for the successful PDR. Pre-PDR Peer Review 7 UCB

Recommendation Sw. PDR-7 Title : Re-Evaluate FSW Risks Reviewer: T. Jackson Recommendation: Re-evaluate your current risks. Add in some realistic risks. Also remove the absurd. Response: The risk list shown in the presentation showed only the format of risks, and was not supposed to represent the actual list. We plan to provide a FSW risk list shortly. Pre-PDR Peer Review 8 UCB

Recommendation Sw. PDR-8 Title : Define Instrument Mode Set Reviewer: E. Taylor Recommendation: Comment during the review that we wouldn't have defined instrument modes, configuration is set by RTS. Suggest a number of modes be explicitly defined for test and used initially for early orbit operations. (Compression algorithms should be run on modes to get accurate (tested) estimate of real data volume). Response: Will define a risk here, that we don’t understand the interplay between instruments and this could lead to resource issues of one sort or another. We would retire the risk by defining the modes carefully. Pre-PDR Peer Review 9 UCB

Recommendation Sw. PDR-9 Title : S/C Interface Concerns Reviewer: E. Taylor Recommendation: Not a lot of confidence in the SC/PFIDPU Interface. Seemed to be confusion on what was in ICD, what should be in there. No LM SC Simulator is being delivered to UCB. ETU test is hardware test only, LM instrument I/F FSW will not be implemented until shortly before Instrument delivery Response: We could define a risk here that the FSW is being developed while the S/C interface is still changing. Of course, it is standard practice to have these TBDs for a while. Nonetheless, we would retire the risk when the ICD is firm. Pre-PDR Peer Review 10 UCB

Recommendation Sw. PDR-10 Title : New Platform, New FSW Risk Reviewer: E. Taylor Recommendation: New software language (0 line of codes re-use), new platform should be a risk. Response: Same response as Sw. PDR-5 Pre-PDR Peer Review 11 UCB

Recommendation Sw. PDR-11 Title : Simplify HV Control Requirements Reviewer: P. Harvey Notes Recommendation: In the requirements section, there are many requirements for HV control that seem to be the same. Perhaps it would be better to reduce these to a single requirement. . Response: Same response as Sw. PDR-2 Pre-PDR Peer Review 12 UCB

Recommendation Sw. PDR-12 Title : Simplify Attenuator Control Requirements Reviewer: P. Harvey Notes Recommendation: In the requirements section, there are many requirements for Attenuator control that seem to be the same. Perhaps it would be better to reduce these to a single requirement. Response: Agreed. Below is a comparison of the Atten requirements. We plan to move common elements to one general requirement, and leave unique elements with each instrument attenuator. Pre-PDR Peer Review 13 UCB

Recommendation Sw. PDR-13 Title : Requirement 516: Typo Reviewer: P. Harvey Notes Recommendation: In the requirements section, PFFSW-516 has "Sun" instead of "Sum" in two places. Needs to be corrected in the SRS. Response: Same response as Sw. PDR-3 Pre-PDR Peer Review 14 UCB

Recommendation Sw. PDR-14 Title : Requirement 802 Correction Reviewer: P. Harvey Notes Recommendation: In the requirements section, PFFSW-802 has 256 bytes per table. This should be 512 bytes per table. Response: Corrected in the MAVEN_PF_SYS_010_FSWRequirements revision F And will be corrected in MAVEN_PF_FSW_002_SRS revision D Pre-PDR Peer Review 15 UCB

Recommendation Sw. PDR-15 Title : Code Coverage Metric Reviewer: P. Harvey Notes Recommendation: In the plan section, page 6, the Code Warrior toolset includes a code coverage metric which may be useful. Response: We already bought the Standard version and it does not include the code coverage analysis. If we buy another copy, we will consider this option. Pre-PDR Peer Review 16 UCB

Recommendation Sw. PDR-16 Title : New Platform, New FSW Risk Reviewer: P. Harvey Notes Recommendation: In the plan section, page 9, a risk should be written that captures the "new" development environment, new language. Response: Same response as Sw. PDR-5 Pre-PDR Peer Review 17 UCB

Recommendation Sw. PDR-17 Title : Document Due Dates Reviewer: P. Harvey Notes Recommendation: In the plan section, page 17, the STP and SMP are not due at this time. Response: Same response as Sw. PDR-6 Pre-PDR Peer Review 18 UCB

Recommendation Sw. PDR-18 Title : SRAM typo Reviewer: P. Harvey Notes Recommendation: In the design section, page 3, the DRAM should be SRAM. Response: Corrected in the figure for PDR. Pre-PDR Peer Review 19 UCB

Recommendation Sw. PDR-19 Title : TM Output requirement Reviewer: P. Harvey Notes Recommendation: In the design section, page 7, the TM only has one low rate output. There is no high rate transmitter Response: Corrected in the figure for PDR. Pre-PDR Peer Review 20 UCB

Recommendation Sw. PDR-20 Title : SWIA description Reviewer: P. Harvey Notes Recommendation: In the design section, page 7, the SWIA has a "manager" too Response: Corrected in the figure for PDR. Pre-PDR Peer Review 21 UCB

Recommendation Sw. PDR-21 Title : TM NOOP Requirement Reviewer: P. Harvey Notes Recommendation: In the design section, page 11, the TM needs to produce something every 5 seconds, at minimum a NOOP Response: Corrected in the figure for PDR. Pre-PDR Peer Review 22 UCB

Recommendation Sw. PDR-22 Title : TM Allocation Table is Old Reviewer: P. Harvey Notes Recommendation: In the design section, page 11, the PF telemetry breakdown table is old. The new table is SYS 002 K Response: Corrected in the figure for PDR. Pre-PDR Peer Review 23 UCB

Recommendation Sw. PDR-23 Title : Flash Capacity Typo Reviewer: P. Harvey Notes Recommendation: In the design section, page 25, the capacity of the Flash is 8 GB, not 32 Response: Corrected in the figure for PDR. Pre-PDR Peer Review 24 UCB