Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding Interactive Assignments "Lista de Exercícios I" as Quiz1 #69

Merged
merged 9 commits into from
May 24, 2021

Conversation

AngelaRemolina
Copy link
Collaborator

Partially solves #7

I have added all exercises from "Lista de Exercícios I Python para Zumbis.pdf" with unit test from each excercise, in file Quiz1.rst

I fixed some small bugs that were happening because of an idented block on quiz5mcq.rst (see reference images below)

Bug

Bug2

@@ -17,15 +17,17 @@ Quiz - 5
:answer_d: 3
:correct: a

¿Qué imprime el siguiente programa (dado en forma de pseucódigo)? |br|
¿Qué imprime el siguiente programa (dado en forma de pseucódigo)? |br|
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Este espacio está a propósito?

Suggested change
¿Qué imprime el siguiente programa (dado en forma de pseucódigo)? |br|
¿Qué imprime el siguiente programa (dado en forma de pseucódigo)? |br|

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sí, este espacio es el error de identación que menciono. En la imagen que adjunté en el PR, el error de "No correct answer specified" se daba porque hacia falta un espacio para que la pregunta estuviera dentro del bloque mchoice.

x = x * 2 |br|
imprime (x + y) |br|


Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Estas dos líneas están a propósito

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Estas dos líneas que añadí fueron por error. Voy a hacer un commit para borrarlas y dejar un estándar de espacios entre problema y bloque de activecode.

Dejo como estandar 4 lineas vacias entre la pregunta y el bloque de activecode. Corrección de inclusivo a inclusive.
@@ -77,6 +76,7 @@ Quiz - 5




Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Esta línea es para cumplir el patrón de 4 líneas entre el enunciado del problema y el bloque de activecode.

Copy link
Collaborator

@y-martinez y-martinez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!! 🚀

Un cambio sencillo :)

.. activecode:: q1_11
:nocodelens:

Sabiendo que str() convierte valores numéricos en cadenas, calcule cuántos dígitos hay en 2 elevados a un millón. |br|
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Puedes encerrar str() en doble ` para formatearlo en código así se ve más legible como función 😄

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Listo ;) ya quedó corregido. Muchas gracias por la sugerencia, se ve mucho mejor.

Copy link
Collaborator

@LeoCumpli21 LeoCumpli21 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Todo me parece que está bien. El único cambio adecuado es el que menciona @y-martinez.

@reingart reingart closed this May 24, 2021
@reingart reingart reopened this May 24, 2021
@github-actions
Copy link

@github-actions
Copy link

@reingart reingart merged commit 347a99d into PyAr:master May 24, 2021
reingart added a commit that referenced this pull request Oct 27, 2021
Adding Interactive Assignments "Lista de Exercícios I" as Quiz1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants