Skip to content

Conversation

@d-callan
Copy link
Collaborator

@d-callan d-callan commented Jan 7, 2026

Description

this does two things:

  1. adds display name for veupath gtf files
  2. makes sure we only pull gtf from the /genes path, not the contributor path. this will ensure when we display options that were contributed to ucsc, we only show them once.

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 enhances VEuPathDB GTF file handling by adding a display name and preventing duplicate entries. The changes ensure that VEuPathDB GTF files contributed to UCSC are only shown once by filtering exclusively from the /genes/ path.

Key Changes:

  • Added display name mapping for VEuPathDB GTF files
  • Implemented path filtering to exclude contributor paths and show only /genes/ GTF files

Reviewed changes

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

File Description
app/.../GTFStep/utils.ts Adds "VEuPathDB" display label pattern for veupathGenes GTF files
app/.../UseUCSCFiles/utils.ts Filters GTF files to only include those from /genes/ path to prevent duplicates

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

@NoopDog
Copy link
Collaborator

NoopDog commented Jan 7, 2026

Hi @d-callan, can you address the CP feedback? Thanks!

@d-callan
Copy link
Collaborator Author

d-callan commented Jan 7, 2026

I mean I can update the jsdoc , the other is intentional

@NoopDog NoopDog merged commit 3c230dc into main Jan 8, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants