Ticket #10137 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Fix pylint whitespace warnings

Reported by: Peter Peterson Owned by: Peter Peterson
Priority: minor Milestone: Release 3.3
Component: Python Keywords:
Cc: martyn.gigg@… Blocked By:
Blocking: Tester:

Description

To make pylint a bit easier to see. Fix whitespace issues. I created a gist to have for later.

Change History

comment:1 Changed 6 years ago by Pete Peterson

Re #10137. Fixing python whitespace issues.

Changeset: ad5b53d0bfc2ee81ed35444ce9485f5fe3a2d596

comment:2 Changed 6 years ago by Nick Draper

  • Status changed from new to assigned

comment:3 Changed 6 years ago by Pete Peterson

  • Status changed from assigned to inprogress

Re #10137. Fixing python whitespace in Test tree

Changeset: 18121e8f812e5316c9720a5b8aef889106076dfc

comment:4 Changed 6 years ago by Pete Peterson

Re #10137. Fixing python whitespace in Code/Tools tree

Changeset: 973b5d1a8182f542c0d931a79879bf4163814626

comment:5 Changed 6 years ago by Pete Peterson

Re #10137. Fixing python whitespace in Code/Mantid/docs tree

Changeset: 676b329f42745c64962cc33ffdb61a5cd0d48802

comment:6 Changed 6 years ago by Pete Peterson

Re #10137. Fixing python whitespace in Code/Mantid/Build tree

Changeset: 942bdae991e71a3e200324d79e219aedb7889ffb

comment:7 Changed 6 years ago by Pete Peterson

Re #10137. Fixing python whitespace in Code/Mantid/Vates tree

Changeset: 094236d40c6edcfbbc220b969a0ff15f99965069

comment:8 Changed 6 years ago by Pete Peterson

Re #10137. Fixing python whitespace in Code/Mantid/scripts tree

Changeset: b777b08750db72f560ad84a5a0933f02d324ffc8

comment:9 Changed 6 years ago by Pete Peterson

Merge branch 'feature/10137_pylint_whitespace' into develop

Re #10137

Conflicts:

Code/Mantid/scripts/Inelastic/IndirectAbsCor.py Code/Mantid/scripts/Inelastic/IndirectCommon.py Code/Mantid/scripts/Inelastic/IndirectDataAnalysis.py Code/Mantid/scripts/Inelastic/IndirectEnergyConversion.py Code/Mantid/scripts/Inelastic/IndirectNeutron.py Code/Mantid/scripts/Inelastic/IndirectSymm.py Code/Mantid/scripts/Inelastic/inelastic_indirect_reduction_steps.py Code/Mantid/scripts/Inelastic/msg_reducer.py Code/Mantid/scripts/Interface/ui/reflectometer/latest_isis_runs.py Code/Mantid/scripts/Interface/ui/reflectometer/refl_gui.py Code/Mantid/scripts/Interface/ui/reflectometer/refl_options.py Code/Mantid/scripts/SANS/SANSUtility.py Code/Mantid/scripts/SANS/isis_reduction_steps.py Code/Mantid/scripts/test/SANSUtilitytests.py

Changeset: a64f2fa4559e62bf78cbeecedf23f3015d1ac6c4

comment:10 Changed 6 years ago by Pete Peterson

Re #10137. Fixing python whitespace in Code/Mantid/MantidQt tree

Changeset: a04df2a2317e068920507601955a737fc12c857b

comment:11 Changed 6 years ago by Pete Peterson

Re #10137. Fixing python whitespace in Code/Mantid/MantidPlot tree

Changeset: c290f1ea63902827c06dc10d33c9f143fc12db64

comment:12 Changed 6 years ago by Pete Peterson

Re #10137. Fixing python whitespace

Changeset: 853c06c90f398cce594ff108c1dee9b54a30a703

comment:13 Changed 6 years ago by Pete Peterson

Re #10137. Fixing python whitespace in Code/Mantid/Framework tree

Changeset: 16b61e4925f27a4daed93123c3ec64cf5b484691

comment:14 Changed 6 years ago by Pete Peterson

Merge branch 'feature/10137_pylint_whitespace' into develop

Re #10137

Conflicts:

Code/Mantid/Framework/PythonInterface/plugins/algorithms/WorkflowAlgorithms/SofQWMoments.py Code/Mantid/Framework/PythonInterface/plugins/algorithms/WorkflowAlgorithms/Symmetrise.py

Changeset: 8635fa5e422fea18f95fd481fe1aeb781e4c553f

comment:15 Changed 6 years ago by Pete Peterson

Re #10137. Reverting to version from master to fix unit test.

Changeset: 3f45e5e8756c3dbe65fbeaa213c04a5d734bcd13

comment:16 Changed 6 years ago by Pete Peterson

Re #10137. Fixing indentation error.

Changeset: 5d347fd6a3c5e1d17621483b5d8ec8251138a3d2

comment:17 Changed 6 years ago by Peter Peterson

  • Blocking 10144 added

comment:18 Changed 6 years ago by Pete Peterson

Re #10137. Fixing more unit tests.

Changeset: a5febc072663a839a6a2065a3e95ec34feb810d6

comment:19 Changed 6 years ago by Pete Peterson

Merge branch 'feature/10137_pylint_whitespace' into develop

Re #10137.

Conflicts:

Code/Mantid/scripts/SANS/SANSUtility.py

Changeset: f359bb99497303759d5e2b47ed5cb5db3829e414

comment:20 Changed 6 years ago by Peter Peterson

  • Blocking 10145 added

comment:21 Changed 6 years ago by Pete Peterson

Re #10137. Fixing unit tests.

Changeset: b360bbf80c1dedda105161a2e77b5992db313850

comment:22 Changed 6 years ago by Pete Peterson

Re #10137. Fixing unit test.

Changeset: cffeb5b0b7ba04390e56bb6e92553311feacb042

comment:23 Changed 6 years ago by Pete Peterson

Re #10137. Going back to version from master to fix unit test.

Created #10145 to address it.

Changeset: 06f5ca137211aeab2b9b7be901423894d619e011

comment:24 Changed 6 years ago by Peter Peterson

  • Blocking 10146 added

comment:25 Changed 6 years ago by Pete Peterson

Re #10137. Fixing unit test.

Changeset: f10391f2f811fca92dfd28b0395183a7e550c912

comment:26 Changed 6 years ago by Pete Peterson

Re #10137. Going back to version from master to fix unit test.

Created #10146 to address it.

Changeset: 4dbee828ce03853ab06e902843573b775d301223

comment:27 Changed 6 years ago by Pete Peterson

Re #10137. Going back to version from master to fix unit test.

Changeset: 5ec575639074e45c095473c04548c3596e0ef0fc

comment:28 Changed 6 years ago by Pete Peterson

Re #10137. Fixing unit test.

Changeset: 8992b5f2175cb2f79fad0b5928c3cc3b4a4de833

comment:29 Changed 6 years ago by Peter Peterson

All unit tests should be passing.

Need to fix whitespace for (reverted to master):

  • SimpleAPITest.py
  • DakotaChiSquared.py

Import errors involving:

  • SelectPowderDiffPeaks.py
  • EnginXCalibrateFull.py
  • ExaminePowderDiffProfile.py
  • BASISReduction.py
  • QLines.py
  • SavePlot1D.py
  • Symmetrise.py

comment:30 Changed 6 years ago by Pete Peterson

Re #10137. Fixing broken unit tests.

Changeset: 412aeb8fa350ae00cb4f7275e2903f317826c438

comment:31 Changed 6 years ago by Pete Peterson

Re #10137. Fixing indentation errors.

Changeset: d456e2dffc0b154caa714950ccc97b7132d718ba

comment:32 Changed 6 years ago by Pete Peterson

Re #10137. Fixing indentation errors.

Changeset: 7c7f3ce45924e0d857673f6bab6ac757b997548f

comment:33 Changed 6 years ago by Pete Peterson

Re #10137. Fixing indentation errors.

Changeset: f33665feae3dabd5c9200b0878f533d69a5b26a0

comment:34 Changed 6 years ago by Pete Peterson

Re #10137. Fixing indentation errors.

Changeset: c46103465733903f10c662c768c23ad5b51b52ce

comment:35 Changed 6 years ago by Pete Peterson

Merge branch 'feature/10137_pylint_whitespace' into develop

Re #10137.

Conflicts:

Code/Mantid/Framework/PythonInterface/plugins/algorithms/WorkflowAlgorithms/Symmetrise.py

Changeset: 4898a4c63318a3e239da69b68ad291a489f6a152

comment:36 Changed 6 years ago by Peter Peterson

  • Blocking 10144 removed

comment:37 Changed 6 years ago by Peter Peterson

  • Blocking 10145 removed

comment:38 Changed 6 years ago by Pete Peterson

Re #10137. Fixing indentation errors.

Changeset: 251527820f480fe6daee109f6c69442629bb47c9

comment:39 Changed 6 years ago by Pete Peterson

Re #10137. Fixing indentation errors.

Changeset: 0c5edd1b69b0b6d5b3293ec00aab753312f24464

comment:40 Changed 6 years ago by Pete Peterson

Merge branch 'feature/10137_pylint_whitespace' into develop

Re #10137.

Conflicts:

Code/Mantid/scripts/SANS/SANSUtility.py

Changeset: a64b0a0c6ccb663cd943f2f5cd20351bba3b32c9

comment:41 Changed 6 years ago by Peter Peterson

  • Blocking 10146 removed

comment:42 Changed 6 years ago by Pete Peterson

Re #10137. Fixing indentation issues.

Changeset: a0900ba525079e9cb1be17e4e85f04bc05f3bf81

comment:43 Changed 6 years ago by Pete Peterson

Re #10137. Fixing unit test error.

Changeset: 10cf773930a43b3bcbc9461747fba39649fb58e9

comment:44 Changed 6 years ago by Peter Peterson

  • Status changed from inprogress to verify
  • Resolution set to fixed

comment:45 Changed 6 years ago by Stuart Campbell

Merge remote branch 'origin/feature/10137_pylint_whitespace'

Conflicts:

Code/Mantid/scripts/Interface/ui/reflectometer/latest_isis_runs.py Code/Mantid/scripts/Interface/ui/reflectometer/refl_gui.py Code/Mantid/scripts/Interface/ui/reflectometer/refl_options.pyi

Refs #10137

Changeset: f0be33d3900de4b06506a22f959f0827a4248296

comment:46 Changed 6 years ago by Stuart Campbell

  • Status changed from verify to closed

Merge remote branch 'origin/feature/10137_pylint_whitespace'

Conflicts:

Code/Mantid/scripts/Interface/ui/reflectometer/latest_isis_runs.py Code/Mantid/scripts/Interface/ui/reflectometer/refl_gui.py Code/Mantid/scripts/Interface/ui/reflectometer/refl_options.pyi

Refs #10137

Full changeset: f0be33d3900de4b06506a22f959f0827a4248296

comment:47 Changed 6 years ago by Stuart Campbell

Merge remote branch 'origin/feature/10137_pylint_whitespace'

Full changeset: 2a71ad58767720e65e0bb67093b7d3e6c3d72db4

comment:48 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 10979

Note: See TracTickets for help on using tickets.