Skip to content

Conversation

sueoglu
Copy link
Collaborator

@sueoglu sueoglu commented Oct 15, 2025

fixes #961

  • Edited the test_missing_values_barplots so that it passes in local hatch env configured same as our CI.
  • Created the baseline image again

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@sueoglu sueoglu changed the title tried to fix the missing_values_barplot errors, this version passes i… fix the missing_values_barplot errors Oct 15, 2025
@github-actions github-actions bot added the bug Something isn't working label Oct 15, 2025
@pytest.mark.parametrize("layer", [None, "layer_2"])
def test_missing_values_barplot(mimic_2, check_same_image, layer):
def test_missing_values_barplot(mimic_2, check_same_image, layer, clean_up_plots):
plt.close("all")
Copy link
Member

Choose a reason for hiding this comment

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

We shouldn't need this here anymore, right?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

let me try without it if it still passes I'll remove it


def test_pca_overview(mimic_2_sample_serv_unit_day_icu, check_same_image):
def test_pca_overview(mimic_2_sample_serv_unit_day_icu, check_same_image, clean_up_plots):
plt.close("all")
Copy link
Member

Choose a reason for hiding this comment

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

Do we still need this?

@sueoglu sueoglu merged commit 31acf53 into main Oct 16, 2025
13 checks passed
@sueoglu sueoglu deleted the bug/issue-961 branch October 16, 2025 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

tests/plot/test_missingno_pl_api.py::test_missing_values_barplot failures

2 participants