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

music_prop_t_statistics matrix vs ExpressionSet issue #107

Open
jaclynbeck-sage opened this issue Nov 7, 2022 · 2 comments
Open

music_prop_t_statistics matrix vs ExpressionSet issue #107

jaclynbeck-sage opened this issue Nov 7, 2022 · 2 comments

Comments

@jaclynbeck-sage
Copy link

The function music_prop_t_statistics seems to expect a matrix as input for both bulk.control.mtx and bulk.case.mtx, and works that way for most of the function. However, at one point it calls

case_s = exprs(bulk.case[exp_genel, colnames(bulk.case) %in% 
        id_d])

This fails because bulk.case is a matrix, not an ExpressionSet. But you can't call the music_prop_t_statistics function successfully using an ExpressionSet for arguments.

I think removing the exprs call should work? i.e.

case_s = bulk.case[exp_genel, colnames(bulk.case) %in% 
        id_d]
@ProsperP
Copy link

ProsperP commented Mar 3, 2023

Hi @jaclynbeck-sage,
I also ran in to this problem, and this is needed to be solved urgently. Besides, I think the correct function name should be music2_prop_t_statistics : ).

For myself, I solved this by copying music2_prop_t_statistics function to an R script file and modifying the code. Then, one can import the correct function by sourcing this R script through Source('path_to_R_script'). Hoping this could help, and hoping the problem could be solved officially. Thanks in advance @xuranw.

@nadinegheziel
Copy link

nadinegheziel commented Apr 11, 2023

Hello !
I had the same issue but modifying the source code as you said worked for me @ProsperP

case_s = bulk.case[exp_genel, colnames(bulk.case) %in% id_d]
It works @jaclynbeck-sage

Thank to both of you !
Nice day

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants