Skip to content

Conversation

@Cristianetaniguti
Copy link
Collaborator

  • New function to check MADC file: check_madc_sanity. By now, it checks presence of required columns, if fixed allele IDs were assigned, presence of IUPAC codes, lower case sequences bases, indels, and chromosome and position information.
  • Add new arguments markers_info for users to add CSV file with marker information such as CHROM, POS, Marker Type and position of indels
  • For BI species, these information is at https://github.com/Breeding-Insight/BIGapp-PanelHub

@Cristianetaniguti Cristianetaniguti added enhancement New feature or request in_progress not ready to merge labels Oct 3, 2025
@codecov
Copy link

codecov bot commented Oct 3, 2025

Codecov Report

❌ Patch coverage is 73.38129% with 37 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.24%. Comparing base (a248e93) to head (0b97b46).

Files with missing lines Patch % Lines
R/madc2vcf_targets.R 63.09% 31 Missing ⚠️
R/check_madc_sanity.R 93.61% 3 Missing ⚠️
R/madc2vcf_all.R 71.42% 2 Missing ⚠️
R/utils.R 0.00% 1 Missing ⚠️
Additional details and impacted files
@@               Coverage Diff               @@
##           development      #45      +/-   ##
===============================================
- Coverage        83.35%   82.24%   -1.12%     
===============================================
  Files               19       20       +1     
  Lines             1358     1436      +78     
===============================================
+ Hits              1132     1181      +49     
- Misses             226      255      +29     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request in_progress not ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants