Skip to content

Conversation

@Jesscode-analyst
Copy link
Owner

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

I completed assignment -2 ; I learned the concept of recursive data structures.

What did you learn from the changes you have made?

I learned how to work as a team and review the work of your team member.

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

I had almost the similar approach that my partner used

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

I was initially not sure of how to assess someone else's code. I read his code to understand and explored it with examples

How were these changes tested?

I ran the codes and tested them

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

Checklist

  • I can confirm that my changes are working as intended
    Yes

Copy link

@tianyi21 tianyi21 left a comment

Choose a reason for hiding this comment

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

Hi Jaskirat,

Thank you for submitting your ADS A2.

✅ You implemented an algorithm to find the closest duplicate in a given tree.
✨ You implemented a helpful function to convert a given list into a tree.
✨ You implemented a BFS-based algorithm to solve the question.
✨ You utilized the queue and set data structures in your implementation.
✅ Your program runs successfully without any errors.
✅ Your program produces the intended results.
✅ You correctly explained your partner's implementation and analyzed its time and space complexities.
✅ You included a paragraph to explain your experience and thoughts to us.

🎉 All your ADS assessments have been completed. Thank you for your participation, and I wish you all the best for the future!

Thanks,
Tianyi [LS]

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