Skip to content

Commit

Permalink
Fix/windows tests (#44)
Browse files Browse the repository at this point in the history
* Remove nnls patch and fix scipy version (with fixed nnls)

* ✨ remove patched nnls (using fixed scipy version now)

* 💄 linting; line breaks

* 🚑 fix dtypes to ensure windows ci to pass

---------

Co-authored-by: Elmar Zander <[email protected]>
  • Loading branch information
gessulat and ezander authored Jul 29, 2024
1 parent 653179f commit 68fd52b
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tests/unit_tests/test_tabular_data.py
Original file line number Diff line number Diff line change
Expand Up @@ -186,9 +186,9 @@ def test_dataframe_reader(psm_df_6):
reader.read(), pd.DataFrame({"test": [1, 2, 3]})
)

reader = DataFrameReader.from_array(np.array([1, 2, 3]), name="test")
reader = DataFrameReader.from_array(np.array([1, 2, 3], dtype=np.int32), name="test")

Check failure on line 189 in tests/unit_tests/test_tabular_data.py

View workflow job for this annotation

GitHub Actions / lint

Ruff (E501)

tests/unit_tests/test_tabular_data.py:189:80: E501 Line too long (89 > 79)
pd.testing.assert_frame_equal(
reader.read(), pd.DataFrame({"test": [1, 2, 3]})
reader.read(), pd.DataFrame({"test": [1, 2, 3]}, dtype=np.int32)
)


Expand Down

0 comments on commit 68fd52b

Please sign in to comment.