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

Review of Report First Draft #10

Open
ZoeRichter opened this issue Jul 3, 2019 · 3 comments
Open

Review of Report First Draft #10

ZoeRichter opened this issue Jul 3, 2019 · 3 comments

Comments

@ZoeRichter
Copy link
Collaborator

Hi, Mitchell. This issue is a request for you to review my report, a pdf of which I've attached.

Past what you'd normally do, I'd really like your feedback on the diagrams I made of the processing schemes. I want to make sure they're clear to everyone, not just me. I am also aware that the image size and location is messed up right now - I plan to work on tweaking that during this round of revisions, but it seemed silly to wait for a first review just for that.

Also, Professor Huff gave me a step-by-step of the review process, and I'm going to copy-paste it here, because I know I'll need to reference it again.

  • Create an issue asking for a document review from Mitchell Whalen [done]
  • Build the pdf, attach it to the issue [done]
  • [Mitchell will] complete the review and close the issue
  • Mitchell should open a new issue with his recommended changes
  • You can close that issue by creating a [pull request] that implements those changes
  • Mitchell will review your [pull request]

Please and thank you!
msr-list.pdf

@ZoeRichter
Copy link
Collaborator Author

I just realized I forgot to update the bibliography before compiling, sorry.
msr-list.pdf

@mitchellwhalen
Copy link

Hey, I apparently suck at GitHub and while in DC I only glanced at it and didn't see this until like now. I'll try to finish this up today or tomorrow if I end up having to deal with a bunch of leasing crap in the next couple of days.

@ZoeRichter
Copy link
Collaborator Author

@mitchellwhalen

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants