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

Process TEStribute results #37

Closed
uniqueg opened this issue Oct 11, 2019 · 1 comment
Closed

Process TEStribute results #37

uniqueg opened this issue Oct 11, 2019 · 1 comment
Labels
status: on hold Won't be fixed for now

Comments

@uniqueg
Copy link
Member

uniqueg commented Oct 11, 2019

Is your feature request related to a problem? Please describe.

After integrating TEStribute (#16), its results need to be processed. In particular, the resulting list of ranked services should be looped over and the incoming TES request modified such that it uses the recommended DRS object IDs for input files. It is open for discussion how this is to be handled for output files (probably not at all, for the moment). The TES request should then be forwarded to the recommended TES instance.

Describe the solution you'd like

DRS object IDs should be replaced according to TEStributes recommendations in the original TES request and the request should then be forwarded to the recommended TES instance. If any of the services is unavailable, the next combination should be tried.

Describe alternatives you've considered

N/A

Additional context

Depends on #16

Note that TEStribute currently works slightly beyond the GA4GH specs, i.e., it makes an assumption of data repository services that is not warranted by their specification and it amends a property in a model of one endpoint and adds another required endpoint to the TES specs, thus making it depend on specifically tuned TES implementations for now. For this reason, the use of the task distribution logic middleware must be optional (set a switch and necessary config parameters in the config file.

@uniqueg
Copy link
Member Author

uniqueg commented Oct 19, 2023

This will not be addressed. Alternative task distribution logics are now available and will be refactored as plugins (#160).

@uniqueg uniqueg closed this as completed Oct 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: on hold Won't be fixed for now
Projects
None yet
Development

No branches or pull requests

1 participant