Skip to content

Support exclude_samples_col/exclude_samples_values in contrast validation#87

Open
pinin4fjords wants to merge 1 commit intodevelopfrom
feature/exclude-samples-validation
Open

Support exclude_samples_col/exclude_samples_values in contrast validation#87
pinin4fjords wants to merge 1 commit intodevelopfrom
feature/exclude-samples-validation

Conversation

@pinin4fjords
Copy link
Owner

Summary

  • Filter samples per-contrast before building the design matrix and checking full rank, aligning with differentialabundance pipeline behaviour where exclude_samples_col and exclude_samples_values can be used to exclude samples for specific contrasts
  • Add validate_design parameter to read_contrasts() to disable design matrix checks when not needed

Fixes #86

Test plan

  • Test with contrasts file containing exclude_samples_col and exclude_samples_values
  • Verify full rank validation passes when excluded samples would cause confounding
  • Test validate_design = FALSE skips design matrix checks
  • Verify backward compatibility when exclude columns are not present

🤖 Generated with Claude Code

…tion

- Filter samples per-contrast before building the design matrix and checking
  full rank, aligning with differentialabundance pipeline behaviour
- Add validate_design parameter to disable design matrix checks when not needed

Fixes #86

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@pinin4fjords pinin4fjords force-pushed the feature/exclude-samples-validation branch from 2a3d9fc to c921f15 Compare December 8, 2025 13:37
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.

Contrast full-rank validation vs exclude_samples_col

1 participant