Skip to content

Commit 54956ef

Browse files
committed
fixup
1 parent 83789ee commit 54956ef

File tree

2 files changed

+9
-4
lines changed

2 files changed

+9
-4
lines changed

src/ccompass/core.py

-2
Original file line numberDiff line numberDiff line change
@@ -826,7 +826,6 @@ def ndarray_constructor(loader, node):
826826
def series_constructor(loader, node):
827827
"""Custom YAML constructor for pandas Series."""
828828
file_path = temp_dir / loader.construct_scalar(node)
829-
print(file_path, type(file_path))
830829
df = pd.read_csv(
831830
file_path,
832831
sep="\t",
@@ -835,7 +834,6 @@ def series_constructor(loader, node):
835834
float_precision="round_trip",
836835
)
837836
assert df.shape[1] == 1
838-
print(df.iloc[:, 0], type(df.iloc[:, 0]))
839837
return df.iloc[:, 0]
840838

841839
def tuple_constructor(loader, node):

tests/test_session_model.py

+9-2
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ def test_serialization():
2222
assert_session_equal(session, session2)
2323

2424

25-
def assert_equal(obj1, obj2):
25+
def assert_equal(obj1, obj2) -> None:
2626
"""Check if two objects are equal."""
2727
if isinstance(obj1, pydantic.BaseModel):
2828
assert isinstance(obj2, pydantic.BaseModel)
@@ -37,9 +37,16 @@ def assert_equal(obj1, obj2):
3737
for i in range(len(obj1)):
3838
assert_equal(obj1[i], obj2[i])
3939
elif isinstance(obj1, pd.DataFrame):
40+
assert isinstance(obj2, pd.DataFrame)
41+
if obj1.empty and obj2.empty:
42+
# if both are empty, we don't compare columns
43+
return
4044
pd.testing.assert_frame_equal(obj1, obj2, check_dtype=False)
4145
elif isinstance(obj1, pd.Series):
42-
pd.testing.assert_series_equal(obj1, obj2)
46+
assert isinstance(obj2, pd.Series), f"{obj1} != {obj2}"
47+
pd.testing.assert_series_equal(
48+
obj1, obj2, atol=1e-14, rtol=1e-14, check_dtype=False
49+
)
4350
elif isinstance(obj1, np.ndarray):
4451
np.testing.assert_almost_equal(obj1, obj2)
4552
elif isinstance(obj1, float) and pd.isna(obj1):

0 commit comments

Comments
 (0)