-
Notifications
You must be signed in to change notification settings - Fork 24
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
New .peaksapply fails with MsBackendMassbankSql #187
Comments
Can you please provide the output of your |
Oh man. I fixed it now. Thanks so much @meowcat for reporting and the reproducible example! The problem was that the |
fix: drop names on peaks list (fixes issue #187)
Can you please verify that it's working now @meowcat and eventually close the issue? |
Works, thanks! |
Hi,
filing this here, perhaps it's a MsBackendMassbankSql issue. Using
addProcessing
with the "new".peaksapply
, fails for me on aMsBackendMassbankSql
backend at time of applying a lazy queue.Error in (function (x, ...) : argument "x" is missing, with no default
It works if I convert the data to
MsBackendDataFrame
first. Everything worked with the old.peaksapply
where argumentsms2Level
andcentroided
were expected.I can't get far in debugging because the error handler obscures to me where the issue actually is coming from.
Below is a reprex with the
minimassbank.sqlite
:The text was updated successfully, but these errors were encountered: