holistic

[untzag]

function derived from TOPAS preamp tuning

[ksunden]

This pull request introduces 3 alerts when merging 8c5e463 into 56a85f9 - view on LGTM.com

new alerts:

• 1 for Constant in conditional expression or statement
• 1 for Unused import
• 1 for Unreachable code

---

Comment posted by LGTM.com

[ksunden]

This pull request introduces 5 alerts when merging 40148e8 into 56a85f9 - view on LGTM.com

new alerts:

• 2 for Unused local variable
• 1 for Mismatch in multiple assignment
• 1 for Constant in conditional expression or statement
• 1 for Unreachable code

---

Comment posted by LGTM.com

[ksunden]

Data file updated using wright-group/WrightTools#909. Simply called from_PyCMDS and save, no stripping of variables (only one channel to begin with)

[lgtm-com]

This pull request introduces 5 alerts when merging d69909d into 56a85f9 - view on LGTM.com

new alerts:

• 2 for Unused local variable
• 1 for Mismatch in multiple assignment
• 1 for Constant in conditional expression or statement
• 1 for Unreachable code

[lgtm-com]

This pull request introduces 4 alerts when merging 95e2422 into bbc03ef - view on LGTM.com

new alerts:

• 2 for Unused local variable
• 1 for Constant in conditional expression or statement
• 1 for Unreachable code

[ghost]

DeepCode Report (#81716c)

DeepCode analyzed this pull request.
There are 4 new info reports.

[lgtm-com]

This pull request introduces 5 alerts when merging 23164af into 3e28412 - view on LGTM.com

new alerts:

• 3 for Unused local variable
• 1 for Constant in conditional expression or statement
• 1 for Unreachable code

[lgtm-com]

This pull request introduces 1 alert when merging 24516be into 3e28412 - view on LGTM.com

new alerts:

• 1 for Unused local variable

[ksunden]

I did some blackening in here... Probably should have held off, luckily only one other PR is open

[lgtm-com]

This pull request introduces 4 alerts when merging a7b8ca3 into 3e28412 - view on LGTM.com

new alerts:

• 4 for Unused import

[lgtm-com]

This pull request introduces 4 alerts when merging 28613e9 into 3e28412 - view on LGTM.com

new alerts:

• 4 for Unused import

[ksunden]

Tests are failing due to wright-group/WrightTools#923 not being in the released version, tests pass on current master WT (at least locally)

[ksunden]

@darienmorrow Have you had a chance to see if this is usable/makes sense?

[darienmorrow]

@ksunden I have briefly perused the PR and used holistic on my OSX machine.

I think we should merge this PR.