Skip to content

Commit

Permalink
@mashehu suggested that downloading the containers should not be opti…
Browse files Browse the repository at this point in the history
…onal for Tower downloads. Given that there is the option to provide the list of remote containers to skip their download, I agree that this is reasonable.
  • Loading branch information
MatthiasZepper committed May 9, 2023
1 parent c0ccd2c commit c7cede3
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions nf_core/download.py
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,7 @@ def __init__(
self.force = force
self.tower = tower
self.include_configs = None
self.container = container if not singularity_cache_index else "singularity"
self.container = container if not singularity_cache_index or not tower else "singularity"
self.singularity_cache = (
singularity_cache if not singularity_cache_index else "remote"
) # if a singularity_cache_index is given, use the file and overrule choice.
Expand Down Expand Up @@ -377,7 +377,7 @@ def prompt_config_inclusion(self):
def prompt_container_download(self):
"""Prompt whether to download container images or not"""

if self.container is None and stderr.is_interactive:
if self.container is None and stderr.is_interactive and not self.tower:
stderr.print("\nIn addition to the pipeline code, this tool can download software containers.")
self.container = questionary.select(
"Download software container images:",
Expand Down

0 comments on commit c7cede3

Please sign in to comment.