Skip to content

Commit

Permalink
Merge pull request #6681 from markotoplak/display-nans
Browse files Browse the repository at this point in the history
Display NaNs in StringVariable as "?"
  • Loading branch information
janezd committed Jan 5, 2024
2 parents 7b68883 + 02ad5e6 commit 6fec5d6
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 0 deletions.
2 changes: 2 additions & 0 deletions Orange/data/tests/test_variable.py
Original file line number Diff line number Diff line change
Expand Up @@ -634,6 +634,8 @@ def test_val(self):
self.assertEqual(a.str_val(""), "?")
self.assertEqual(a.str_val(Value(a, "")), "?")
self.assertEqual(a.repr_val(Value(a, "foo")), '"foo"')
self.assertEqual(a.str_val(np.nan), "?")
self.assertEqual(a.str_val(None), "?")


@variabletest(TimeVariable)
Expand Down
3 changes: 3 additions & 0 deletions Orange/data/variable.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
from pickle import PickleError

import numpy as np
import pandas
import scipy.sparse as sp

from Orange.data import _variable
Expand Down Expand Up @@ -905,6 +906,8 @@ def str_val(val):
if not val.value:
return "?"
val = val.value
if pandas.isnull(val):
return "?"
return str(val)

def repr_val(self, val):
Expand Down

0 comments on commit 6fec5d6

Please sign in to comment.