how to solve memory

meInvent bbird jobmattcon at gmail.com
Fri Jun 10 11:23:57 EDT 2016


On Friday, June 10, 2016 at 10:08:44 PM UTC+8, Steven D'Aprano wrote:
> On Fri, 10 Jun 2016 10:58 am, meInvent bbird wrote:
> 
> [snip unreadable code]
> 
> I just ran your code, and it almost crashed my computer. I think it is quite
> rude to post code without an explanation of what the problem is.
> 
> 
> I'm afraid that your code is virtually unreadable to me. It is too verbose,
> the indentation is awful (single space indents is impossible for me to
> track by eye) and it makes too much work out of simple operations. And none
> of the variable names mean anything.
> 
> I've started to simplify and clean the code, and got to the following, which
> I hope you will agree is easier to read and more compact:
> 
> 
> 
> import itertools
> M1 = {'00': 0, '01': 2, '02': 1, '10': 1, '11': 1, 
>       '12': 1, '20': 1, '21': 1, '22': 2}
> M2 = {'00': 0, '01': 1, '02': 1, '10': 1, '11': 1, 
>       '12': 1, '20': 1, '21': 1, '22': 1}
> M3 = {'00': 2, '01': 2, '02': 2, '10': 0, '11': 2, 
>       '12': 1, '20': 0, '21': 1, '22': 2}
> M4 = {'00': 1, '01': 2, '02': 1, '10': 2, '11': 2, 
>       '12': 2, '20': 0, '21': 1, '22': 2}
> M5 = {'00': 0, '01': 1, '02': 1, '10': 0, '11': 2, 
>       '12': 1, '20': 0, '21': 1, '22': 1}
> # Why V instead of M?
> V6 = {'00': 1, '01': 1, '02': 2, '10': 1, '11': 2, 
>       '12': 1, '20': 1, '21': 2, '22': 2}
> 
> MM = {0: M1, 1: M2, 2: M3, 3: M4, 4: M5, 5: V6}
> m = 3
> b = [str(i)+str(j)+str(k) for i in range(m) 
>      for j in range(m) for k in range(m)]
> mylist = [MM[i] for i in range(6)]
> 
> def DFS(b, deep, maxx, sourceoperators, path):
>     initlist = []
>     if deep > 0:
>         print("deep=", deep)
>         for aa,bb in itertools.combinations(sourceoperators, 2):
>             print(aa,bb)
>             if deep == maxx:
>                 finalresult = []
>                 op1xy = [aa[b[i][0:1]] for i in range(len(b))]
>                 op1yz = [aa[b[i][1:2]] for i in range(len(b))]
>                 op1xz = [aa[b[i][0]+b[i][2]] for i in range(len(b))]
>                 op2xy = [bb[b[i][0:1]] for i in range(len(b))]
>                 op2yz = [bb[b[i][1:2]] for i in range(len(b))]
>                 op2xz = [bb[b[i][0]+b[i][2]] for i in range(len(b))]
>                 if sum(op1xy) == 54:
>                     path.append([(deep, aa, "xy")])
>                 if sum(op1yz) == 54:
>                     path.append([(deep, aa, "yz")])
>                 if sum(op1xz) == 54:
>                     path.append([(deep, aa, "xz")])
>                 if sum(op2xy) == 54:
>                     path.append([(deep, bb, "xy")])
>                 if sum(op2yz) == 54:
>                     path.append([(deep, bb, "yz")])
>                 if sum(op2xz) == 54:
>                     path.append([(deep, bb, "xz")])
>                 initlist.extend([op1xy, op1yz, op1xz, op2xy, op2yz, op2xz])
>             else:
>                 level = []
>                 for j in range(len(b)):
>                     op1xy = [aa[b[j][i]] for i in range(len(b[j]))]
>                     op2xy = [bb[b[j][i]] for i in range(len(b[j]))]
>                     if sum(op1xy) == 54:
>                         path.append([(deep, aa, "xy")])
>                     if sum(op2xy) == 54:
>                         path.append([(deep, bb, "xy")])
>                     level.extend([op1xy, op2xy])
>                     initlist.extend([op1xy, op2xy])
>     if deep == maxx:
>         b = []
>     for aaa,bbb in itertools.combinations(initlist, 2):
>         b.append([str(i)+str(j) for i,j in zip(aaa, bbb)])
>     path = DFS(b, deep-1, maxx, sourceoperators, path)
>     return path
> 
> path = []
> mresult = DFS(b, 2, 2, mylist, path)
> 
> 
> 
> Unfortunate, in cleaning up your code, I have changed something, because my
> version and your version do not do the same thing.
> 
> After nearly crashing my computer running your version, I am not going to
> spend the time trying to debug this. If you want our help, I suggest that
> you clean up the code. Currently it is an incomprehensible mess to me. Even
> after the cleanup, I have no idea what this piece of code is supposed to do
> or what it is calculating.
> 
> Can you explain the purpose of the code? What is it calculating? Can you
> show the expected results? What does "DFS" mean? Why do you have M1 - M5
> but then a mysterious V6?
> 
> I can see you are calculating *something* to do with permutations of 0 1 2,
> but I don't understand what.
> 
> I think I can see one problem: DFS is a recursive function, but it never
> ends the recursion. It ALWAYS calls DFS(... deep-1, ...) even if deep is
> zero. So I expect that you will have an infinite loop that just recurses
> over and over, until you run out of memory or hit the recursion limit.
> 
> 
> 
> -- 
> Steven

i recorded a video to show what i am searching for

https://drive.google.com/file/d/0Bxs_ao6uuBDUbjFxbjJReXZoNHc/view?usp=sharing

something i hide because i invented some math, hope your mercy.



More information about the Python-list mailing list