Bug fix related to correct default targets file in GUI

[samarth8392]

Changes

Updated gui.py to include another if-else statement to choose the correct default targets file based on the genome selected.

Issues

Resolves #108

PR Checklist

(Strikethrough any points that are not applicable.)

• This comment contains a description of changes with justifications, with any relevant issues linked.
  - [ ] Update docs if there are any API changes.
• Update CHANGELOG.md with a short description of any user-facing changes and reference the PR number. Guidelines: https://keepachangelog.com/en/1.1.0/

[kelly-sovacool]

run() selects the targets file in the genome config if the value of --targets is None or "" (empty string). i.e. --targets is now optional but can be used to override the genome's default targets file.

XAVIER/src/xavier/run.py

Lines 373 to 379 in dd7509e

	config["input_params"]["EXOME_TARGETS"] = (
	str(sub_args.targets)
	if sub_args.targets
	else os.path.join(
	config["project"]["workpath"], config["references"]["exome_targets"]
	)
	)

Instead of hardcoding targets with an if-else statement in the GUI logic, can you allow GUI users to pass an empty string through -TARGETS-, so it will work the same way as the CLI?

[kelly-sovacool]

@samarth8392 See comment below and let me know!

[samarth8392]

Thanks @kelly-sovacool , the updated code works as expected. I have pushed the changes. Ready for final review.

[kelly-sovacool]

@samarth8392 Looks good, thanks for fixing this!