Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added Rasesh_Udayakumar_Shetty_hw4.ipynb #217

Closed
wants to merge 1 commit into from

Conversation

Rasesh2005
Copy link

@Rasesh2005 Rasesh2005 commented Jul 19, 2024

PR Description

Womanism 6.12 Assignment

Some notes

  • Please make sure that you placed the files in an appropriate folder

  • And that the files have indicative names.

  • Please note that Classiq runs automatic code linting, which may minorly alter some files.

    • If you're familiar with pre-commit, you may run pre-commit install, and then at each commit, your files will be altered in a similar way

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@Nadav138 Nadav138 added the Womanium Womanium 2024 Summer Program label Aug 5, 2024
@Nadav138
Copy link
Contributor

Nadav138 commented Aug 5, 2024

Good job! The states could be implemented with a loop and if to be more elegant :)
Will be merged in next version :)
Score: 7/10

@Nadav138 Nadav138 closed this Aug 5, 2024
@Nadav138 Nadav138 reopened this Aug 5, 2024
@Nadav138 Nadav138 closed this Aug 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Womanium Womanium 2024 Summer Program
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants