Skip to content

Add a cost estimate for Terra#113

Merged
aofarrel merged 1 commit intomasterfrom
feature/alignerWDLcosts
Feb 3, 2022
Merged

Add a cost estimate for Terra#113
aofarrel merged 1 commit intomasterfrom
feature/alignerWDLcosts

Conversation

@aofarrel
Copy link
Collaborator

@aofarrel aofarrel commented Dec 5, 2020

One of the biggest things our users on BDC ask about are costs. Right now we do not have a reliable way of predicting them -- Brian Hannafious made a cost estimator notebook, and while it is quite accurate when it gets good data from fiss, but I discovered that sometimes fiss can give wildly incorrect times that don't match the logs, so this is the result of the raw data I can glean from Terra's UI and isn't all that granular.

@aofarrel
Copy link
Collaborator Author

aofarrel commented Dec 7, 2020

It might be more appropriate to put this information in the Dockstore metadata, but I'm a little confused as to how that's set up on this repo -- the metadata on Dockstore clearly refers to just the aligner but .Dockstore.yml in this repo doesn't have that information inside of it, so I don't know where the Dockstore-specific information is being pulled from.

Also, this does not contain the changes of #112 so the checks showing as failed are expected.

@wshands
Copy link
Member

wshands commented Dec 7, 2020

There is information about adding metadata to workflows here:
https://docs.dockstore.org/en/develop/advanced-topics/best-practices/best-practices.html
https://docs.dockstore.org/en/develop/advanced-topics/best-practices/wdl-best-practices.html

https://docs.dockstore.org/en/develop/getting-started/getting-started-with-cwl.html?highlight=readme
https://docs.dockstore.org/en/develop/getting-started/getting-started-with-wdl.html?highlight=readme
Dockstore uses the authorship information and description from the descriptor file to populate metadata for tools.
Note
If no description is defined in the descriptor file, the README from the corresponding Git repository is used.

@aofarrel aofarrel merged commit 209e5b1 into master Feb 3, 2022
@aofarrel aofarrel deleted the feature/alignerWDLcosts branch February 3, 2022 00:19
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