GATK wrappers should use workaround for optional arrays #70

Open
kshakir opened this Issue Nov 9, 2016 · 1 comment

Comments

Projects
None yet
2 participants
Contributor

kshakir commented Nov 9, 2016

Until #25 is fixed, the gatk wrappers should not be using optional arrays, such as the intervals files. Instead they should use this bash workaround, with something similar to:

Array[String]? foo

command {
  if [ -n '${sep=',' foo}' ]; then
    FLAG=--prefix=${sep=',' foo}
  else
    FLAG=''
  fi

  echo $FLAG
}

vdauwera was assigned by kshakir Nov 9, 2016

Collaborator

vdauwera commented Jan 15, 2017

So we never got around to fixing the GATK wrappers... do we still need to or is this handled in Cromwell 24?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment