Skip to content

fixed pandas future warn about as_matrix#37

Merged
dwhswenson merged 1 commit intodwhswenson:masterfrom
sroet:fix_pandas_future_warn
May 30, 2018
Merged

fixed pandas future warn about as_matrix#37
dwhswenson merged 1 commit intodwhswenson:masterfrom
sroet:fix_pandas_future_warn

Conversation

@sroet
Copy link
Collaborator

@sroet sroet commented May 21, 2018

Since pandas version 0.23 a FutureWarning popped up:

contact_map/tests/test_contact_count.py::TestContactCount::()::test_df
  /home/sander/github_files/contact_map/contact_map/tests/test_contact_count.py:74: FutureWarning: Method .as_matrix will be removed in a future version. Use .values instead.
    assert_array_equal(atom_df.to_dense().as_matrix(),
  /home/sander/github_files/contact_map/contact_map/tests/test_contact_count.py:76: FutureWarning: Method .as_matrix will be removed in a future version. Use .values instead.
    assert_array_equal(residue_df.to_dense().as_matrix(),

This fixes that warning in the way proposed by the warning

Copy link
Owner

@dwhswenson dwhswenson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Merging.

@dwhswenson dwhswenson merged commit 277289a into dwhswenson:master May 30, 2018
@dwhswenson dwhswenson added misc PR upstream_changed PR in response to changes in dependencies labels May 31, 2018
@dwhswenson dwhswenson mentioned this pull request Jun 3, 2019
@sroet sroet deleted the fix_pandas_future_warn branch July 7, 2020 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

misc PR upstream_changed PR in response to changes in dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants