Skip to content

Conversation

@djhammill
Copy link
Contributor

@amcdavid adding PR here to implement features or fixes required by CytoExploreR please see https://ozette.atlassian.net/browse/ODC-182

NOTE: This is draft PR it is known to causes issues for some edge cases, I will investigate a better solution and update this PR.

@djhammill djhammill marked this pull request as draft November 5, 2024 04:38
@djhammill djhammill requested a review from amcdavid November 5, 2024 04:38
@djhammill djhammill self-assigned this Nov 5, 2024
@djhammill
Copy link
Contributor Author

I am thinking that maybe we should only invoke the new code if the gating function is cyto_gate_clust() - that will ensure that everything behaves normally within openCyto and CytoExploreR.

@djhammill
Copy link
Contributor Author

Alternatively we separate out filter and filters objects from logical and factor and treat those separately. It doesn't make sense to replicate event-wise logical or factor filters across multiple samples where the event number may be different. I will try converting the gates within CytoExploreR to be either logical or factor to see if I can get it to work.

@djhammill djhammill added the WIP Work in Progress label Mar 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement WIP Work in Progress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants