Skip to content

polyrad_bugfix#147

Merged
alex-sandercock merged 3 commits intodevelopmentfrom
polyrad_bugfix
Sep 12, 2025
Merged

polyrad_bugfix#147
alex-sandercock merged 3 commits intodevelopmentfrom
polyrad_bugfix

Conversation

@Cristianetaniguti
Copy link
Collaborator

@Cristianetaniguti Cristianetaniguti commented Sep 5, 2025

  • Bugfix polyRAD dosage call

    • DosageCall_functions.R:
      • fix names match between Geno and hindhe.obj
      • genotypes from polyrad now named GTP in VCF format - to avoid redundancy with GT if VCF already has genotype called
    • mod_DosageCall.R
      • polyrad_items$vcf_path was replaced by polyRAD_dosage_call result
    • Add test
      • example file iris_DArT_VCF.vcf.gz didn’t have chromosome IDs in the header
  • Bugfix on vcf_sanity_check

    • Avoid first five columns (Marker ID included) to check for | (phased genotypes)

Copy link
Collaborator

Choose a reason for hiding this comment

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

Will need to revisit this at a later point to overwrite the existing GT info with the polyRAD info, but this works for now.

@alex-sandercock alex-sandercock merged commit ea5b19e into development Sep 12, 2025
0 of 4 checks passed
@alex-sandercock alex-sandercock deleted the polyrad_bugfix branch September 12, 2025 16:53
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.

2 participants