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

true_min not found #1

Closed
GuangchuangYu opened this issue Aug 22, 2017 · 5 comments
Closed

true_min not found #1

GuangchuangYu opened this issue Aug 22, 2017 · 5 comments

Comments

@GuangchuangYu
Copy link

> library(gggenes)
>
> dummies <- make_alignment_dummies(
+   gggenes_example_genes,
+   aes(xmin = start, xmax = end, y = molecule, id = gene),
+   on = "genE"
+ )
Error in ifelse(side == "left", true_min, true_max) :
  object 'true_min' not found

This may due to the upgrade of dplyr.

> packageVersion("dplyr")
[1] ‘0.7.2’
@wilkox wilkox closed this as completed in a4d1062 Aug 22, 2017
@wilkox
Copy link
Owner

wilkox commented Aug 22, 2017

Thanks @GuangchuangYu, looks like it was a dplyr change. Should be working now.

@GuangchuangYu
Copy link
Author

GuangchuangYu commented Aug 22, 2017

I also figure it out. I think it would be more safe to use select_ and all using column name (character).

I am think about using it with ggtree similar to https://stackoverflow.com/questions/45384281/ggjoy-facet-with-ggtree. It is not possible to work with facet_plot as you are using facet_wrap .

Maybe you can work out another version without facet_wrap, so that we can have lawremi/ggbio#83 using gggenes.

@GuangchuangYu
Copy link
Author

GuangchuangYu commented Aug 22, 2017

FYI: a wrapper function geom_motif that works with ggtree.
YuLab-SMU/ggtree#148

@wilkox
Copy link
Owner

wilkox commented Aug 22, 2017

geom_motif looks like a good implementation of this idea. Since you're importing geom_gene_arrow, would it help if I tidy up 'gggenes' and put it on CRAN? Or you're welcome to just copy the functions straight into ggtree if you want.

BTW, thanks for making ggtree! It's a great package.

@GuangchuangYu
Copy link
Author

thanks @wilkox for putting the gggenes on CRAN.

I think the geom_motif should be refined and better incorporated in gggenes.

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

No branches or pull requests

2 participants