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

02-loop example should not suggest naive evaluation of polynomials #378

Open
godmar opened this issue May 16, 2017 · 3 comments
Open

02-loop example should not suggest naive evaluation of polynomials #378

godmar opened this issue May 16, 2017 · 3 comments

Comments

@godmar
Copy link

@godmar godmar commented May 16, 2017

According to the main website, Software Carpentry's goal is to teach basic lab skills "for research computing."

For this reason, I would remove or replace the example: "Computing the Value of a Polynomial" as it suggests a naive method of evaluating polynomials that is inefficient in general and particularly unsuited for floating point numbers due to its lack of numerical stability and accuracy - but in the context of this lesson, floating point numbers are otherwise frequently used. A novice (and SWC is targeting novices) may conclude that this is an actual usable method.

Even though the point of the exercise appears to be to practice Python's enumerate function, I would still avoid this particularly bad example, or at the very least, attach a strongly worded warning: "you would not use this method of evaluating polynomials in real life applications."

@tbekolay
Copy link
Contributor

@tbekolay tbekolay commented May 16, 2017

I'm cool with adding a warning, though I would prefer one kindly worded rather than strongly worded (i.e., point the reader to better alternatives rather than telling them that this option is a bad one).

I'd be okay with removing the example if another example using enumerate were put in its place.

@godmar
Copy link
Author

@godmar godmar commented May 18, 2017

What is the recommended format for including references to other sources?

@ntmoore
Copy link

@ntmoore ntmoore commented May 18, 2017

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

Successfully merging a pull request may close this issue.

None yet
3 participants