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 rank to the caprieval module output #212

Closed
amjjbonvin opened this issue Dec 20, 2021 · 4 comments · Fixed by #213
Closed

Add rank to the caprieval module output #212

amjjbonvin opened this issue Dec 20, 2021 · 4 comments · Fixed by #213
Assignees
Labels
enhancement Enhancing an existing feature of adding a new one

Comments

@amjjbonvin
Copy link
Member

The current capri.tsv generated output file would benefit from adding a rank column, either as first or second column.

@amjjbonvin amjjbonvin added the enhancement Enhancing an existing feature of adding a new one label Dec 20, 2021
@rvhonorato
Copy link
Member

What should the ranking be based on, the same value that is being used to do the sorting or should it be another parameter?

@amjjbonvin
Copy link
Member Author

amjjbonvin commented Dec 20, 2021 via email

@rvhonorato
Copy link
Member

So the rank should always be the score? Probably also good to have a something there to handle situations in which the score should be ranked descending/ascending.

@rvhonorato rvhonorato changed the title Add rank to the caprieval module output Add rank to the caprieval module output Dec 20, 2021
@amjjbonvin
Copy link
Member Author

amjjbonvin commented Dec 20, 2021 via email

@joaomcteixeira joaomcteixeira added this to To do in Enhancements via automation Dec 20, 2021
@joaomcteixeira joaomcteixeira added this to the v3.0.0 stable release milestone Dec 20, 2021
Enhancements automation moved this from To do to Done Dec 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancing an existing feature of adding a new one
Projects
Enhancements
  
Done
Development

Successfully merging a pull request may close this issue.

3 participants