Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

LGTM has 6 errors

See original GitHub issue

Describe the bug LGTM shows 6 errors, at least one seems easy to fix there is a self alignment that seems redundant

To Reproduce https://lgtm.com/projects/g/pvlib/pvlib-python/alerts/?mode=list&severity=error

Expected behavior No errors?

Screenshots https://lgtm.com/projects/g/pvlib/pvlib-python/alerts/?mode=list&severity=error

Versions:

pvlib.__version__: 0.9ish
pandas.__version__: ?
python: 3

Additional context https://lgtm.com/projects/g/pvlib/pvlib-python/alerts/?mode=list&severity=error

Issue Analytics

State:
Created 2 years ago
Comments:5 (5 by maintainers)

Top GitHub Comments

1reaction

chrisornercommented, Sep 27, 2022

I’ve clicked the Hide button on a handful of similar alerts in the past. I didn’t know about the comment option until now.

I tried looking into it a few years ago and didn’t figure it out. Maybe someone smarter than me can figure it out.

This is a bug in LGTM. It raises this error because of the user function test in test_modelchain.py:

Where constant_aoi_loss is a Python partial function which is the known bug in LGTM. So this error can be ignored. I am going to take a look at the others as well.

1reaction

wholmgrencommented, Aug 9, 2021

I’ve clicked the Hide button on a handful of similar alerts in the past. I didn’t know about the comment option until now.

I tried looking into it a few years ago and didn’t figure it out. Maybe someone smarter than me can figure it out.

Top Results From Across the Web

Fixing NO_SUITABLE_WORKER errors - LGTM

When jobs fail due to labeling problems they are listed on the Logs page with the status "Returned From Queue (NO_SUITABLE_WORKER)": A Labeling...

Fake Code Review with 'LGTM' - Medium

Some fake Agile software companies focus on 'sound good' but poorly conducted practices, Code Review is one of them. The objective of Code ......

Google Code Archive - Google Code

Trailer Go's had pretty dece. Rob Pike, Apr 10, 2014, 6b826a65a629, doc/go1.3.html: fix spelling mistakes Keep those builders busy. LGTM=bradfitz R.

397984 - Presubmit check misses LGTM from an OWNER ...

But still, the presubmit check failed with: ** Presubmit ERRORS ** Missing LGTM from an OWNER for these files: chrome/common/content_settings_helper.cc

#2381 - Fix tests code to prevent some runtime errors - gitea - Gitea ...

gitea - Git with a cup of tea, painless self-hosted git service.