github / github/codeql

LGTM.com - false positive - float is numbers.Number

未關閉
#6,519 2 則留言 0 個 reaction 已指派 0 人 在 GitHub 檢視
acknowledged false-positive not security Python
主要語言
CodeQL
星號
10.1k
分支
2.1k
平均合併
2 天 15 小時
30 天內合併 PR
141

描述

**Description of the false positive**

I may be missing something here, but the message doesn't make much sense:
> `[ 3 Values ]` of class '`[ 3 Values ]`' may be used in for-loop.`

What are these "`[ 3 Values ]`" about?

Also, note that a `float` **is** a `numbers.Number`:
```python
>>> import numbers
>>> x = 0.5
>>> isinstance(x, numbers.Number)
True
>>>
```
So, since `l1_ratios` is usually initialised to a float as far as I can see, it will be changed into an iterable list, won't it?
```python
if isinstance(l1_ratios, numbers.Number):
l1_ratios = [l1_ratios]
```

**URL to the alert on the project page on LGTM.com**

https://lgtm.com/projects/g/nilearn/nilearn/snapshot/0cd925e33787e277990b524efeba0c8125d04816/files/nilearn/decoding/space_net.py#x4b41318e0293a848:1

貢獻指南

開啟貢獻指南

評估

這個 Issue 還沒有評估資料。

把新 issue 寄到你的電子郵件信箱

精選適合新手參與的 GitHub issue 摘要。