Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Parser bug #13

Closed
albertop210 opened this issue Apr 18, 2024 · 2 comments
Closed

Parser bug #13

albertop210 opened this issue Apr 18, 2024 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@albertop210
Copy link

Dear authors, compliments for the great work and useful tool!
Just found a bug (I suppose?) in some of you parser code, for example at line 399 in load_VCF_smallVar_DRAGEN definition, when running separate(id, into = c('chr', 'from', 'to', 'ref', 'alt'), sep = ':') %>% I get an error of the separate tidyr function, telling me that a function cannot be extracted Error in tidyr::separate(): ! Can't extract column with !!enquo(col). ✖ !!enquo(col) must be numeric or character, not a function.
I wonder this issue is because R considers 'id' as a function (plyr package); I solved the issue by changing the pullAD_DRAGEN function and the load_VCF_smallVar_DRAGEN function, substituting 'ID' to 'id'.
Now it seems to work great. Hope this helps!
Thanks,
Alberto

@caravagn caravagn added the bug Something isn't working label Apr 19, 2024
@caravagn
Copy link
Collaborator

Thanks, @Militeee is now on this.

@Militeee
Copy link
Contributor

Hi @albertop210 thanks for catching this, I am not a plyr user so I completely miss that id is also a function. Should be fixed now.

Feel free to reopen if you encounter similar bugs.
Thx again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants