-
Notifications
You must be signed in to change notification settings - Fork 364
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replace infs with nans to avoid crash when creating a heatmap #442
Conversation
Codecov Report
@@ Coverage Diff @@
## master #442 +/- ##
==========================================
+ Coverage 65.55% 65.62% +0.06%
==========================================
Files 55 55
Lines 4419 4422 +3
==========================================
+ Hits 2897 2902 +5
+ Misses 1522 1520 -2
Continue to review full report at Codecov.
|
Signed-off-by: Cristian Garcia <cgarcia.e88@gmail.com>
7df184e
to
ac88f82
Compare
lux/executor/PandasExecutor.py
Outdated
@@ -11,6 +11,7 @@ | |||
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | |||
# See the License for the specific language governing permissions and | |||
# limitations under the License. | |||
# fmt: off |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is this line for?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will remove it, vscode was formatting this file that is supposed to be ignored.
Thanks @cgarciae! I left a few minor suggestions, otherwise, this looks good to merge soon! |
Co-authored-by: Doris Lee <dorisjunglinlee@gmail.com>
Signed-off-by: Cristian Garcia cgarcia.e88@gmail.com
Overview
Fixes #440 by replacing
inf
s withnan
s.Changes
Uses
inside
PandasExecutor.py
on several places to removeinf
s.