python-ecology-lesson icon indicating copy to clipboard operation
python-ecology-lesson copied to clipboard

small grammatical change

Open raynamharris opened this issue 6 years ago • 7 comments

this is a small grammatical change as suggested during the translation to Spanish in https://github.com/Carpentries-ES/python-ecology-lesson-es/pull/30#discussion_r232508766

An alternative could be to say "The SciPy ecosystem..."

raynamharris avatar Nov 11 '18 23:11 raynamharris

I think it sounds better with which:

SciPy is an ecosystem for Python which provides the tools necessary for scientific computing.

orchid00 avatar Nov 12 '18 22:11 orchid00

What is the motivation for this change?

maxim-belkin avatar Nov 16 '18 16:11 maxim-belkin

One our our Spanish translators thought it was missing a word to make a complete sentence. But, the more I read it, I think it's okay as is. I think it's okay if you want to close without merging @maxim-belkin

raynamharris avatar Nov 16 '18 17:11 raynamharris

We're conveying two messages here: 1. that SciPy is an ecosystem (not just one "thing"). 2. that it provides tools for scientific computing. Then, I don't think that (1) deserves a separate keypoint on its own as this is a rather general statement. So, we remove emphasis on the fact that SciPy is an ecosystem (by not saying "SciPy is an ecosystem) and state it as if this is a known fact.

Nonetheless, I think the right fix is the one you mention in your original message, that is: The SciPy ecosystem. Then, the ecosystem is not really for Python (it's within it), so I think the right change should really be:

The SciPy ecosystem provides tools for scientific computing in Python.

This is more or less the same as what is said on their webpage: https://www.scipy.org/about.html

However, this statement is rather weak. I think we could say:

The SciPy ecosystem provides tools that enable scientific computing in Python.

but this is open for a discussion.

maxim-belkin avatar Nov 16 '18 17:11 maxim-belkin

Nice. I really like your last option "The SciPy ecosystem provides tools that enable scientific computing in Python."

raynamharris avatar Nov 16 '18 17:11 raynamharris

I went ahead and inserted that language as a suggestion. Feel free to accept if it looks good, @raynamharris. If it still looks off, we can discuss more.

wrightaprilm avatar Nov 24 '18 02:11 wrightaprilm

Hey @raynamharris. I'm just getting back into the swing of things over here - it looks like there's just one outstanding issue to resolve. I made a suggestion; if it looks good, go ahead and accept and I'll merge this.

wrightaprilm avatar Jan 17 '19 21:01 wrightaprilm

This suggested change was rendered obsolete when #415 was merged.

tobyhodges avatar Feb 28 '23 10:02 tobyhodges