allpairspy icon indicating copy to clipboard operation
allpairspy copied to clipboard

incorrect distribution for one parameter

Open tirana opened this issue 4 years ago • 2 comments

result = AllPairs(OrderedDict( {"a1": ["0", "1"], "a2": ["0", "1"], "a3": ["0", "1"], "a4": ["0", "1"], "a5": ["0", "1"], "a6": ["0", "1"]}))

generates: [ Pairs(a1='0', a3='0', a2='0', a5='0', a4='0', a6='0'), Pairs(a1='1', a3='1', a2='1', a5='1', a4='1', a6='0'), Pairs(a1='1', a3='0', a2='1', a5='0', a4='1', a6='1'), Pairs(a1='0', a3='1', a2='0', a5='1', a4='0', a6='1'), Pairs(a1='0', a3='1', a2='1', a5='0', a4='0', a6='1'), Pairs(a1='1', a3='0', a2='0', a5='1', a4='0', a6='1'), Pairs(a1='0', a3='0', a2='0', a5='1', a4='1', a6='1')]

as you may see the a6 has 5 "1" while the other 3 or 4.

similar picture for 8 params: [Pairs(a1='0', a3='0', a2='0', a5='0', a4='0', a7='0', a6='0', a8='0'), Pairs(a1='1', a3='1', a2='1', a5='1', a4='1', a7='1', a6='1', a8='0'), Pairs(a1='1', a3='0', a2='1', a5='0', a4='1', a7='0', a6='1', a8='1'), Pairs(a1='0', a3='1', a2='0', a5='1', a4='0', a7='1', a6='0', a8='1'), Pairs(a1='0', a3='1', a2='1', a5='0', a4='0', a7='1', a6='1', a8='1'), Pairs(a1='1', a3='0', a2='0', a5='1', a4='1', a7='1', a6='0', a8='1'), Pairs(a1='1', a3='1', a2='0', a5='1', a4='0', a7='0', a6='1', a8='1'), Pairs(a1='0', a3='1', a2='1', a5='1', a4='1', a7='1', a6='0', a8='1')]

tirana avatar Aug 22 '19 16:08 tirana

Thank you for your report.

This issue probably the same cause with #2

thombashi avatar Jul 24 '20 14:07 thombashi

Fixed in this PR.

result = AllPairs(OrderedDict({
    "a1": ["0", "1"], 
    "a2": ["0", "1"], 
    "a3": ["0", "1"], 
    "a4": ["0", "1"], 
    "a5": ["0", "1"], 
    "a6": ["0", "1"],
    "a7": ["0", "1"],
    "a8": ["0", "1"],
}))

generates:

PAIRWISE:
 0: Pairs(a1='0', a2='0', a3='0', a4='0', a5='0', a6='0', a7='0', a8='0')
 1: Pairs(a1='1', a2='1', a3='1', a4='1', a5='1', a6='1', a7='1', a8='0')
 2: Pairs(a1='1', a2='0', a3='1', a4='0', a5='1', a6='0', a7='1', a8='1')
 3: Pairs(a1='0', a2='1', a3='0', a4='1', a5='0', a6='1', a7='0', a8='1')
 4: Pairs(a1='0', a2='1', a3='1', a4='0', a5='0', a6='1', a7='1', a8='1')
 5: Pairs(a1='1', a2='0', a3='0', a4='1', a5='1', a6='0', a7='0', a8='0')
 6: Pairs(a1='1', a2='0', a3='0', a4='1', a5='0', a6='1', a7='1', a8='0')
 7: Pairs(a1='0', a2='1', a3='1', a4='0', a5='1', a6='0', a7='0', a8='1')

pavelicii avatar Dec 10 '21 20:12 pavelicii