Technical Report Review

[grindstaffhl]

Users
"There is a full lab work sheet that accompanies the lab activity which the simulator is meant to supply the data for."

• maybe make "work sheet" one more word to avoid some confusion

The Current Simulator

• try to have pictures on their own line. When it starts on the same line as the text it's a little hard to read. Probably just some weird git markdown tweaking

" The user can not even choose between them with the arrow keys on the keyboard, they must enter a number one through six."

• Doesn't flow super well, maybe reword to something like "User cannot navigate through options with arrow keys, they must enter a number 1-6"?

• Possibly just a git markdown issue, but the text under the picture of the solar system refers to the picture above it. Could probably just cut and paste pieces of the paragraphs and put them under the corresponding pictures

Objectives
"This means eliminating the menu system entirely, and having a single interface that does what it took four to do in the old simulator."

• Sentence reads a little awkwardly. Maybe reword to something like "refactoring/reducing four interfaces from the old simulator into one clean interface that performs the same tasks"

"Another objective is to add controls that allow for users to have greater control over the simulation including the simulation speed to eliminate the wait for outer planets reach points in their orbit, the ability to jump directly to dates, and control over how much data is displayed at a given time."

• bit of a run on sentence, just needs a little proofreading

Feature

• I don't think bullet points need periods at the end
  "A display of planetary positions relative to each other relative to a certain date along with the orbital rings they travel on."
• reads a little awkwardly. Maybe splice up this feature into multiple or make them sub-bullet points?

"A preset library of zoom levels to best focus on bodies making it easier to navigate."

• mentioned not wanting presets in an earlier bullet point. I know these aren't the same presets but maybe reword a little just for more consistency

"A control that allows users to have the simulation display lines between planets, and lines that connect planets to the sun. This makes it easier to answer various lab questions that deal with exact planetary positioning."

• second sentence could probably be a sub-bullet just for cleanliness

Design
"The benefits of this are two fold, the design is tried and true, and by keeping with convention, any future changes made by later individuals should be more straightforward to impliment."

• spelling, also i think the comma after two-fold should be a colon cause it follows a list?

Development and Breakdown

• model section is entirely bold for some reason?

Overall it was definitely easy to understand what you were doing. I only recommend some grammar/proofreading/clarity stuff. Good report overall!