-
Notifications
You must be signed in to change notification settings - Fork 582
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
Error values remain at coordinate after rotating components #1939
Comments
I am unable to reproduce this problem. I have tried it with the 3.8.0-all.jar file and with our current head of main. I tried it with Java 16 and Java 21. I used the TTL and moved it all over the area, specifically checking the 210,130 location shown above. The only errors I see are the outputs from the NAND gates in the TTL (which are correct as no inputs are provided). I do not see the errors on the inputs as shown in your picture. I also tried the clock as shown above and did not see any problem. Could you be more specific about what you mean by "at startup"?
|
The coordinates at which the error appears will change each time I launch using the command below, so we must look for it. As shown in the video below (0:26, 0:36-), an error occurs at some coordinates after launching Therefore, the answers to your questions are below.
logisim_issue_1939.mov |
@tomoish: Thank you for the answers. After more experimenting, I can now conclude that this is not random. I can reproduce the behavior. Open a new document. Place a 7400 TTL on the canvas. Drag a wire up 2 spaces from the left top red colored output. Drag the bottom of the wire up 1 space. This results in an unconnected wire pointing to the location of the red output, so we can find it. Now select the TTL and rotate it south with down arrow key. Now drag the TTL so the upper left input is located at the space pointed to by the wire. It will show as red. The workaround is to reset the simulation. The problem seems to be in the disconnecting of the rotated item from the old location. I'm not very familiar with that part of the code. Perhaps @BFH-ktt1 has thoughts. I will change the name of this issue to reflect the cause. |
I have tried more experiments. The error values remain when rotating a component. But they do not remain when dragging or deleting a component. So those two actions must do something to update the value at the coordinate. If so, we should be able to fix this by applying the same logic on the rotate action. |
Describe the bug
An error occurs with the initial values of coordinates upon startup.
The coordinates where the error occurs are random at startup. For example, as shown in Figure 1, the error occurs at coordinates (250, 230).
Fig 1. Error occurs at (250, 230).
Fig 2. The initial values are errors for many coordinates ((180, 140), (180, 200), (240, 160), (240, 220)).
Reproduction Steps
The coordinates where the initial values cause an error are random, but they generally occur within the range of (200, 200) to (300, 300).
The issue is more easily identified when using components with a large number of pins, such as TTL.
I have confirmed that errors occur at coordinates (180, 140), (180, 200), (210, 180), (240, 160), (240, 220), (250, 230), (270, 230), (280, 230) and (290, 260).
logisim-evolution version
logisim-evolution-3.8.0-all.jar
Java version
java 16.0.1 2021-04-20
Java(TM) SE Runtime Environment (build 16.0.1+9-24)
Java HotSpot(TM) 64-Bit Server VM (build 16.0.1+9-24, mixed mode, sharing)
The text was updated successfully, but these errors were encountered: