Quickfix to make a CI test occupy less ram#842
Conversation
Run tests in parallel using max 2 processes.
Bencher Report…with_single_soft_constraint
| - name: Run tests | ||
| run: | | ||
| pytest -v | ||
| pytest -v -n 2 |
There was a problem hiding this comment.
-n auto is specified as the default in pyproject.toml — you're halving the execution speed with this.
There was a problem hiding this comment.
Regardless: If you think about changing this kind of thing, please talk to me. I have put significant thought into the testing and CI setup — there are reasons for these values.
There was a problem hiding this comment.
You can undo this part. I tried out a couple things because the windows ci runner was crashing. We want passing tests for the 1.1.0 release. But the fix has been to limit the max_nodes_rate and max_repetition_rate.
There was a problem hiding this comment.
@joszamama and I decided to just skip the test for now until we have a better understanding of what the problem actually is.
There was a problem hiding this comment.
| pytest -v -n 2 | |
| pytest -v |
There was a problem hiding this comment.
(Sorry, I didn't see this soon enough.)
| gen = fan.generate_solutions( | ||
| max_generations=max_generations, max_repetition_rate=0.0, max_nodes_rate=0.0 | ||
| ) |
|
I added an issue for the broken test in #845. If we find a solution for this, we can then close this issue. Do you believe the |
alex9849
changed the title
2 participants
Quick fix in order to make all CI tests running on Windows.
Set max_repetition_rate and max_nodes_rate to 0 in order to make the test occupy less memory.