Mohammed A#5
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
📝 HackYourFuture auto gradeAssignment Score: 87 / 100 ✅Status: ✅ Passed Test Details |
AgneseGi
left a comment
AgneseGi
left a comment
There was a problem hiding this comment.
very solid work! the only minor issues were:
- missing the required outlier decision/comment
- hiding AI_ASSIST.md content inside HTML comments
- catching broad Azure exceptions as “container already exists”
| ## The prompt I gave | ||
|
|
||
| <!-- Paste the exact prompt you gave the LLM here. --> | ||
| <!-- How do I convert a Pandas column to datetime and replace invalid dates with missing values? --> |
There was a problem hiding this comment.
content is inside HTML comments, so it is effectively invisible when rendered
|
|
||
| logging.info("Cleaned sales rows: %s", len(sales)) | ||
|
|
||
| return sales |
There was a problem hiding this comment.
this does not include the required outlier decision - you could have added a comment or decide to remove them using eg .quantile(0.99)
| container.create_container() | ||
| logging.info("Created container %s", container_name) | ||
| except Exception: | ||
| logging.info("Container %s already exists", container_name) |
There was a problem hiding this comment.
Here any error is treated as “container already exists":this might hide real failures like eg ClientAuthenticationError
you could use instead
except ResourceExistsError:
logging.info("Container %s already exists", container_name)
|
Thanks for the feedback. I updated AI_ASSIST.md so the content is visible, added an outlier decision in the cleaning step, and replaced the broad Azure exception with ResourceExistsError. |
3 participants
Summary
Completed the Week 4 Pandas pipeline assignment.
Changes