Skip to content

Conversation

@mberr
Copy link
Member

@mberr mberr commented Jun 15, 2020

No description provided.

@mberr mberr requested a review from cthoyt June 15, 2020 11:21
@cthoyt
Copy link
Member

cthoyt commented Jun 15, 2020

Ahh its not so easy since there are groupby operations and all of the later scripts rely on the hits@10 being the only column that isn't getting grouped

@mberr
Copy link
Member Author

mberr commented Jun 15, 2020

Hm, so we would need to re-run the collation every time we want to have the plots for a new metric?

@cthoyt
Copy link
Member

cthoyt commented Jun 15, 2020

No, we just need to generalize the pipeline just a bit

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.

3 participants