Skip to content

Update function documentation, minor changes to README#19

Merged
alyciakb merged 5 commits into
masterfrom
update-rotate-scripts
Feb 17, 2019
Merged

Update function documentation, minor changes to README#19
alyciakb merged 5 commits into
masterfrom
update-rotate-scripts

Conversation

@everittB
Copy link
Copy Markdown
Collaborator

  • Added in function documentation
  • Updated usage instructions for rotate on the README

@alyciakb alyciakb added this to the Milestone2 milestone Feb 17, 2019
@alyciakb
Copy link
Copy Markdown
Collaborator

  • I updated the installation directions to be in compliance with the milestone2 directions
  • Fixed a couple spelling typos

@alyciakb
Copy link
Copy Markdown
Collaborator

TO DO:

  • In rotate.py, change the output to include the original image as the first image in the returned array.
  • Change any corresponding test_rotate.py tests to reflect that change

Copy link
Copy Markdown
Collaborator

@alyciakb alyciakb left a comment

Choose a reason for hiding this comment

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

  • In rotate.py, change the output to include the original image as the first image in the returned array.
  • Change any corresponding test_rotate.py tests to reflect that change

@alyciakb alyciakb merged commit 4a05383 into master Feb 17, 2019
@everittB
Copy link
Copy Markdown
Collaborator Author

Addressed feedback. fad7156

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