Skip to content

Completed assignment_1.ipynb, all functions tested and match expected outputs.#1

Open
cheonandrew wants to merge 1 commit intomainfrom
assignment-1
Open

Completed assignment_1.ipynb, all functions tested and match expected outputs.#1
cheonandrew wants to merge 1 commit intomainfrom
assignment-1

Conversation

@cheonandrew
Copy link
Copy Markdown
Owner

@cheonandrew cheonandrew commented Mar 16, 2026

… outputs.

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)

I am uploading the completed .ipynb with all code, comments, and outputs.

What did you learn from the changes you have made?

I learned how to define functions and set a sequence of conditionals to evaluate boolean statements.

Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?

I was considering making new variable names for word_a and word_b for both case-sensitive and -insensitive variations, but for the purposes of this assignment, it could have been verbose.

Were there any challenges? If so, what issue(s) did you face? How did you overcome it?

I am a native MATLAB user so I out of habit I tried to call lower(word_a) to execute the lowercase function, but it took a quick search to realize that .lower() is the correct syntax.

How were these changes tested?

Running/re-running code cells within the .ipynb.

A reference to a related issue in your repository (if applicable)

Checklist

  • I can confirm that my changes are working as intended

Copy link
Copy Markdown

@juliagallucci juliagallucci left a comment

Choose a reason for hiding this comment

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

Good! One note for improvement would be to simplify your conditional logic. Since word 1 != word 2 is already a Boolean, you do not need to explicitly return True or False.

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.

2 participants