Skip to content

Submitting the first Python assignment after incoporating my code to the#1

Open
vikramkatju wants to merge 6 commits intomainfrom
assignment-1
Open

Submitting the first Python assignment after incoporating my code to the#1
vikramkatju wants to merge 6 commits intomainfrom
assignment-1

Conversation

@vikramkatju
Copy link
Copy Markdown
Owner

@vikramkatju vikramkatju commented Nov 29, 2024

given .ipynb file

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

I added python code to the assignment_1.ipynb file in accordance with the assignment questions

What did you learn from the changes you have made?

I learnt about how to create user defined python functions and about the inbuilt function sorted

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

I was thinking of a way to conflate the two functions used in this program into a single function with the is_case_sensitive parameter set to a default value of False.

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

No particular challenges

How were these changes tested?

I ran the code on the Jupyter notebook on which I added the python code.

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

Checklist

  • I can confirm that my changes are working as intended

@pedram-aliniaye-asli
Copy link
Copy Markdown

You did a great job here!
Just consider that in part 1 the function should return a boolean value as the result and not just print out a string.
Please go ahead and do the modifications and then let me know. Thanks!

value (True or False) instead of printing whether the two strings are
anagrams or not anagrams.
@vikramkatju
Copy link
Copy Markdown
Owner Author

Hi Pedram, I've made the correction. Thank you.

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