-
Notifications
You must be signed in to change notification settings - Fork 12
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
Error when running SAKE on Ting et al dataset #81
Comments
After investigation, this appears to be due to the following call:
Which occurs on lines 1809, 1814, 1984 of
My guess is the current version of NMF is no longer returning an |
Hi, The same error is still present, even with NMF version 0.22.0: Compute NMF rank= 2 ... + measures ... OK
Compute NMF rank= 3 ... + measures ... OK
Compute NMF rank= 4 ... + measures ... OK
Compute NMF rank= 5 ... + measures ... OK
Compute NMF rank= 6 ... + measures ... OK
Compute NMF rank= 7 ... + measures ... OK
[1] "### Time to run NMF ###"
user system elapsed
1703.171 74.646 414.704
Warning: Error in h: error in evaluating the argument 'x' in selecting a method for function 'sort': error in evaluating the argument 'x' in selecting a method for function 'unique': Seems like you haven't run NMF 'real' run yet
79: <Anonymous>
Warning: Error in h: error in evaluating the argument 'x' in selecting a method for function 'sort': error in evaluating the argument 'x' in selecting a method for function 'unique': Seems like you haven't run NMF 'real' run yet
79: <Anonymous>
> installed.packages()["NMF", "Version"]
[1]. "0.22.0" Thanks. |
Hey Oliver, After thinking about this some more, I think my initial assumption that the problem was with the NMF package is wrong. My current hunch is that the DESeq2 tab is being called immediately after NMF estimation is done running. The DESeq2 tab expects an It's a long shot, but a potential fix may be downgrading to the package versions I have in my currently working R3.6 installation:
It's also possible, but unlikely, that the change occurred in sake itself. Downgrading sake may also help.
This will downgrade to this commit which closely matches the version I currently have working on R3.6. Edit: Updated post to use a commit/version of SAKE without an install bug. |
Updated previous comment to use a version of sake without the install bug. The |
I also encounter the exact same error message. @ColeWunderlich, I tried your suggestion and downgraded to R3.6.0 along with shiny 1.4.0.2
shinydashboard 0.7.1
shinythemes 1.1.2 but the error remained. Any ideas what might be happening? Could you maybe post your full |
Hi,
We have encountered an error when processing the Ting et al. dataset through SAKE.
Here are the parameters that we used:
When we run SAKE, it would crash after the estimation of
k
step.Error message:
Traceback:
SessionInfo:
Thanks for all your help.
The text was updated successfully, but these errors were encountered: