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

Archana Verma IDEaS Assignment 1 #20

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

averma319
Copy link

I solved numbers 1-3, 5. Will work on number 4 and update soon.

The prime factorization one was interesting because I knew how to do it by hand but not sure how to make a generalizable, systematic approach (it took me time to get the while loop condition about the max factor possible). The first two were fairly simple because the method was very systematic. Overall I had a good practice of python syntax and found the assignment helpful.

Solutions for assignment 1 of the IDEas Workshop.
Copy link
Member

@dsirianni dsirianni left a comment

Choose a reason for hiding this comment

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

Sounds good! Whenever you get Problem 4 done, just add it to your PR by committing/pushing it to your fork. In the meantime, please change the file extensions from *.py.txt to just *.py; I'd like to try to keep these submission directories for Python files only.

When you do, you'll have to remove the markdown-style headers from your scripts to maintain proper syntax; they can be removed, commented out with #, or surround them with triple quotes """. If you want to document your scripts with headers, you can check out an example of Python header style conventions here, from the Psi4NumPy project.

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