Lab 02.04 - Task 3: Run the card - Solucionado

[natandreli]

Buenas tardes:

Solucioné la task 3 del Lab 02.04, cuando lo corro manualmente todo sale perfecto:

Sin embargo, al evaluarlo en el grader me sale el siguiente error:

Ya se utiliza un assert en la función make_heaps(), igualmente lo agregué a la función run() por si este era el problema.

Les dejo el bloque de código de la función run():

def run(c, n, n_heaps=3):
  assert n_heaps%2==1, "must have an odd number of heaps"
  assert len(c)%n_heaps==0, "the length of the deck must be a multiple of the number of heaps"
  for i in range(0, 3):
    h = make_heaps(c, n_heaps=3)
    c = collect_heaps(h, n)
  
  r = c.index(n)
  return r

Gracias, espero me puedan ayudar.

[imEag]

Hola Natalia! EL error se está generando en la función run a la hora de llamar a la función make_heaps. Observa que a la hora de llamar una función podemos pasarle argumentos con la notación nombre_del_argumento=valor en este caso estas asignando n_heaps=3 lo que genera que los heaps siempres se generen con este parámetro en 3 sin importar el valor que haya recibido la función run en ese argumento. Lo que deberías estar haciendo es n_heaps=n_heaps para tener el cuenta el valor que recibe la función run

[natandreli]

¡Vale! Ya vi a qué te referías, estoy pasando el 3 al momento de llamar la función dentro del ciclo.

¡Muchas gracias!

[imEag]

muestrame por favor el código implementando lo que te sugerí y el nuevo mensaje de error, gracias

[imEag]

exacto, a eso me refería inicialmente

[natandreli]

@imEag muchísimas gracias ☺️

[imEag]

Con mucho gusto

[Felipe-RA]

Saludos @natandreli

Me gustaría ver tu implementación de make_heaps() y de collect__heaps() para poderte dar un consejo más preciso sobre una posible solución a tu implementación.

Gracias!

[natandreli]

Claro que sí, mira:

def make_heaps(c, n_heaps=3):
    assert n_heaps%2==1, "must have an odd number of heaps"
    assert len(c)%n_heaps==0, "the length of the deck must be a multiple of the number of heaps"
    
    n_cards = int(len(c)/n_heaps)
    h = []

    for j in range(0, n_heaps):
      l_cards = []
      for i in range(0, n_cards):
        l_cards.append(c[i*n_heaps + j])
      h.append(l_cards)

    return h

def collect_heaps(h, n):

  for i in range(0, len(h)):
    if n in h[i]:
      p_card = i

  if p_card >= len(h)//2:
    h.insert(len(h)//2, h[p_card])
    h.pop(p_card + 1)
  else:
    h.insert(len(h)//2 + 1, h[p_card])
    h.pop(p_card)

  r = []

  for i in range(0, len(h)):
    r.extend(h[i])

  return r            

¡Muchas gracias!

[natandreli]

¡Ya encontré mi error! Dentro del ciclo estoy pasando un argumento igualado a 3, por eso no me funciona.

Muchas gracias igualmente.