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

Fix UCSC trackhub assembly aliases matching #3625

Merged
merged 4 commits into from
Apr 5, 2023

Conversation

andrzejgrzelak
Copy link
Contributor

Currently if you try to add UCSC connection and your assemblies name doesn't match the assembly names used in UCSC trackhub configuration exactly they won't load even if you add proper aliases to them.

This change fixes that by looking into aliases for matching assemblies.

@github-actions github-actions bot added the needs label triage Needs a label to show in changelog (breaking, enhancement, bug, documentation, or internal) label Apr 5, 2023
@rbuels rbuels self-requested a review April 5, 2023 17:09
@rbuels rbuels assigned rbuels and unassigned rbuels Apr 5, 2023
Copy link
Contributor

@rbuels rbuels left a comment

Choose a reason for hiding this comment

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

looks good to me

@cmdcolin
Copy link
Collaborator

cmdcolin commented Apr 5, 2023

can merge as is, may update on main branch a little to use the 'hasName' or 'allAliases.includes' type cleanups and add a integration test

@cmdcolin cmdcolin merged commit 2d419f0 into GMOD:main Apr 5, 2023
@cmdcolin cmdcolin added bug Something isn't working and removed needs label triage Needs a label to show in changelog (breaking, enhancement, bug, documentation, or internal) labels Apr 5, 2023
@cmdcolin cmdcolin changed the title Ucsc trackhub assembly aliases matching Fix UCSC trackhub assembly aliases matching Apr 5, 2023
cmdcolin pushed a commit that referenced this pull request May 18, 2023
* Add aliases matching to existing name matching for assemblies from UCSC Track hub connection.

---------

Co-authored-by: grzelaka <andrzej.grzelak@contractors.roche.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants