Skip to content

Minor changes to populateKinship.r - #1212

Open
bbimber wants to merge 1 commit into
26.7_fb_kinshipOptmizationfrom
26.7_fb_kinshipOptmization2
Open

bbimber wants to merge 1 commit into
26.7_fb_kinshipOptmizationfrom
26.7_fb_kinshipOptmization2

Conversation

@bbimber

@bbimber bbimber commented Sep 18, 2026

Copy link
Copy Markdown
Collaborator

@labkey-martyp: this adds a few minor changes to kinship. it:

  • MIN_COEFFICIENT was a non-used constant. This promotes it to a script argument. It's still not used by the pipeline. This would make it easier in the future for a java-only PR to pass this if there is a reason to support it.
  • This write a TSV with the FamilyIds. This is mostly a QC step if we need to debug anything.

@github-actions

Copy link
Copy Markdown

ERROR: A pull request from 26.7_fb_kinshipOptmization2 is expected to target release26.7-SNAPSHOT, not 26.7_fb_kinshipOptmization

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