Skip to content

Don't copy arrays that don't change in normalise #102

Description

@tomwhite

The variant arrays are not used in append (except for checking) so there is no need to copy them in the normalise step:

# copy direct from vcz1
vcz1_copy_vars = [
var
for var in root1.keys()
if not var.startswith("call_") and var != "sample_id"
]
copy_store(vcz1, vcz2_norm, array_keys=vcz1_copy_vars)

The reason for copying them was to ensure that normalise produces a self-contained, internally-consistent vcz. But that is not a requirement, particularly if we also do #90.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions