Skip to content

Conversation

@Ulthran
Copy link
Contributor

@Ulthran Ulthran commented Oct 7, 2025

Summary

  • rename the extension to sbx_cenote_taker and update documentation
  • remove VirSorter support and associated configuration, scripts, and environments
  • ensure the workflow exclusively runs Cenote-Taker3 and reference the new environment assets

Testing

  • not run (not requested)

https://chatgpt.com/codex/tasks/task_e_68e51710b9b08323b599c7b42168c193

Copilot AI review requested due to automatic review settings October 7, 2025 19:20
Copy link
Contributor

Copilot AI left a comment

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 renames the sbx_virus_id extension to sbx_cenote_taker and removes VirSorter2 support to focus exclusively on Cenote-Taker3 for viral identification. The changes streamline the workflow by eliminating the dual-tool approach and standardizing on a single viral identification method.

  • Renamed extension from sbx_virus_id to sbx_cenote_taker throughout codebase
  • Removed VirSorter2 integration including rules, scripts, environments, and configuration options
  • Updated documentation to reflect Cenote-Taker3-only workflow

Reviewed Changes

Copilot reviewed 12 out of 13 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
scripts/filter_virsorter_contigs.py Completely removed VirSorter filtering script
sbx_cenote_taker.smk Main workflow file with extensive renaming and VirSorter rule removal
envs/virsorter_env.yml Removed VirSorter conda environment definition
envs/virsorter_env.linux-64.pin.txt Removed VirSorter environment pinned dependencies
envs/virsorter_env.Dockerfile Removed VirSorter Docker environment
envs/spades_env.Dockerfile Updated working directory to reflect new extension name
envs/sbx_virus_id.Dockerfile Removed old Docker environment
envs/sbx_cenote_taker.yml Renamed environment from sbx_virus_id to sbx_cenote_taker
envs/sbx_cenote_taker.Dockerfile New Docker environment with updated naming
envs/cenote_taker_env.Dockerfile Updated working directory to reflect new extension name
config.yml Simplified configuration removing VirSorter options
README.md Updated documentation for new extension name and Cenote-Taker3 focus

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Ulthran Ulthran merged commit 69f855b into main Oct 7, 2025
2 of 3 checks passed
@Ulthran Ulthran deleted the codex/clean-up-extension-and-rename-to-sbx_cenote_taker branch October 7, 2025 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants