Changes between Initial Version and Version 1 of Ticket #20705, comment 13
 Timestamp:
 Jun 2, 2016, 3:54:52 PM (7 years ago)
Legend:
 Unmodified
 Added
 Removed
 Modified

Ticket #20705, comment 13
initial v1 1 Hey, so correct me if i am wrong, [x for x in base_field_tuple] is implicitly implemented using lists. Also check if1 Hey, so correct me if i am wrong, [x for x in base_field_tuple] is implicitly implemented using iterator. If that's so i think the following implementation should get rid all the redundant iterations through the sets. 2 2 3 3 exponent=exponents.first() 4 4 for i in range(dimension): 5 matrix_list.append([reduce(mul, [x[i] for i in exponent],1) for x in base_field_tuple])6 exponent=exponents.next(exponent)7 5 8 is thais what you meant. I believe the above implementation skips all the unecessary iterations through exponents and elements of the base_field_tuple 6 matrix_list.append([reduce(mul, [x[i] for i in exponent],1) for x in base_field_tuple]) 7 exponent=exponents.next(exponent) 9 8 10 9 Replying to [comment:10 dlucas]: 11 12 10 13 11 > Disclaimer: the following remarks are only related to the code in itsef. I did not run (yet) extensive tests on border cases and larger cases than the ones covered by doctests.