Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add common methods.set_pipeline_logs method #56

Open
nwiltsie opened this issue Dec 22, 2023 · 1 comment
Open

Add common methods.set_pipeline_logs method #56

nwiltsie opened this issue Dec 22, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@nwiltsie
Copy link
Member

Every pipeline has a block like this in methods.config:

    set_pipeline_logs = {
        trace.enabled = true
        trace.file = "${params.log_output_dir}/nextflow-log/trace.txt"

        timeline.enabled = true
        timeline.file = "${params.log_output_dir}/nextflow-log/timeline.html"

        report.enabled = true
        report.file = "${params.log_output_dir}/nextflow-log/report.html"
    }

That seems like something that we should pull into a common repository.

@nwiltsie nwiltsie added the enhancement New feature or request label Dec 22, 2023
@yashpatel6
Copy link
Collaborator

Yeah that makes sense, we may want to parameterize the params.log_output_dir portion as an input for the common function

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants