git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
From: Varun Varada <varuncvarada@gmail.com>
To: "Michal Suchánek" <msuchanek@suse.de>
Cc: Felipe Contreras <felipe.contreras@gmail.com>, git@vger.kernel.org
Subject: Re: [PATCH] doc: replace jargon word "impact" with "effect"/"affect"
Date: Tue, 11 May 2021 16:38:34 -0500	[thread overview]
Message-ID: <CAD2i4DALKgw2wG6QGs-oQhAHnS3AG1j1BSq2bxjPojVOtw+WjA@mail.gmail.com> (raw)
In-Reply-To: <20210511202502.GM12700@kitsune.suse.cz>

On Tue, 11 May 2021 at 15:25, Michal Suchánek <msuchanek@suse.de> wrote:
>
> On Tue, May 11, 2021 at 02:59:32PM -0500, Felipe Contreras wrote:
> > Varun Varada wrote:
> > > On Tue, 6 Apr 2021 at 04:24, Michal Suchánek <msuchanek@suse.de> wrote:
> > > > while using "will not impact" in an incorrect or unclear way may be a
> > > > problem the word "impact" in itself is not "jargon".
> > >
> > > The word means "to have a strong or marked effect on" (v.) and "a
> > > strong or market influence" (n.) when used figuratively; it is not
> > > synonymous with "affect" and "effect", respectively, as shown even by
> > > all of the entries you've cited. Using it as such is the incorrect
> > > part, so those are the instances I've changed in the diff.
> >
> > There are two ways impact can be used as a verb: transitive and
> > intransitive, but git doesn't seem to be using the intransitive form. In
> > the transitive form it usually means to strike "the car impacted the
> > tree". But it can also mean to have a desired effect "reducing CO2
> > emissions impacted climate change".
>
> I don't know where you find the 'desired' effect meaning. Certainly none
> of the dictionaries I consulted at random provides such definition.
>
> >
> > None of these are used in the documentation, we have things like:
> >
> >   the index does not impact the outcome
> >
> > Which is clearly wrong (unless we are talking about possitive outcome of
> > the outcome, which makes no sense).
>
> It is not clearly wrong. To me it makes perfect sense. If you want to
> claim it's wrong please provide a source for your claim. Otherwise it's
> just matter of different opinions of more fitting formulation.

You agreed that the word "impact" means to "significantly affect". The
idea of whether something is *significantly* affected, as opposed to
conveying the idea that something is affected at all, only arises in
situations where degrees of an influence are possible; that's not the
case in any of the examples my change is editing (except for the one I
conceded where it says "badly impacted"). Assuming that a reader would
know which of those involve degrees of influence are possible and
which aren't, even if every reader of the documentation/command output
was familiar, is entirely unnecessary. That's the point here: there is
this point of confusion that is entirely avoidable with a simple
one-word change.

Re: your point about me not pointing out specific examples: the
command output for detached HEAD state reads "you can discard any
commits you make in this state without impacting any branches by
switching back to a branch". I'm incredibly passionate about this
example. Here, the user is left to think, "wait...so this will not
impact (significantly affect) any branches, but will it affect them?
As in, are there side effects that I should be aware of? Where do I go
to find out what they are?" All of this mental energy is completely
unnecessary. Mind you, this is regarding discarding commits, which is
a destructive action.

You might feel that this is just one example that might need fixing,
but I assure you, I've analysed all the other examples and they all
have similar problems. It's entirely unnecessary to have this
confusion.

>
> >
> > As a noun it can mean a siginificant or major effect: "the impact of
> > science".
> >
> > However, the documentation is not using it that way:
> >
> >   the runtime impact of tracking all omitted objects
> >
> > The noun usage is less wrong than the verb usage, but it's still wrong.
>
> Why is that wrong?
>
> How did you infer that the effect is insignificant or minor?

No one did, and that's the point. All impacts are effects, but not all
effects are impacts. You yourself acknowledged that there is a
prevalent tendency to hyperbolize, and the fact that one doesn't know
which it is in this case (or frankly any of the other cases my commit
touches) is problematic. This kind of confusion surely doesn't belong
in technical documentation. And if it is indeed an "impact", where is
that conveyed? What's so notable about the effect on the runtime?
That's the point.

>
> In fact while some dictionaries list 'impact' as 'have strong effect'
> the Oxford dicrionary lists is as simply synonymous to 'affect'.
>
> > The verb usage could be corrected by changing "the index does not
> > impact", to "the index does not have an impact on".
>
> Why is that change needed at all?
>
> > But why bother? The word "affect" is a much superior choice.
>
> Why bother with a chenge at all?

It seems like you already previously agreed with the premise that the
word means "a significant effect" or "to significantly affect". I
understand and appreciate your thoroughness to scrutinize changes to
the repo, but I'm frankly surprised that such a small change is
attracting such fierce debate. This is meant to be a change that is
probably one of the easiest ones to decide on: it only consists of
one-word changes that don't change functionality, yet undeniably
reduce confusion.

Re: your previous point about linguistic authorities: yes, there is no
authority on usage, but therein lies my point. This doesn't even need
to rise to the domain of usage, because it is squarely within the
realm of semantics. Words mean something, and we all use dictionaries
to learn about / confirm those meanings. Insofar as all the major
dictionaries cite the word as "a significant effect" / "to affect
significantly", that semantic concept doesn't belong in the cases
where I've made changes. And if it does, then those need to be
clarified (because that's where the real confusion/ambiguity is).
I.e., it's not "why is not every case a significant effect?", but "why
are some cases a significant effect?"

>
> Thanks
>
> Michal

  reply	other threads:[~2021-05-11 21:39 UTC|newest]

Thread overview: 68+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-05 21:48 [PATCH] doc: replace jargon word "impact" with "effect"/"affect" Varun Varada
2021-04-06  9:24 ` Michal Suchánek
2021-04-06 19:36   ` Varun Varada
2021-04-06 23:01     ` Jeff King
2021-04-07  0:06       ` Varun Varada
2021-04-28  0:39         ` Varun Varada
2021-04-28  8:58           ` Michal Suchánek
2021-04-28 18:15             ` Varun Varada
2021-04-28 18:49               ` Michal Suchánek
2021-04-30  1:51                 ` Varun Varada
2021-04-30  7:59                   ` Michal Suchánek
2021-05-10 17:19                     ` Varun Varada
2021-05-10 17:35                       ` Michal Suchánek
2021-05-10 18:37                         ` Varun Varada
2021-05-11 10:43                           ` Michal Suchánek
2021-05-11 13:22                             ` Junio C Hamano
2021-05-12  3:02                               ` Felipe Contreras
2021-05-12  2:59                             ` Felipe Contreras
2021-05-12  2:48                         ` Felipe Contreras
2021-05-12  2:38                 ` Felipe Contreras
2021-05-12  2:34             ` Felipe Contreras
2021-05-12  2:24       ` Felipe Contreras
2021-05-11 19:59     ` Felipe Contreras
2021-05-11 20:25       ` Michal Suchánek
2021-05-11 21:38         ` Varun Varada [this message]
2021-05-12  3:43           ` Felipe Contreras
2021-05-12  4:09             ` Michal Suchánek
2021-05-12  5:13               ` Felipe Contreras
2021-05-12  6:47                 ` Michal Suchánek
2021-05-12  9:06                   ` Felipe Contreras
2021-05-12 10:08                     ` Michal Suchánek
2021-05-12 10:33                       ` Michal Suchánek
2021-05-12 11:05                       ` Felipe Contreras
2021-05-12 11:20                         ` Michal Suchánek
2021-05-12 11:45                           ` Robert P. J. Day
2021-05-12 15:19                             ` Kerry, Richard
2021-05-12 16:47                   ` Varun Varada
2021-05-12 17:01                     ` Michal Suchánek
2021-05-12 17:32                       ` Felipe Contreras
2021-05-12 18:04                         ` Michal Suchánek
2021-05-12 19:42                           ` Felipe Contreras
2021-05-13  7:46                             ` Michal Suchánek
2021-05-13  8:28                               ` Felipe Contreras
2021-05-13  8:55                               ` Robert Coup
2021-05-13  9:48                                 ` Michal Suchánek
2021-05-13  9:59                                   ` Felipe Contreras
2021-05-26 23:49                                   ` Varun Varada
2021-05-27 11:46                                     ` Michal Suchánek
2021-05-27 14:08                                       ` Felipe Contreras
2021-05-27 14:35                                         ` Michal Suchánek
2021-05-27 16:43                                           ` Felipe Contreras
2021-06-12 23:13                                           ` Varun Varada
2021-06-13 11:40                                             ` Michal Suchánek
2021-06-13 14:06                                               ` Felipe Contreras
2021-06-13 16:28                                                 ` Michal Suchánek
2021-06-13 17:12                                                   ` Felipe Contreras
2021-05-12 22:52                       ` Varun Varada
2021-05-13  6:19                         ` Felipe Contreras
2021-05-12  3:21         ` Felipe Contreras
2021-05-11 19:21   ` Felipe Contreras
2021-05-11 19:57     ` Michal Suchánek
2021-05-12  3:09       ` Felipe Contreras
2021-05-12  4:11         ` Michal Suchánek
2021-05-12  5:22           ` Felipe Contreras
2021-05-12 16:39           ` Varun Varada
2021-05-13 10:40 ` Philip Oakley
2021-05-26 23:52   ` Varun Varada
2021-05-27 11:20     ` Philip Oakley

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: http://vger.kernel.org/majordomo-info.html

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAD2i4DALKgw2wG6QGs-oQhAHnS3AG1j1BSq2bxjPojVOtw+WjA@mail.gmail.com \
    --to=varuncvarada@gmail.com \
    --cc=felipe.contreras@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=msuchanek@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://80x24.org/mirrors/git.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).