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

'x@assays' is not parallel to 'x' #96

Closed
lazappi opened this issue May 23, 2023 · 3 comments
Closed

'x@assays' is not parallel to 'x' #96

lazappi opened this issue May 23, 2023 · 3 comments
Labels
bug Something isn't working

Comments

@lazappi
Copy link
Member

lazappi commented May 23, 2023

          Hi everyone,

I would like to jump in on this. I am working with the dataset from GSE174188 (GSE174188_CLUES1_adjusted.h5ad.gz).

In R/4.1.3, zellkonverter 1.4.0, I was able to simply obtain the data with

lupus <- readH5AD(file=lupus_file, use_hdf5 = TRUE, raw = TRUE, verbose = TRUE)

However, after switching to R/4.2.0, zellkoverter 1.8.0, the same code leads to following error:

Error in validObject(.Object) :
 invalid class “SummarizedExperiment” object:
  'x@assays' is not parallel to 'x'
Calls: readH5AD ... new_SummarizedExperiment -> new -> initialize -> initialize -> validObject
In addition: Warning messages:
1: raw 'X' matrix does not support transposition and has been skipped
2: The names of these selected raw var columns have been modified to match R
conventions:
'feature_types-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0-0'
->
'feature_types.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0'
Execution halted

It turns out that that the warning "raw 'X' matrix does not support transposition and has been skipped" is the underlying cause of the assays and rowdata dimensions being mismatched, which finally results in SummarizedExperiment complaining.

In the light of this thread, I also tried with zellkonverter 1.11.0, but this did not resolve my issues.

Originally posted by @jgilis in #42 (comment)

@lazappi lazappi changed the title 'x@assays' is not parallel to 'x' in **{zellkonverter}** v1.10.0 'x@assays' is not parallel to 'x' May 23, 2023
@lazappi lazappi added the bug Something isn't working label May 23, 2023
@lazappi
Copy link
Member Author

lazappi commented May 23, 2023

Ok, so there were two things here. One is that the wrong dimensions were being used when reading raw (that is what was causing the error). The other is that the format used for storing backed sparse arrays wasn't been recognised correctly (this was the cause of the transposition warning and maybe some other stuff in #42).

These should both be fixed now in release (v1.10.1) and devel (v1.11.1).

@jgilis
Copy link

jgilis commented May 23, 2023

Thank you for the fast adjustments, I will check if this works on my side before the weekend.

@jgilis
Copy link

jgilis commented May 23, 2023

Okay I already checked, this indeed solves the issue for me! Thank you for the quick fix 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants