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

add script to create ENA compliant file by adding info to CDS, specif… #412

Closed
wants to merge 1 commit into from

Conversation

LucileSol
Copy link
Contributor

We needed a quick and dirty (and it is dirty) script to copy the attributes info from mRNA to CDS to make ENA compliant file, without changing the IDs or rerunning the all manage functional annotation script (specific for functionally annotated maker input).

…ic to maker gff format coming from the functional annotation pipeline
@MartinPippel
Copy link

so for braker annotations that went into NBIS functional annotation pipeline, only a very tiny change needs to be done:

line 87:
$desc2 =~/ID=(\S+);Parent=\S+;(Dbxref=\S+;)_AED=\S+;makerName=\S+;(.*)/; #;Parent=*;(*)maker_name=*;(*)/;
needs to be adapted to:
$desc2 =~/ID=(\S+);Parent=\S+;(Dbxref=\S+;)makerName=\S+;(.*)/; #;Parent=*;(*)maker_name=*;(*)/;

so we might just add a braker and maker argument to that script, (as well as some sanity checks) ?

@Juke34
Copy link
Collaborator

Juke34 commented Jan 9, 2024

Unfortunatly this script does not follow AGAT's requirements to be compatible with parameter file and other peculiarities. Any reason why you want to add that into AGAT? I guess the best would be to put it into GAAS repository (https://github.com/NBISweden/GAAS) which collect various kind of scripts.

@LucileSol
Copy link
Contributor Author

@MartinPippel I agree just an argument is better
@Juke34 we can add it to GAAS sure, I wanted to add it AGAT because it is gff related.

@Juke34
Copy link
Collaborator

Juke34 commented Jan 9, 2024

I can prepare a script more generalized that can be AGAT compatible
see #413

@Juke34 Juke34 closed this Jan 10, 2024
@Juke34 Juke34 deleted the add_ENA_compliant_script branch January 11, 2024 13:37
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.

4 participants