Fix end-state (allow searches to finish) #7

Closed
opened 2025-01-24 19:19:22 +01:00 by sacha · 2 comments
Owner

For some reason, long running searches fail to finish. Short running ones are fine. I need to figure out a better way to do this.

For some reason, long running searches fail to finish. Short running ones are fine. I need to figure out a better way to do this.
Author
Owner

for solver.iter != solver.counter {} never works. For some reason the counter never reaches iter, making this a while true. Due to how if track <= int(percentage) {} is setup and all the variables before and after there is never really an end-state with the difference between iter and counter is too big ( why this is too big is another issue, which I hope to solve with #8 ).

I need to rework and rethink this. I will keep the track/percentage, but I will need to rework everything outside of it.

That or track if variable values haven't changed after the last run. But that comes off as a dirtier hack than my ex-wife.

`for solver.iter != solver.counter {}` never works. For some reason the `counter` never reaches `iter`, making this a while true. Due to how `if track <= int(percentage) {}` is setup and all the variables before and after there is never really an end-state with the difference between `iter` and `counter` is too big ( why this is too big is another issue, which I hope to solve with #8 ). I need to rework and rethink this. I will keep the track/percentage, but I will need to rework everything outside of it. That or track if variable values haven't changed after the last run. But that comes off as a dirtier hack than my ex-wife.
Author
Owner

This commit should fix the end-state issue. But it seems #6 ruined the estimation for long running estimations (rates seem to be off, halved).

This commit should fix the end-state issue. But it seems #6 ruined the estimation for long running estimations (rates seem to be off, halved).
sacha closed this issue 2025-01-25 17:33:19 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: golang/sudoku-funpark#7
No description provided.