KasperSkytte / ampvis2

Tools for visualising microbial community amplicon data
https://kasperskytte.github.io/ampvis2/
GNU General Public License v3.0
66 stars 23 forks source link

facet_by error in 'amp_heatmap' function #88

Closed Jessie-HOU closed 4 years ago

Jessie-HOU commented 4 years ago

Thank you for using ampvis2!

If you encounter a problem with ampvis2 that you are unable to resolve, this is the place for discussing the problem with developers to find a solution. To best help you quickly, please first make sure that:

Should the problem still persist, then please provide a minimal reproducible example, preferably using reprex, with self-contained code so that anyone can copy the code and reproduce the problem to easily start debugging without further communication.

If you suspect that the data used is involved in causing the trouble, then please upload it (preferably a minimal subset) to figshare or similar services and provide a link to it.

Jessie-HOU commented 4 years ago

Hello,

I am trying to using amp_heatmap function to draw heatmap for 16S data. But the facet_by parameter gave the following error. I have searched google and tried, but have not fixed this issue yet.

amp_heatmap(d,group_by = "Time_point",

  • facet_by = "Group",
  • tax_aggregate = "Genus",
  • tax_show = 25) Error in fix.by(by.y, y) : 'by' must specify a uniquely valid column

Any help or suggestion will be appreciated. Thanks in advance!

Best regards, Jessie

Jessie-HOU commented 4 years ago

Hi there!

I have find the solution to this error. That is, I changed the column name "Group" to "Subject_type". It does works! It seems that "Group" cannot be used as column names (Is it the 'key words' of this ampvis2 package?) when calling the amp_heatmap function. I am not sure about this currently, though.

Best regards, Jessie

KasperSkytte commented 4 years ago

Hi Jessie.

Yes it could be that the variable name Group can cause some trouble. But simply rename it and it should work fine.

Jessie-HOU commented 4 years ago

Dear Kasper Skytte,

I see. Thank you for your information!

Best regards, Jessie

On Tue, Mar 31, 2020 at 4:19 PM Kasper Skytte Andersen < notifications@github.com> wrote:

Closed #88 https://github.com/MadsAlbertsen/ampvis2/issues/88.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/MadsAlbertsen/ampvis2/issues/88#event-3182103405, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHE7VRDNL2EK7AZNIYTBAALRKGRRBANCNFSM4LVPH5XQ .