AllenInstitute / MIES

Multichannel Igor Electrophysiology Suite
https://alleninstitute.github.io/MIES/user.html
Other
23 stars 7 forks source link

Important code which is not executed in tests #1007

Open t-b opened 3 years ago

t-b commented 3 years ago

Latest coverage data: https://byte-physics.de/public-downloads/aistorage/transfer/report/coverage

Using UTF code coverage feature it was revealed that the following code paths are never executed:

00000128|00000000|00000128|         if(powerSpectrum)
________|________|________|             DACAmp[i][%TPAMP] = 0
________|________|________|         endif
00000913|00000000|00000913|         if(V_Value != -1)
________|________|________|             return [1, V_row, V_col]
________|________|________|         endif
00000406|________|________|         else
________|________|________|             DC_AddEpoch(panelTitle, channel, epochBegin, epochEnd, EPOCH_BASELINE_REGION_KEY, 2, lowerlimit = stimsetBegin, upperlimit = stimsetEnd)
________|________|________|         endif
412|________|________|________|            else

413|________|________|________|                tags = ReplaceStringByKey(EPOCH_SUBTYPE_KEY, "", EPOCH_BASELINE_REGION_KEY, STIMSETKEYNAME_SEP, EPOCHNAME_SEP)

414|________|________|________|                EP_AddEpoch(device, channel, epochBegin, epochEnd, tags, shortNameEpTypePTBaseline, 2, lowerlimit = stimsetBegin, upperlimit = stimsetEnd)
855|________|________|________|            // non present required parameters are an error

856|________|________|________|            errorMessages[index++] = name + ": is required but missing"

857|________|________|________|            continue
334|________|________|________|        AI_UpdateAmpModel(device, "check_DatAcq_HoldEnable", headStage, value=1, sendToAll=0)

335|________|________|________|        AI_UpdateAmpModel(device, "setvar_DataAcq_Hold_IC", headstage, value=current * ONE_TO_PICO,sendToAll=0)
t-b commented 2 years ago

Last update: 7d6c22322 (Merge pull request #2279 from AllenInstitute/bugfix/2279-better-manual-installation-instructions, 2024-10-18)