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

Facets Refits -> Tempo Re-runs #735

Open
arichards2564 opened this issue Jan 9, 2020 · 2 comments
Open

Facets Refits -> Tempo Re-runs #735

arichards2564 opened this issue Jan 9, 2020 · 2 comments
Assignees
Labels
backburner probably won't address in a near future enhancement New feature or request

Comments

@arichards2564
Copy link
Collaborator

An idea for improvement, perhaps in the next phase of Tempo: There are some Tempo results that depend on FACETS (e.g. ccf, LOHHLA, etc). When we do re-fits, these results become incorrect. It would be nice to have a way to re-run only the Tempo results that depend of FACETS, once we've done the re-fits.

@gongyixiao
Copy link
Collaborator

gongyixiao commented Jan 9, 2020

In TEMPO, the -resume function depending on the ID of the file, which includes size, timestamp, location, etc.

If manual curation was done for a certain file, then it can't be used as a valid resume caching, meaning the pipeline won't just skip DoFacets and use the curated result to do other works after DoFacets.

To achieve this, Facets results need to be accepted as an input for TEMPO, and also let TEMPO can start from after Facets.

This is definitely something we need to think about @md09

@gongyixiao gongyixiao added enhancement New feature or request help wanted Extra attention is needed labels Mar 9, 2020
@anoronh4 anoronh4 self-assigned this Mar 18, 2020
@gongyixiao gongyixiao linked a pull request Mar 24, 2020 that will close this issue
@gongyixiao gongyixiao removed a link to a pull request Mar 25, 2020
@gongyixiao
Copy link
Collaborator

Related mskcc/facets-suite#35

@gongyixiao gongyixiao added backburner probably won't address in a near future and removed help wanted Extra attention is needed labels Apr 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backburner probably won't address in a near future enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants