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 IndexOutOfBoundsException in the initial run #895

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kaorahi
Copy link
Contributor

@kaorahi kaorahi commented Sep 1, 2021

  1. Delete the file config.txt.
  2. Start Lizzie 0.7.4.
  3. Open the menu Settings > Engine and click ok button.

Then I get the following error.

Exception in thread "Thread-2" java.lang.IndexOutOfBoundsException: Index 1 out of bounds for length 1
        at java.base/jdk.internal.util.Preconditions.outOfBounds(Preconditions.java:64)
        at java.base/jdk.internal.util.Preconditions.outOfBoundsCheckIndex(Preconditions.java:70)
        at java.base/jdk.internal.util.Preconditions.checkIndex(Preconditions.java:248)
        at java.base/java.util.Objects.checkIndex(Objects.java:372)
        at java.base/java.util.ArrayList.get(ArrayList.java:459)
        at featurecat.lizzie.analysis.EngineManager.updateEngine(EngineManager.java:231)

kaorahi pushed a commit to kaorahi/lizzie that referenced this pull request Sep 1, 2021
@kaorahi kaorahi mentioned this pull request Sep 1, 2021
@hope366
Copy link

hope366 commented Sep 3, 2021

In my environment, I have detected an error that is different from the one you suggested.
Will it be of any help?

  1. Delete the file config.txt.
  2. Start Lizzie 0.7.4.
  3. Change the engine command./Leelaz to./Leelazero/leelaz and click the okbutton.
Exception in thread "Thread-6" java.lang.NullPointerException
        at java.awt.Container.addImpl(Unknown Source)
        at java.awt.Container.add(Unknown Source)
        at javax.swing.JPopupMenu.add(Unknown Source)
        at javax.swing.JMenu.add(Unknown Source)
        at featurecat.lizzie.gui.Menu.updateEngineMenu(Menu.java:1382)
        at featurecat.lizzie.gui.LizzieFrame.updateEngineMenu(LizzieFrame.java:1502)
        at featurecat.lizzie.analysis.EngineManager.lambda$new$2(EngineManager.java:77)
        at java.util.Optional.ifPresent(Unknown Source)
        at featurecat.lizzie.analysis.EngineManager.lambda$new$3(EngineManager.java:47)
        at java.lang.Thread.run(Unknown Source)

@kaorahi
Copy link
Contributor Author

kaorahi commented Sep 6, 2021

@hope366, I cannot reproduce your issue. Does it still occur in this? https://github.com/kaorahi/lizzie/releases/tag/patched074_210826a

If the answer is yes, would you open a new issue and show the concrete, detailed, and accurate steps of your operation?

@hope366
Copy link

hope366 commented Sep 6, 2021

I tried https://github.com/kaorahi/lizzie/releases/tag/patched074_210826a.

I moved the config.txt file to another location and then started Lizzie.vbs.
Naturally, an error occurred, so I ran the engine command from ./leelaz --gtp --lagbuffer 0 --weights lznetwork.gz to katago/katago-opencl gtp -config katago/default_gtp.cfg -model katago/default_model.bin. gz.
Then, the following error was reported.

Engine process ended.
java.io.IOException: パイプを閉じています。
        at java.io.FileOutputStream.writeBytes(Native Method)
        at java.io.FileOutputStream.write(Unknown Source)
        at java.io.BufferedOutputStream.flushBuffer(Unknown Source)
        at java.io.BufferedOutputStream.flush(Unknown Source)
        at java.io.BufferedOutputStream.flush(Unknown Source)
        at featurecat.lizzie.analysis.Leelaz$WriterThread.writeToStream(Leelaz.java:1084)
        at featurecat.lizzie.analysis.Leelaz$WriterThread.run(Leelaz.java:1073)

However, this error was reported only the first time, and after the second time, it started normally without this error.
I intend to do the same operation as the first time, but for some reason the error did not occur after the second time.

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.

2 participants