multiple-pipeline-{capture,playback}.sh are broken #324

Activity

1. lyakh

   added

   type:bugSomething doesn't work as expectedSomething doesn't work as expected

   on Aug 7, 2020
2. lyakh

   changed the title [-]multiple-pipeline-playback.sh is broken[/-] [+]multiple-pipeline-{capture,playback}.sh are broken[/+] on Aug 7, 2020
3. plbossart commented on Aug 12, 2020

   

   plbossart

   on Aug 12, 2020

   Member

   More actions

   @lyakh do you mean to say that this line is wrong?

   [[ $PIPELINE_COUNT -gt ${OPT_VALUE_lst['c']} ]] && max_count=${OPT_VALUE_lst['c']} || max_count=$PIPELINE_COUNT
   f
   

   I must say I have zero understanding of these scripts.
4. lyakh commented on Sep 1, 2020

   

   lyakh

   on Sep 1, 2020

   ContributorAuthor

   More actions

   @lyakh do you mean to say that this line is wrong?

   [[ $PIPELINE_COUNT -gt ${OPT_VALUE_lst['c']} ]] && max_count=${OPT_VALUE_lst['c']} || max_count=$PIPELINE_COUNT
   f
   

   I must say I have zero understanding of these scripts.

   @plbossart it isn't about the contents of that line, it's about when it is executed, respectively when the $PIPELINE_COUNT variable is evaluated. Evaluating it before it is set makes no sense and it is only set in func_pipeline_export()
5. aiChaoSONG commented on Sep 23, 2020

   

   aiChaoSONG

   on Sep 23, 2020

   More actions

   @lyakh This is fixed by #380, thanks. let's close this
6. aiChaoSONG

   closed this as completedon Sep 23, 2020