cyclus / fundamentals-paper

A repository to hold the fundamentals paper
Creative Commons Zero v1.0 Universal
1 stars 8 forks source link

Review and Resubmit #141

Closed katyhuff closed 8 years ago

katyhuff commented 8 years ago

This PR contains the minor revisions suggested by the reviewers from Advances in Engineering Software. This doesn't complete all suggested revisions. A few less trivial revisions remain to adequately satisfy those needs.

Each of the recommendations was addressed with a separate commit message, where possible. The recommendations addressed include:

Reviewer 1

gidden commented 8 years ago

I have a PR into your branch with some suggested edits. Some other thoughts:

Reviewer 1: Line 398: I feel like "to meaningfully optimize the cost and environmental impact" is a bit strong (and we know that some people don't think these tools should be used for "cost optimization"). Maybe we could just say "meaningful models of spent nuclear fuel storage transport, and disposal strategies, require representation of discrete casks and their varying isotopic compositions."

Reviewer 2: I agree that most of the suggestions would require nontrivial rewrites or additional work. I'm happy to look at whatever you decide this week regarding those.

I think this is largely ready to go!

katyhuff commented 8 years ago

On Mon, Nov 30, 2015 at 12:35 AM, Matthew Gidden notifications@github.com wrote:

I have a PR into your branch with some suggested edits. Some other thoughts:

Reviewer 1: Line 398: I feel like "to meaningfully optimize the cost and environmental impact" is a bit strong (and we know that some people don't think these tools should be used for "cost optimization"). Maybe we could just say "meaningful models of spent nuclear fuel storage transport, and disposal strategies, require representation of discrete casks and their varying isotopic compositions."

Good call.

Reviewer 2: I agree that most of the suggestions would require nontrivial rewrites or additional work. I'm happy to look at whatever you decide this week regarding those.

Thanks. We'll see what we can do.

I think this is largely ready to go!

Yay!

— Reply to this email directly or view it on GitHub https://github.com/cyclus/fundamentals-paper/pull/141#issuecomment-160554545 .

http://katyhuff.github.com

gonuke commented 8 years ago

I PR'ed 2 typos, but otherwise am happy with all the changes. Thanks @katyhuff !

katyhuff commented 8 years ago

Pulling these minor changes. Resolutions to the broader questions upcoming in a separate PR.