Meeting #353
Updated by André Neto over 6 years ago
The v0.6 adds support to EPICSPVA and aligns with the v0.19 of the core.
h1. Requirements review
*Date of the review:* 21/12/2018
*Person who did the review:* André Neto
*Version of requirements:* N/A
*Result of review:* N/A
*List of non-conformities:* N/A
*Comments*: Not applicable to this release (each individual component requirements was analysed separately).
h1. Architecture & design review
*Date of the review:* 21/12/2018
*Person who did the review:* André Neto
*Version of architecture & design document:* 0.6
*Result of review:* N/A
*List of non-conformities:* N/A
*Comments*: Not applicable to this release (each individual component architecture was analysed separately).
h1. Code and documentation review
*Date of the review:* 21/12/2018
*Person who did the review:* André Neto
*Version of source code:* 0.6
*Result of review:* PASS
*List of non-conformities:*
The NI6259 and NI6368 do not allow access to more than one GAM at the time. This is to prevent a synchornisation bug (see GetLastBufferIdx). The impact is minor but they should be updated to use the new Circular Buffer of the core framework.
*List of warnings reported by Doxygen:*
<pre>
NONE
</pre>
h1. Unit test review
*Date of the review:* 21/12/2018
*Person who did the review:* André Neto
*Version of unit tests:* 0.6
*Result of coverage tests review:* PASS
*Result of functional tests review:* PASS
*Result of review:* PASS
ITER components tested @ mini-codac2 (but I did not test the hardware)
*List of non-conformities:*
_LCOV report (files containing functions not executed at all):_
Note: execute with <pre>lcov -l Build/MARTe2.coverage.info</pre> and compare with the latest master release using tkdiff.
Files containing functions with coverage < 90% or which have changed w.r.t. to last master release. Exclamation marks inform of differences w.r.t. to the last master.:_
<pre>
|Lines |Functions|Branches
Filename |Rate Num|Rate Num|Rate Num
================================================================================
[/home/aneto/Projects/MARTe2-components/Source/Components/]
DataSources/EPICSCA/EPICSCAInput.cpp |94.0% 182|95.7% 23| - 0
GAMs/ConversionGAM/ConversionGAM.cpp |99.3% 305|66.7% 12| - 0
GAMs/FilterGAM/FilterGAM.cpp |98.2% 341|86.4% 22| - 0
GAMs/SSMGAM/SSMGAM.cpp |97.7% 444|76.9% 13| - 0
GAMs/WaveformGAM/Waveform.cpp |98.5% 324|91.7% 12| - 0
GAMs/WaveformGAM/WaveformChirp.cpp |99.0% 97|86.4% 22| - 0
GAMs/WaveformGAM/WaveformPointsDef.cpp |96.4% 167|88.0% 25| - 0
GAMs/WaveformGAM/WaveformSin.cpp |98.7% 75|86.4% 22| - 0
Interfaces/EPICS/EPICSCAClient.cpp |75.0% 100|94.1% 17| - 0
Interfaces/EPICS/EPICSPV.cpp |93.6% 251|96.3% 27| - 0
================================================================================
Total:|92.5% 11637|96.0% 1k| - 0
</pre>
_LCOV report (files containing functions with coverage < 90%):_
<pre>
|Lines |Functions|Branches
Filename |Rate Num|Rate Num|Rate Num
================================================================================
[/home/aneto/Projects/MARTe2-components/Source/Components/]
DataSources/EpicsDataSource/Platform.cpp |81.0% 58| 100% 4| - 0
Interfaces/EPICS/EPICSCAClient.cpp |89.0% 100|94.1% 17| - 0
================================================================================
Total:|92.5% 11637|96.0% 1k| - 0
</pre>
_GTest report_
<pre>
[----------] Global test environment tear-down
[==========] 2024 tests from 64 test cases ran. (480116 ms total)
[ PASSED ] 2024 tests.
</pre>
h1. Integration test review
*Date of the review:* 05/09/2018
*Person who did the review:* André Neto
*Version of integration tests:* 0.5
*Result of review:* N/A
*List of non-conformities:* N/A
*Comments*: Not performed.
h1. Acceptance test review
*Date of the review:* 05/09/2018
*Person who did the review:* André Neto
*Version of acceptance tests:* 0.5
*Result of review:* N/A
*List of non-conformities:* N/A
*Comments*: Not performed.
h1. Requirements review
*Date of the review:* 21/12/2018
*Person who did the review:* André Neto
*Version of requirements:* N/A
*Result of review:* N/A
*List of non-conformities:* N/A
*Comments*: Not applicable to this release (each individual component requirements was analysed separately).
h1. Architecture & design review
*Date of the review:* 21/12/2018
*Person who did the review:* André Neto
*Version of architecture & design document:* 0.6
*Result of review:* N/A
*List of non-conformities:* N/A
*Comments*: Not applicable to this release (each individual component architecture was analysed separately).
h1. Code and documentation review
*Date of the review:* 21/12/2018
*Person who did the review:* André Neto
*Version of source code:* 0.6
*Result of review:* PASS
*List of non-conformities:*
The NI6259 and NI6368 do not allow access to more than one GAM at the time. This is to prevent a synchornisation bug (see GetLastBufferIdx). The impact is minor but they should be updated to use the new Circular Buffer of the core framework.
*List of warnings reported by Doxygen:*
<pre>
NONE
</pre>
h1. Unit test review
*Date of the review:* 21/12/2018
*Person who did the review:* André Neto
*Version of unit tests:* 0.6
*Result of coverage tests review:* PASS
*Result of functional tests review:* PASS
*Result of review:* PASS
ITER components tested @ mini-codac2 (but I did not test the hardware)
*List of non-conformities:*
_LCOV report (files containing functions not executed at all):_
Note: execute with <pre>lcov -l Build/MARTe2.coverage.info</pre> and compare with the latest master release using tkdiff.
Files containing functions with coverage < 90% or which have changed w.r.t. to last master release. Exclamation marks inform of differences w.r.t. to the last master.:_
<pre>
|Lines |Functions|Branches
Filename |Rate Num|Rate Num|Rate Num
================================================================================
[/home/aneto/Projects/MARTe2-components/Source/Components/]
DataSources/EPICSCA/EPICSCAInput.cpp |94.0% 182|95.7% 23| - 0
GAMs/ConversionGAM/ConversionGAM.cpp |99.3% 305|66.7% 12| - 0
GAMs/FilterGAM/FilterGAM.cpp |98.2% 341|86.4% 22| - 0
GAMs/SSMGAM/SSMGAM.cpp |97.7% 444|76.9% 13| - 0
GAMs/WaveformGAM/Waveform.cpp |98.5% 324|91.7% 12| - 0
GAMs/WaveformGAM/WaveformChirp.cpp |99.0% 97|86.4% 22| - 0
GAMs/WaveformGAM/WaveformPointsDef.cpp |96.4% 167|88.0% 25| - 0
GAMs/WaveformGAM/WaveformSin.cpp |98.7% 75|86.4% 22| - 0
Interfaces/EPICS/EPICSCAClient.cpp |75.0% 100|94.1% 17| - 0
Interfaces/EPICS/EPICSPV.cpp |93.6% 251|96.3% 27| - 0
================================================================================
Total:|92.5% 11637|96.0% 1k| - 0
</pre>
_LCOV report (files containing functions with coverage < 90%):_
<pre>
|Lines |Functions|Branches
Filename |Rate Num|Rate Num|Rate Num
================================================================================
[/home/aneto/Projects/MARTe2-components/Source/Components/]
DataSources/EpicsDataSource/Platform.cpp |81.0% 58| 100% 4| - 0
Interfaces/EPICS/EPICSCAClient.cpp |89.0% 100|94.1% 17| - 0
================================================================================
Total:|92.5% 11637|96.0% 1k| - 0
</pre>
_GTest report_
<pre>
[----------] Global test environment tear-down
[==========] 2024 tests from 64 test cases ran. (480116 ms total)
[ PASSED ] 2024 tests.
</pre>
h1. Integration test review
*Date of the review:* 05/09/2018
*Person who did the review:* André Neto
*Version of integration tests:* 0.5
*Result of review:* N/A
*List of non-conformities:* N/A
*Comments*: Not performed.
h1. Acceptance test review
*Date of the review:* 05/09/2018
*Person who did the review:* André Neto
*Version of acceptance tests:* 0.5
*Result of review:* N/A
*List of non-conformities:* N/A
*Comments*: Not performed.