Skip to content

Conversation

DriesSchaumont
Copy link
Member

Changelog

Backport PR #631 to 0.12.x

Issue ticket number and link

Closes #xxxx (Replace xxxx with the GitHub issue number)

Checklist before requesting a review

  • I have performed a self-review of my code

  • Conforms to the Contributor's guide

  • Check the correct box. Does this PR contain:

    • Breaking changes
    • New functionality
    • Major changes
    • Minor changes
    • Documentation
    • Bug fixes
  • Proposed changes are described in the CHANGELOG.md

  • CI tests succeed!

DriesSchaumont and others added 3 commits January 25, 2024 11:11
…631)

* Concat: fix using 'move' with a missing conflicting metadata column.

This commit fixes `TypeError`s when trying to concatenate three or more
samples using `--mode move`. The error occurs when one of the samples
does not contain the column that does trigger a conflict in the other
samples. In mode `move`, the conflicting metadata columns are gathered
across the samples and moved to the multidimensional annotation data slot
(obsm or varm). The problem is that the sample names are used as the
column namnes for the dataframe, but because one or more columns is
missing, the number of input samples is differs from the number of
columns.

* Update CHANGELOG with PR number

* Fix test looking at wrong attribute

* Revert changing attribute in tests
@DriesSchaumont DriesSchaumont marked this pull request as ready for review January 25, 2024 10:32
This reverts commit 827d483.
@DriesSchaumont DriesSchaumont merged commit 3e5cb1b into 0.12.x Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant