Skip to content

Commit 4fba550

Browse files
committed
Update CheckExpectations and SeuratPrototype steps to include hashing/cite-seq data if needed
1 parent 06f7da5 commit 4fba550

File tree

7 files changed

+32
-8
lines changed

7 files changed

+32
-8
lines changed

singlecell/api-src/org/labkey/api/singlecell/pipeline/AbstractSingleCellPipelineStep.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -187,13 +187,13 @@ protected List<Chunk> getChunks(SequenceOutputHandler.JobContext ctx) throws Pip
187187
}
188188

189189
@Override
190-
public boolean requiresHashing()
190+
public boolean requiresHashing(SequenceOutputHandler.JobContext ctx)
191191
{
192192
return false;
193193
}
194194

195195
@Override
196-
public boolean requiresCiteSeq()
196+
public boolean requiresCiteSeq(SequenceOutputHandler.JobContext ctx)
197197
{
198198
return false;
199199
}

singlecell/api-src/org/labkey/api/singlecell/pipeline/AbstractSingleCellStep.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,9 +29,9 @@ default boolean createsSeuratObjects()
2929
return true;
3030
}
3131

32-
public boolean requiresCiteSeq();
32+
public boolean requiresCiteSeq(SequenceOutputHandler.JobContext ctx);
3333

34-
public boolean requiresHashing();
34+
public boolean requiresHashing(SequenceOutputHandler.JobContext ctx);
3535

3636
public boolean isIncluded(SequenceOutputHandler.JobContext ctx, List<SequenceOutputFile> inputs) throws PipelineJobException;
3737

singlecell/src/org/labkey/singlecell/analysis/AbstractSingleCellHandler.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -177,12 +177,12 @@ public void init(JobContext ctx, List<SequenceOutputFile> inputFiles, List<Recor
177177
SingleCellStep step = stepCtx.getProvider().create(ctx);
178178
step.init(ctx, inputFiles);
179179

180-
if (step.requiresCiteSeq())
180+
if (step.requiresCiteSeq(ctx))
181181
{
182182
requiresCite = true;
183183
}
184184

185-
if (step.requiresHashing())
185+
if (step.requiresHashing(ctx))
186186
{
187187
String methods = step.getProvider().getParameterByName("methods").extractValue(ctx.getJob(), step.getProvider(), step.getStepIdx(), String.class);
188188
if (methods != null && methods.contains(CellHashingService.CALLING_METHOD.demuxem.name()))

singlecell/src/org/labkey/singlecell/pipeline/singlecell/AppendCiteSeq.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ private static List<ToolParameterDescriptor> getParams()
6666
}
6767

6868
@Override
69-
public boolean requiresCiteSeq()
69+
public boolean requiresCiteSeq(SequenceOutputHandler.JobContext ctx)
7070
{
7171
return true;
7272
}

singlecell/src/org/labkey/singlecell/pipeline/singlecell/CheckExpectations.java

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,18 @@ public void init(SequenceOutputHandler.JobContext ctx, List<SequenceOutputFile>
5050

5151
}
5252

53+
@Override
54+
public boolean requiresHashing(SequenceOutputHandler.JobContext ctx)
55+
{
56+
return getProvider().getParameterByName("requireHashing").extractValue(ctx.getJob(), getProvider(), getStepIdx(), Boolean.class, false);
57+
}
58+
59+
@Override
60+
public boolean requiresCiteSeq(SequenceOutputHandler.JobContext ctx)
61+
{
62+
return getProvider().getParameterByName("requireCiteSeq").extractValue(ctx.getJob(), getProvider(), getStepIdx(), Boolean.class, false);
63+
}
64+
5365
@Override
5466
protected Chunk createParamChunk(SequenceOutputHandler.JobContext ctx, List<SeuratObjectWrapper> inputObjects, String outputPrefix) throws PipelineJobException
5567
{

singlecell/src/org/labkey/singlecell/pipeline/singlecell/RunCellHashing.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ public RunCellHashing create(PipelineContext ctx)
5454
}
5555

5656
@Override
57-
public boolean requiresHashing()
57+
public boolean requiresHashing(SequenceOutputHandler.JobContext ctx)
5858
{
5959
return true;
6060
}

singlecell/src/org/labkey/singlecell/pipeline/singlecell/SeuratPrototype.java

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,18 @@ public SeuratPrototype create(PipelineContext ctx)
6464
}
6565
}
6666

67+
@Override
68+
public boolean requiresHashing(SequenceOutputHandler.JobContext ctx)
69+
{
70+
return getProvider().getParameterByName("requireHashing").extractValue(ctx.getJob(), getProvider(), getStepIdx(), Boolean.class, false);
71+
}
72+
73+
@Override
74+
public boolean requiresCiteSeq(SequenceOutputHandler.JobContext ctx)
75+
{
76+
return getProvider().getParameterByName("requireCiteSeq").extractValue(ctx.getJob(), getProvider(), getStepIdx(), Boolean.class, false);
77+
}
78+
6779
@Override
6880
public void init(SequenceOutputHandler.JobContext ctx, List<SequenceOutputFile> inputFiles) throws PipelineJobException
6981
{

0 commit comments

Comments
 (0)