Skip to content

ENSCORESW-2810: pass RFAMParser even if no xrefs found

Marek Szuba requested to merge bugfix/empty_rfam into master

Created by: magaliruffier

Requirements

  • Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion;
  • Review the contributing guidelines for this repository; remember in particular:
    • do not modify code without testing for regression
    • provide simple unit tests to test the changes
    • if you change the schema you must patch the test databases as well, see Updating the schema
    • the PR must not fail unit testing

Description

Using one or more sentences, describe in detail the proposed changes. The proposed change prevents the RFAMParser from failing if no xrefs are created. The change has already been merged onto release/94, now ensuring it also gets included onto master

Use case

Describe the problem. Please provide an example representing the motivation behind the need for having these changes in place. The RFAMParser relies on finding transcripts which have RFAM accessions as supporting features. This happens when the ncRNA pipeline has been run for the species in question. Not all species will have that data, but we still want to run the xref pipeline for those species without failing on the RFAMParser. Two species that triggered this change are mus spretus and mus castanus.

Benefits

If applicable, describe the advantages the changes will have. The xref pipeline does not fail if no RFAM annotation is available for a species.

Possible Drawbacks

If applicable, describe any possible undesirable consequence of the changes. Most species should have RFAM annotation and we will not be picking this up with the xref pipeline. However, this should probably be spotted by other mechanisms than the xref update.

Testing

Have you added/modified unit tests to test the changes? The parser has been run on several species before and after the change. Before the change, species without RFAM annotation fail the parser. With the fix, the parser passes for all species, with or without RFAM annotation. This is also noted in ENSCORESW-2810

Merge request reports