Skip to content

Conversation

@mgermain
Copy link

@mgermain mgermain commented Sep 7, 2021

Minimal edit to fix #653, I also tried to stick to the recommendation in the issue.

There are much better ways to do this. but that would require a deeper dive into the code.

What do you suggest?

@bouthilx
Copy link
Member

bouthilx commented Sep 8, 2021

Good enough! We can optimize this latter on if needed. Could you please do it as well for delete_experiments and add a test for each? Tests would be here. You can simply verify if the correct number of trials is printed using capsys (doc here).

Co-authored-by: Xavier Bouthillier <[email protected]>
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.

Add match count to user prompt in DB commands

2 participants