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

Plot for Report Question 1 #6

Open
GITGoing4321 opened this issue May 25, 2016 · 4 comments
Open

Plot for Report Question 1 #6

GITGoing4321 opened this issue May 25, 2016 · 4 comments
Labels

Comments

@GITGoing4321
Copy link

I'm confused about the plot for Report Question 1. It says that the solutions should all be on the same graph, but it also suggests that all of the solutions should be plotted in the same color, red. I feel like this would make the graph hard to read. I just wanted to confirm that you truly do want them all in red.

@alyfarahat
Copy link

If you take a look at the plots generated by the tests, you should see that serial and parallel plots have the same color, yet visible because the serial line is think-dashed and the parallel line is continuous.

@werrah
Copy link

werrah commented May 25, 2016

Yes, but it says: "create a single plot using the example initial condition displaying the results of Nt = 100, 200, 300, 400" and specifies they should all be on the same plot.

You can't tell which curve is for each Nt. They're all red. I did it both ways - one all in red, and one with a different color for each Nt so it was clearer to me.

@cswiercz
Copy link
Member

For each increasing number of time steps Nt the heat profile will "decay" a bit more. The corresponding solutions should be fairly distinct and slowly approach a constant profile.

You can plot them in different colors if you like. (@mvelegar) Or, you can try increasing Nt past the suggested number.

@mvelegar
Copy link

Please feel free to plot using different colors, or change Nt values. It'll be much easier for me to grade clear, well-presented plots!

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

No branches or pull requests

5 participants