Git Product home page Git Product logo

nitpicker's People

Stargazers

avatar

Watchers

avatar avatar avatar avatar

nitpicker's Issues

Improve rule Wrong conjugation (stem)

A user of Nitpicker reported an improvement for rule Wrong conjugation (stem).

Context

the men of law kidnapped the kid

Proposed improval

proposed correction is wrong

Nitpicker's data

{
  "tokens": [
    {
      "word": "kidnapped", 
      "uid": 4, 
      "offset": [
        15, 
        24
      ]
    }
  ], 
  "proposals": [
    "kidnaped"
  ], 
  "rephrase": "Replace <bad>kidnapped</bad> by <good>kidnaped</good>.", 
  "example": "The past tense of <q>to kidnap</q> should be <q>kidnaped</q> rather than <q>kidnapped</q>.", 
  "patches": [
    [
      {
        "action": "replace", 
        "token": 4, 
        "value": "kidnaped"
      }
    ]
  ]
}

Improve rule Unmatched pair of on the one hand/on the other hand (hands)

Improvement for rule Unmatched pair of on the one hand/on the other hand (hands)

As user of Nitpicker reported an improvement for rule Unmatched pair of on the one hand/on the other hand (hands).

Context

On the one hand.

Proposed improval

.

Nitpicker's data

{
  "tokens": [
    {
      "word": "On", 
      "uid": 0, 
      "offset": [
        0, 
        2
      ]
    }, 
    {
      "word": "the", 
      "uid": 1, 
      "offset": [
        3, 
        6
      ]
    }, 
    {
      "word": "one", 
      "uid": 2, 
      "offset": [
        7, 
        10
      ]
    }, 
    {
      "word": "hand", 
      "uid": 3, 
      "offset": [
        11, 
        15
      ]
    }
  ], 
  "rephrase": "If two aspects are discussed, add <q>on the other hand</q> to the second item. Otherwise drop <q>on the one hand</q>.", 
  "proposals": [], 
  "example": "The phrase <q>on the one hand</q> is not followed by a <q>on the other hand</q> phrase.", 
  "patches": []
}

Improve rule Passive voice or missing agent (bugs_001)

A user of Nitpicker reported an improvement for rule Passive voice or missing agent (bugs_001).

Context

i was kissed by my boyfriend

Proposed improval

my boyfriend kissed me

Nitpicker's data

{
  "tokens": [
    {
      "word": "i", 
      "uid": 0, 
      "offset": [
        0, 
        1
      ]
    }, 
    {
      "word": "was", 
      "uid": 1, 
      "offset": [
        2, 
        5
      ]
    }, 
    {
      "word": "kissed", 
      "uid": 2, 
      "offset": [
        6, 
        12
      ]
    }, 
    {
      "word": "my", 
      "uid": 4, 
      "offset": [
        16, 
        18
      ]
    }, 
    {
      "word": "boyfriend", 
      "uid": 5, 
      "offset": [
        19, 
        28
      ]
    }
  ], 
  "proposals": [
    "my boyfriend kissed i"
  ], 
  "rephrase": "Make the agent for <q>kissed i</q> explicit.", 
  "example": "It is unclear <em>who or what</em> kissed i.", 
  "patches": [
    [
      {
        "action": "delete", 
        "token": 1
      }, 
      {
        "action": "delete", 
        "token": 3
      }, 
      {
        "action": "move", 
        "source": 4, 
        "target": 0
      }, 
      {
        "action": "move", 
        "source": 5, 
        "target": 0
      }, 
      {
        "action": "move", 
        "source": 2, 
        "target": 0
      }, 
      {
        "action": "replace", 
        "token": 2, 
        "value": "kissed"
      }
    ]
  ]
}

Rule: proof

"It tries to proof the property."

"Proof" is not a verb.

Improve rule Passive voice or missing agent (bugs_001)

A user of Nitpicker reported an improvement for rule Passive voice or missing agent (bugs_001).

Context

The goal was hit by everyone.

Proposed improval

In the proposal, "The" is still capitalized whereas "everyone" is not.

It would be nice if Nitpicker would propose the correct spelling. Here. "Everyone hit the goal".

Nitpicker's data

{
  "tokens": [
    {
      "word": "The", 
      "uid": 0, 
      "offset": [
        0, 
        3
      ]
    }, 
    {
      "word": "goal", 
      "uid": 1, 
      "offset": [
        4, 
        8
      ]
    }, 
    {
      "word": "was", 
      "uid": 2, 
      "offset": [
        9, 
        12
      ]
    }, 
    {
      "word": "hit", 
      "uid": 3, 
      "offset": [
        13, 
        16
      ]
    }, 
    {
      "word": "everyone", 
      "uid": 5, 
      "offset": [
        20, 
        28
      ]
    }
  ], 
  "rephrase": "Make the agent for <q>hit the goal</q> explicit.", 
  "proposals": [
    "everyone hit The goal"
  ], 
  "example": "It is unclear <em>who or what</em> hit the goal.", 
  "patches": [
    [
      {
        "action": "delete", 
        "token": 2
      }, 
      {
        "action": "delete", 
        "token": 4
      }, 
      {
        "action": "move", 
        "source": 5, 
        "target": 0
      }, 
      {
        "action": "move", 
        "source": 3, 
        "target": 0
      }, 
      {
        "action": "replace", 
        "token": 3, 
        "value": "hit"
      }
    ]
  ]
}

Improve rule It's vs. its (itsits)

A user of Nitpicker reported an improvement for rule It's vs. its (itsits).

Context

Its been a pleasure meeting you!

Proposed improval

Nitpicker's data

{
  "tokens": [
    {
      "word": "Its", 
      "uid": 0, 
      "offset": [
        0, 
        3
      ]
    }
  ], 
  "rephrase": "Replace <bad>Its</bad> by <good>It's</good>.", 
  "proposals": [
    "It's"
  ], 
  "example": "<q>Its</q> is used as verb like <q>it is</q> or <q>it has</q>, but actually is a neuter possesor.", 
  "patches": [
    [
      {
        "action": "replace", 
        "token": 0, 
        "value": "It's"
      }
    ]
  ]
}

Test issue

Nitpicker Issue

This is an automatically generated issue.

code

  1. enumeration
  2. enumeration

really nice

Improve rule It's vs. its (itsits)

A user of Nitpicker reported an improvement for rule It's vs. its (itsits).

Context

It is knwon

Proposed improval

The sentence is strange (sorry), but the proposal makes sense ("Its" instead of "It is"). However, in the "rephrase" part, it states "Itis" (without the space).

Nitpicker's data

{
  "tokens": [
    {
      "word": "It", 
      "uid": 0, 
      "offset": [
        0, 
        2
      ]
    }, 
    {
      "word": "is", 
      "uid": 1, 
      "offset": [
        3, 
        5
      ]
    }
  ], 
  "proposals": [
    "Its"
  ], 
  "rephrase": "Replace <bad>Itis</bad> by <good>Its</good>.", 
  "example": "<q>It's</q> is used like the neuter possesor <q>its</q>, but actually is an abbreviation for <q>it is</q> or <q>it has</q>.", 
  "patches": [
    [
      {
        "action": "replace", 
        "token": 0, 
        "value": "Its"
      }, 
      {
        "action": "delete", 
        "token": 1
      }
    ]
  ]
}

Improve rules

A user of Nitpicker reported an improvement.

Context

foo

Proposed improval

bar

Test issue

Nitpicker Issue

This is an automatically generated issue.

code

  1. enumeration
  2. enumeration

really nice

Improve rule Wrong conjugation (stem)

Improvement for rule Wrong conjugation (stem)

As user of Nitpicker reported an improvement for rule Wrong conjugation (stem) for the following sentence.

The ball was dropped.

The following proposal was made:

Nitpicker proposes "droped" as correction, which is apparently wrong.

Nitpicker's data

{
  "tokens": [
    {
      "word": "dropped", 
      "uid": 3, 
      "offset": [
        13, 
        20
      ]
    }
  ], 
  "rephrase": "Replace <bad>dropped</bad> by <good>droped</good>.", 
  "proposals": [
    "droped"
  ], 
  "example": "The past participle of <q>to drop</q> should be <q>droped</q> rather than <q>dropped</q>.", 
  "patches": [
    [
      {
        "action": "replace", 
        "token": 3, 
        "value": "droped"
      }
    ]
  ]
}

Improve rule Wrong indefinite article (aan)

A user of Nitpicker reported an improvement for rule Wrong indefinite article (aan).

Context

This is an NSA-controlled computer.

Proposed improval

This is a false positive: it should be "an NSA-controlled" because of the pronunciation.

Nitpicker's data

{
  "tokens": [
    {
      "word": "an", 
      "uid": 2, 
      "offset": [
        8, 
        10
      ]
    }, 
    {
      "word": "NSA-controlled", 
      "uid": 3, 
      "offset": [
        11, 
        25
      ]
    }
  ], 
  "proposals": [
    "a NSA-controlled"
  ], 
  "rephrase": "Replace <bad>an</bad> with <good>a</good>.", 
  "example": "The indefinite article <q>an</q> is followed by <q>NSA-controlled</q>.", 
  "patches": [
    [
      {
        "action": "replace", 
        "token": 2, 
        "value": "a"
      }
    ]
  ]
}

Improve rule Proven vs. proved (bugs_013)

A user of Nitpicker reported an improvement for rule Proven vs. proved (bugs_013).

Context

We used a proved technique.

Proposed improval

Nothing - the rule is just fine :)

Nitpicker's data

{"tokens": [{"word": "proved", "uid": 3, "offset": [10, 16]}], "rephrase": "Replace <bad>proved</bad> with the adjective <good>proven</good>.", "proposals": ["proven"], "example": "The verb <q>proved</q> is used as an adjective.", "patches": [[{"action": "replace", "token": 3, "value": "proven"}]]}

Improve rule Passive voice or missing agent (bugs_001)

A user of Nitpicker reported an improvement for rule Passive voice or missing agent (bugs_001).

Context

Unlike in a centralized monolithic system, services may change and are not completely controlled by a single party.

Proposed improval

The proposal "a single party control services" does not make sense in the context of the whole sentence.

Nitpicker's data

{
  "tokens": [
    {
      "word": "services", 
      "uid": 7, 
      "offset": [
        43, 
        51
      ]
    }, 
    {
      "word": "are", 
      "uid": 11, 
      "offset": [
        67, 
        70
      ]
    }, 
    {
      "word": "controlled", 
      "uid": 14, 
      "offset": [
        86, 
        96
      ]
    }, 
    {
      "word": "a", 
      "uid": 16, 
      "offset": [
        100, 
        101
      ]
    }, 
    {
      "word": "single", 
      "uid": 17, 
      "offset": [
        102, 
        108
      ]
    }, 
    {
      "word": "party", 
      "uid": 18, 
      "offset": [
        109, 
        114
      ]
    }
  ], 
  "rephrase": "Make the agent for <q>control services</q> explicit.", 
  "proposals": [
    "a single party control services"
  ], 
  "example": "It is unclear <em>who or what</em> control services.", 
  "patches": [
    [
      {
        "action": "delete", 
        "token": 8
      }, 
      {
        "action": "delete", 
        "token": 9
      }, 
      {
        "action": "delete", 
        "token": 10
      }, 
      {
        "action": "delete", 
        "token": 11
      }, 
      {
        "action": "delete", 
        "token": 12
      }, 
      {
        "action": "delete", 
        "token": 13
      }, 
      {
        "action": "delete", 
        "token": 15
      }, 
      {
        "action": "move", 
        "source": 16, 
        "target": 7
      }, 
      {
        "action": "move", 
        "source": 17, 
        "target": 7
      }, 
      {
        "action": "move", 
        "source": 18, 
        "target": 7
      }, 
      {
        "action": "move", 
        "source": 14, 
        "target": 7
      }, 
      {
        "action": "replace", 
        "token": 14, 
        "value": "control"
      }
    ]
  ]
}

Test issue

Nitpicker Issue

This is an automatically generated issue.

code

  1. enumeration
  2. enumeration

really nice

Test issue

Nitpicker Issue

This is an automatically generated issue.

code

  1. enumeration
  2. enumeration

really nice

Test issue

Nitpicker Issue

This is an automatically generated issue.

code

  1. enumeration
  2. enumeration

really nice

Improve rule It's vs. its (itsits)

Improvement for rule It's vs. its (itsits)

Its been a pleasure.

Proposed correction:

Nitpicker's data

{"example": "\u003cq\u003eIts\u003c/q\u003e is used as verb like \u003cq\u003eit is\u003c/q\u003e or \u003cq\u003eit has\u003c/q\u003e, but actually is a neuter possesor.", "patches": [[{"action": "replace", "token": 0, "value": "It\u0027s"}]], "proposals": ["It\u0027s"], "rephrase": "Replace \u003cbad\u003eIts\u003c/bad\u003e by \u003cgood\u003eIt\u0027s\u003c/good\u003e.", "tokens": [{"offset": [0, 3], "uid": 0, "word": "Its"}]}

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.