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

[batch_int] fastmnn_feature fail #270

Open
KaiWaldrant opened this issue Oct 18, 2023 · 1 comment
Open

[batch_int] fastmnn_feature fail #270

KaiWaldrant opened this issue Oct 18, 2023 · 1 comment
Labels
batch_integration relates to task batch_integration bug Something isn't working

Comments

@KaiWaldrant
Copy link
Member

KaiWaldrant commented Oct 18, 2023

Describe the bug
An error occured for fastmnn_feature process on the cengen_logcp10k dataset in batch_integration full benchmark run:

Loading dependencies
Read input
Run mnn
Reformat output
Error in rbind(...) : negative extents to matrix
Calls: as ... <Anonymous> -> standardGeneric -> eval -> eval -> eval -> rbind
Execution halted
@KaiWaldrant KaiWaldrant added bug Something isn't working batch_integration relates to task batch_integration labels Oct 18, 2023
@KaiWaldrant KaiWaldrant changed the title [batch_int] fastmnn_py fail [batch_int] fastmnn_feature fail Oct 18, 2023
@KaiWaldrant
Copy link
Member Author

After elaborate investigation I have narrowed it down but not yet confirmed.

I could not replicate the issue when running the component locally without docker. The difference between local system and docker system is the R version and the package versions:

Docker:

  • R: 4.3.1
  • anndata: 0.7.5.6
  • Matrix: 1.6.0
  • batchelor: 1.17.2
  • SinglCellExperiment: 1.22.0

local:

  • R: 4.2.3
  • anndata: 0.7.5.6
  • Matrix: 1.6.0
  • batchelor: 1.14.1
  • SingleCellExperiment: 1.20.1

Increasing the batchelor version on local machine did not invoke the expected error. Which would seem like the SingelCellExperiment package is the likely culprit. This has not yet been confirmed due to not being able to update R on my local system.

Further investigation/confirmation is required.

@KaiWaldrant KaiWaldrant added this to the v2 (migration) milestone Oct 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
batch_integration relates to task batch_integration bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant