Skip to content

Node degree distribution#180

Merged
Schansiate merged 20 commits into
devfrom
node_degree_distribution
Jun 23, 2026
Merged

Node degree distribution#180
Schansiate merged 20 commits into
devfrom
node_degree_distribution

Conversation

@Schansiate

Copy link
Copy Markdown
Collaborator

PR checklist

  • This comment contains a description of changes (with reason).
  • If you've fixed a bug or added code that should be tested, add tests!
  • If you've added a new tool - have you followed the pipeline conventions in the contribution docs
  • If necessary, also make a PR on the nf-core/diseasemodulediscovery branch on the nf-core/test-datasets repository.
  • Make sure your code lints (nf-core pipelines lint).
  • Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
  • Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
  • Usage Documentation in docs/usage.md is updated.
  • Output Documentation in docs/output.md is updated.
  • CHANGELOG.md is updated.
  • README.md is updated (including new tool citations and authors/contributors).

Copilot AI review requested due to automatic review settings April 10, 2026 13:31
@nf-core-bot

nf-core-bot commented Apr 10, 2026

Copy link
Copy Markdown
Member

Warning

Newer version of the nf-core template is available.

Your pipeline is using an old version of the nf-core template: 3.5.1.
Please update your pipeline to the latest version.

For more documentation on how to update your pipeline, please see the Synchronisation documentation.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a node degree distribution summary for parsed input networks and integrates it into the final MultiQC report.

Changes:

  • Extend graph_tool_parser.py / GRAPHTOOLPARSER to emit per-network node degree distribution YAML.
  • Add a new MULTIQCFORMATTER module + multiqc_formatter.py to combine per-network distributions into a MultiQC custom-content YAML.
  • Register the new MultiQC section and header config in assets/ and pipeline workflow wiring.

Reviewed changes

Copilot reviewed 6 out of 10 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
workflows/diseasemodulediscovery.nf Wires the new node degree distribution output into MultiQC inputs via MULTIQCFORMATTER.
modules/local/multiqcformatter/main.nf New process to generate MultiQC-ready YAML from distribution inputs + header.
modules/local/graphtoolparser/main.nf Exposes node degree distribution YAML as an additional optional output.
conf/modules.config Adds publishDir rules for MULTIQCFORMATTER.
bin/graph_tool_parser.py Generates per-network node degree distribution YAML when parsing to gt.
bin/multiqc_formatter.py New formatter script producing MultiQC custom-content YAML for the distribution plot.
assets/network_node_degree_distribution_header.yaml New MultiQC custom-content header/template for node degree distribution.
assets/multiqc_config.yml Adds the new section id to the custom_content ordering.
bin/network_annotation.py Adds full-network and module-degree vertex properties to annotated subnetworks.
bin/visualize_modules.py Adds runtime printing of pyvis version.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread conf/modules.config
Comment thread assets/network_node_degree_distribution_header.yaml Outdated
Comment thread workflows/diseasemodulediscovery.nf Outdated

@JohannesKersting JohannesKersting left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Schansiate
Please also update the output documentation and change to the new dependency container

Comment thread bin/visualize_modules.py Outdated
Comment thread modules/local/graphtoolparser/main.nf
Comment thread modules/local/multiqcformatter/main.nf
@Schansiate Schansiate merged commit 9bb5542 into dev Jun 23, 2026
8 of 9 checks passed
@Schansiate Schansiate deleted the node_degree_distribution branch June 23, 2026 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants