-
Notifications
You must be signed in to change notification settings - Fork 14
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
2D Box gates treated as Quadrant Gates in .wsp output file #121
Comments
Thanks, Alex. I think this is a good idea, we'll see about implementing the feature. |
Could you please provide the example wsp file? |
@mikejiang Here is the .wsp file for the example I showed above. The specific gates related to this issue are: /Cells/Singlets-H/Singlets-W/Cleanup/Viable/Leukocytes/CD3+ Cells/T Cells/ab T Cells/CD4 T Cells/Non-Treg CD4/Non-naive CD4 T Cells/CXCR5- PD-1 High and .../Non-naive CD4 T Cells/cTfh |
These two gates don't share the same intersection (center of quadgate) so they must belong to two different quadgates.(and they happen to share the same x,y axis and under that same parents.) |
If previous version of outputting simple 2d rectangle gate worked, then I will go ahead to add the option to disable quadgate as you suggested. |
Here is a little recap of my thoughts (mainly as the notes for future trace back on this topic). gs_add_gating_method(gs, pop = "+/-+/-", parent = "root", dims = "FSC-A,SSC-A", gating_method = "mindensity")
EDIT: Actually the new behavior of storing as gs_pop_add(gs, quadGate(filterId="myQuadGate1", "FSC-A"=100, "SSC-A"=400)) And for So to be less intrusive to |
Hey @mikejiang, Sorry for the delay getting back to you. I think your solution would be great, especially since you can determine which gates you want to be quads and rectangles specifically. Would this be part of Also, if you need my gs archive for this sample still, I can send it. What is the best way of getting you files that are above the 10MB limit for github issue threads? |
I will put it in |
I've corrected my inaccurate comment above regarding to the > rectangleGate(`FSC-A` = c(100, Inf), `SSC-A` = c(200, Inf))
Rectangular gate 'defaultRectangleGate' with dimensions:
FSC-A: (100,Inf)
SSC-A: (200,Inf) We used to replace <gating:RectangleGate>
<gating:dimension gating:min="100" gating:max="2147483647">
<data-type:fcs-dimension data-type:name="FSC-A"/>
</gating:dimension>
<gating:dimension gating:min="200" gating:max="2147483647">
<data-type:fcs-dimension data-type:name="SSC-A"/> Now the new logic introduced in this release is always to omit infinite bounds, so we ends up to <gating:RectangleGate>
<gating:dimension gating:min="100">
<data-type:fcs-dimension data-type:name="FSC-A"/>
</gating:dimension>
<gating:dimension gating:min="200">
<data-type:fcs-dimension data-type:name="SSC-A"/> So when FlowJo sees this type of So |
Hey @mikejiang, Just wondering if there are any updates for this issue, thanks! Alex |
Hi @gfinak and @mikejiang,
Sorry to bother you again, I have one more thing to work out with the FlowJo .wsp files that I'm exporting with CytoML.
In the .wsp file I generate from openCyto gating, there are some 2D box gates that are offset, like this one:
When I go to move one of the gates in FlowJo, it acts like a quadrant gate, and moves the lines for both gates at once. The corner vertices can't be selected, only the center where the boxes connect:
Once the gate is moved, it can't be moved back from the quadrant gate format:
I believe this was changed recently, when I used a previous version of CytoML each gate was treated as it's own 2D box gate. I understand the reason for this change (I think), in that when you need to move a quadrant gate, you would have to select all four boxes to move them all at once. Would it possible to add an input for the
gs_to_flowjo
function to chose how to interpret 2D gates, either as box gates or quadrant gates?Let me know what you think, and I can provide an example if necessary.
Thanks,
Alex
The text was updated successfully, but these errors were encountered: