[Montserrat Labrada] - String Operations Challenge 1#67
Open
MonVala wants to merge 1 commit intota-data-mexpt:masterfrom
Open
[Montserrat Labrada] - String Operations Challenge 1#67MonVala wants to merge 1 commit intota-data-mexpt:masterfrom
MonVala wants to merge 1 commit intota-data-mexpt:masterfrom
Conversation
Owner
|
Hola Montse! Un favorsote, cuando resuelvas tus notebooks guárdalos con las celdas ejecutadas para que así pueda ver el resultado de tu código desde github porfa :) Acá te dejo un par de comentarios de lo que pude ver:
Me avisas cuando cargues el update del challenge 2 (: |
Author
|
Muchas gracias Nay! Yo te aviso cuando termine el challenge 2 :). Una pregunta, todo bien con el lab de git?
Saludos!
…________________________________
From: ta-data-mexpt ***@***.***>
Sent: Monday, October 3, 2022 10:50:23 PM
To: ta-data-mexpt/string-operations ***@***.***>
Cc: Valdes Labrada, Montserrat ***@***.***>; State change ***@***.***>
Subject: Re: [ta-data-mexpt/string-operations] [Montserrat Labrada] - String Operations Challenge 1 (PR #67)
CAUTION: External to USask. Verify sender and use caution with links and attachments. Forward suspicious emails to ***@***.***
Hola Montse!
Un favorsote, cuando resuelvas tus notebooks guárdalos con las celdas ejecutadas para que así pueda ver el resultado de tu código desde github porfa :)
Acá te dejo un par de comentarios de lo que pude ver:
* Me gusto que te aventaras la lambda para el filtro, otra opción podría ser aplicar directamente el método sobre los elementos de la lista, algo como 'Grocery list: '+ ', '.join([food.lower() for food in food_list if food.lower().startswith('b')==True]) + '.'
* Tu función para el cálculo del área estuvo bien pero nos faltó el return :( acuérdate que sino declaramos lo que queremos que nos devuelva la función no podría identificar qué necesitamos, en este caso bastaría con añadir el "return" en lugar de "area = ", otra cosa para llamar a las funciones necesitas añadir los paréntesis, en el print se vería cómo:
print(string1 + ' ' + str(radius) + " " + string2 + ' ' + str(area(4.5)))
* Bien por usar librerías para resolver.
* Para la parte de regex me parece que es un poco más eficiente la función que viene en el hint: .search() además usar '[0-9]' o '/d' nos ayudan a encontrar cualquier dígito (igual no te apures mucho con esto, lo vamos a ver más adelante).
Me avisas cuando cargues el update del challenge 2 (:
—
Reply to this email directly, view it on GitHub<#67 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/A3EXKZT2BB32EVPG3AYQBGDWBOLO7ANCNFSM6AAAAAAQ3GXX6M>.
You are receiving this because you modified the open/close state.Message ID: ***@***.***>
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hice lo que pude :(