FlowKit icon indicating copy to clipboard operation
FlowKit copied to clipboard

Support for matrix-inverted-already attribute in GatingML

Open krcurtis opened this issue 3 years ago • 5 comments

Is your feature request related to a problem? Please describe. FlowKit does not appear to use the matrix-inverted-already flag in GatingML

Describe the solution you'd like It would be nice if FlowKit could use an already inverted matrix. For example, the DIVA vendor specific XML files already include a compensation matrix, but it appears to be already inverted. I'd like to use their matrix as-is without going thru a matrix inversion to avoid adding more noise.

Describe alternatives you've considered I can invert the matrix found in DIVA vendor specific files and not use the matrix-inverted-already flag when generating GatingML files to use with FlowKit

Additional context

krcurtis avatar Oct 19 '22 01:10 krcurtis

Thanks for the feedback. I must have missed this option in the specification. I'll take a look at adding support for this. Any chance you can provide an example GatingML document that uses this?

-Scott

whitews avatar Dec 15 '22 00:12 whitews

On-hold until I have an example GatingML file using matrix-inverted-already

whitews avatar May 18 '23 20:05 whitews

I'm not sure if my request is useful anymore. I guess I'd suggest closing as "won't fix" until a more compelling use case comes up

krcurtis avatar May 18 '23 20:05 krcurtis

I'm definitely open to supporting it, just need an example of a valid example Diva XML file using the matrix-inverted-already attribute, preferably with FCS files. I'll tag this as help wanted.

whitews avatar May 18 '23 21:05 whitews

I'm a bit late to the party, but:

I'd like to use their matrix as-is without going thru a matrix inversion to avoid adding more noise.

Matrix inversions are basically exact up to a negligible numerical error, you'll lose nothing of relevance by inverting your spillover matrix a couple of times.

valentinwust avatar Jul 08 '24 10:07 valentinwust