Skip to content

Commit fd3a930

Browse files
committed
Support sharedPosixOptimizations for GenotypeGVCFs
1 parent 9e8ea66 commit fd3a930

File tree

2 files changed

+17
-2
lines changed

2 files changed

+17
-2
lines changed

SequenceAnalysis/resources/web/SequenceAnalysis/panel/VariantProcessingPanel.js

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -218,7 +218,14 @@ Ext4.define('SequenceAnalysis.panel.VariantProcessingPanel', {
218218
commandLineParam: '--max_alternate_alleles',
219219
defaultValue: 12
220220
},{
221-
fieldXtype: 'checkbox',
221+
fieldXtype: 'checkbox',
222+
name: 'sharedPosixOptimizations',
223+
label: 'Use Shared Posix Optimizations',
224+
description: 'This enabled optimizations for large shared filesystems, such as lustre.',
225+
commandLineParam: '--genomicsdb-shared-posixfs-optimizations',
226+
defaultValue: true
227+
},{
228+
fieldXtype: 'checkbox',
222229
name: 'includeNonVariantSites',
223230
label: 'Include Non-Variant Sites',
224231
description: 'If checked, all sites will be output into the VCF, instead of just those where variants are detected. This can dramatically increase the size of the VCF.',

SequenceAnalysis/src/org/labkey/sequenceanalysis/analysis/GenotypeGVCFHandler.java

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -121,6 +121,9 @@ public GenotypeGVCFHandler()
121121
// ToolParameterDescriptor.createCommandLineParam(CommandLineParam.create("-stand_call_conf"), "stand_call_conf", "Threshold For Calling Variants", "The minimum phred-scaled confidence threshold at which variants should be called", "ldk-numberfield", null, 30),
122122
// ToolParameterDescriptor.createCommandLineParam(CommandLineParam.create("--max_alternate_alleles"), "max_alternate_alleles", "Max Alternate Alleles", "Maximum number of alternate alleles to genotype", "ldk-integerfield", null, 12),
123123
// ToolParameterDescriptor.createCommandLineParam(CommandLineParam.createSwitch("--includeNonVariantSites"), "includeNonVariantSites", "Include Non-Variant Sites", "If checked, all sites will be output into the VCF, instead of just those where variants are detected. This can dramatically increase the size of the VCF.", "checkbox", null, false)
124+
// ToolParameterDescriptor.create("sharedPosixOptimizations", "Use Shared Posix Optimizations", "This enabled optimizations for large shared filesystems, such as lustre.", "checkbox", new JSONObject(){{
125+
// put("checked", true);
126+
// }}, true),
124127
}
125128

126129
@Override
@@ -367,12 +370,17 @@ private File runGenotypeGVCFs(PipelineJob job, JobContext ctx, ProcessVariantsHa
367370
});
368371
}
369372

370-
if (ctx.getParams().optBoolean("disableFileLocking", false))
373+
if (ctx.getParams().optBoolean("variantCalling.GenotypeGVCFs.disableFileLocking", false))
371374
{
372375
ctx.getLogger().debug("Disabling file locking for TileDB");
373376
wrapper.addToEnvironment("TILEDB_DISABLE_FILE_LOCKING", "1");
374377
}
375378

379+
if (ctx.getParams().optBoolean("variantCalling.GenotypeGVCFs.sharedPosixOptimizations", false))
380+
{
381+
toolParams.add("--genomicsdb-shared-posixfs-optimizations");
382+
}
383+
376384
wrapper.execute(genome.getSourceFastaFile(), outputVcf, toolParams, inputVcf);
377385

378386
action.addOutput(outputVcf, "VCF", outputVcf.exists(), true);

0 commit comments

Comments
 (0)