Skip to content

main marco#9

Open
marcoayora wants to merge 1 commit intoIronhack-data-bcn-oct-2023:mainfrom
Ironhack-DA:main
Open

main marco#9
marcoayora wants to merge 1 commit intoIronhack-data-bcn-oct-2023:mainfrom
Ironhack-DA:main

Conversation

@marcoayora
Copy link
Copy Markdown

No description provided.

@sh-ih
Copy link
Copy Markdown

sh-ih commented Oct 23, 2023

Keep up the good work!

Some comments:

On the question that asked you to find all numbers divisible from 1 to 1000 by 7 you didn’t use the complete condition. That’s why it gave you all the numbers: instead of i/7, you needed to use the modulus: i%7==0

On the question ‘Convert the numbers of the following nested list to floats. Use floats as the name of the list.’, you left the numbers as strings. You needed to add the conversion to the comprehension list:

floats = [float(i) for row in list_of_lists for i in row]

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.

3 participants