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

Fix #781 (slow play/undo/redo) #804

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

kaorahi
Copy link
Contributor

@kaorahi kaorahi commented Nov 1, 2020

In Panel UI, the layouts of all the UI parts are rearranged in every play/undo/redo after d2c587c. #781 seems to be fixed if we stop this redundant work. Can anybody test this PR for a week or so and report whether you notice any problem?

@hope366
Copy link

hope366 commented Nov 1, 2020

I'd like to verify the PR, but I can't reproduce #781, probably because of the GPU.would it be difficult to do so without a CPU-only PC?
I tried katago-v1.6.1-cpu-eigen.
The GPU doesn't seem to work, but undo was fast.

@kaorahi kaorahi mentioned this pull request Nov 2, 2020
@kaorahi
Copy link
Contributor Author

kaorahi commented Nov 2, 2020

thx! You don't need to reproduce the issue #781, that may depend on the environments.

I'm worried about unexpected side effects of the PR. If you notice a problem like broken layouts etc., please let me know the steps to reproduce it.

@hope366
Copy link

hope366 commented Nov 2, 2020

In my environment, applying #804 hasn't done anything different I'll continue to monitor it carefully.

By the way, I found a minor issue with the panel UI mode.
Press Alt+W. This will hide the comments panel. You can detach the panel by dragging the mouse, but if you revert the detached panel, the comments panel will remain hidden.
When I restarted lizzie, it was back to normal, so it's not a very serious problem.

@NTUST-MITLAB
Copy link

I will start a live test during today's (and hopefully tomorrow's) Samsung Fire Cup final match, and see how it fairs.

@kaorahi
Copy link
Contributor Author

kaorahi commented Nov 3, 2020

@hope366
You are still in the design mode. Hit Alt+W again to exit.

@kaorahi
Copy link
Contributor Author

kaorahi commented Nov 3, 2020

1b7761d is unacceptable because it prevents the message "Engine is loading...". So I reverted it and pushed another workaround. How about this one? Do you notice any change of the graphics quality?

@hope366
Copy link

hope366 commented Nov 3, 2020

You are still in the design mode. Hit Alt+W again to exit.

Okay, that worked out well.
Alt+W toggles only the comments panel to show/hide, but it's not something that I find particularly inconvenient.

Do you notice any change of the graphics quality?

I couldn't really see the difference.
I'm curious to see where the difference in quality lies.

@NTUST-MITLAB
Copy link

During the live game today the old fix did seem to make it faster but has the side effect listed. I'll try another live game sometime with the new fix. And here is today's live game save file, with lots of branches in the variation tree, and all the best moves embedded, maybe others can see if adding new nodes and undo in these big branches would cause lag...
The 25th Samsung Fire Cup Match 2 - Ke Jei (B) vs Shin Jinseo (W).zip

@kaorahi
Copy link
Contributor Author

kaorahi commented Nov 4, 2020

thx for your reports. Though 28f2c33 accelerates the response of Lizzie remarkably in my environment, I'm afraid of unexpected side effects like degraded graphics. So if you notice nothing, it is a good news for this PR.

kaorahi pushed a commit to kaorahi/lizzie that referenced this pull request Dec 5, 2020
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