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 split-output option #209

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ozguzMete
Copy link
Contributor

@ozguzMete ozguzMete commented Aug 2, 2022

This option creates two additional files for failed and successful annotations.

@ozguzMete
Copy link
Contributor Author

option behavior added to the GET request.

@ozguzMete ozguzMete force-pushed the retry-failed-annotations branch 3 times, most recently from 41e99ad to 94133cf Compare August 6, 2022 09:15
@ozguzMete
Copy link
Contributor Author

dump-failed option is added.

if the output file is x.txt and this option is present, failed annotations will be written additionally to a file named x.txt.FAILED.

@ozguzMete ozguzMete force-pushed the retry-failed-annotations branch from 94133cf to edda1f1 Compare August 6, 2022 09:48
@ozguzMete
Copy link
Contributor Author

The removed test file is old and fails ci/circleci:build.
#200 is covering this and much more

@ozguzMete ozguzMete force-pushed the retry-failed-annotations branch 6 times, most recently from 8bb978d to fe56fcb Compare August 16, 2022 07:55
@ozguzMete ozguzMete changed the title Add retry-failed option Add dump-failed option Aug 16, 2022
@@ -65,6 +65,7 @@ private static Options getOptions(String[] args)
.addOption("i", "isoform-override", true, "Isoform Overrides (mskcc or uniprot)")
.addOption("e", "error-report-location", true, "Error report filename (including path)")
.addOption("r", "replace-symbol-entrez", false, "Replace gene symbols and entrez id with what is provided by annotator" )
.addOption("s", "dump-failed", false, "Write failed annotations into an additional file" )
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
.addOption("s", "dump-failed", false, "Write failed annotations into an additional file" )
.addOption("s", "dump-failed", false, "Write failed annotations into an additional file <output-filename>.FAILED" )

@ozguzMete ozguzMete force-pushed the retry-failed-annotations branch 2 times, most recently from ab753d8 to 6aacd60 Compare August 16, 2022 20:42
@ozguzMete ozguzMete changed the title Add dump-failed option Add split-output option Aug 16, 2022
@ozguzMete
Copy link
Contributor Author

I have renamed dump-failed to split-output because we started to produce SUCCESS file.
it is split-output instead of split hence split is a method name

@ozguzMete ozguzMete force-pushed the retry-failed-annotations branch from 6aacd60 to feaee5a Compare August 31, 2022 14:43
@ozguzMete
Copy link
Contributor Author

we might want to keep this on hold until the big file support goes live

@ozguzMete ozguzMete force-pushed the retry-failed-annotations branch 3 times, most recently from bd14959 to 9c18cb5 Compare September 21, 2022 13:48
@ozguzMete ozguzMete force-pushed the retry-failed-annotations branch from 9c18cb5 to 327e277 Compare September 21, 2022 13:56
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.

2 participants