-
Notifications
You must be signed in to change notification settings - Fork 86
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
Fixed lots of bugs and made it so SHRDLU and PLNR can be compiled. #2323
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Now, we build SHRDLU;TS SHRDLU and SHRDLU;TS PLNR from compiled lisp code. This should allow it to run faster and have fewer issues with garbage collection and running out of LIST space. Found lots of bugs going through the compilation effort. Also fixed several issues that were resulting in weird/incorrect responses to standard demo prompts. Below is a list of prompts that I have tested: pick up a big red block. grasp the pyramid find a block that is taller than the one you are holding and put it into the box. what does the box contain? what is the pyramid supported by? how many blocks are not in the box? is the red cube supported? can the table pick up blocks? can a pyramid be supported by a block? can a pyramid support a pyramid? stack up two pyramids. stack up two red blocks. put the green pyramid on the red cube. which cube is sitting on the table? is there a large block behind a pyramid? put a small one onto the green cube that supports a pyramid. put the littlest pyramid on top of it. does the red cube support anything? what color is the block that supports the green pyramid? how many things are on top of green cubes? had you touched any pyramid before you put the green pyramid on the little cube? when did you pick it up? why did you pick it up? why did you clear off that cube? how did you clean off the red cube? how many objects did you touch while you were doing it? put the blue pyramid on the block in the box. is there anything which is bigger than every pyramid but is not as wide as the thing that supports it? thank you. Of course, lots more things are possible. It is still best to do this without answering "Y" to the Type 340 display prompt at the start. There is either an issue with the GRAPHF module or the SLAVE module (or perhaps with the Type 340 simulator) that is causing crashes after several successful commands with graphic output.
eswenson1
force-pushed
the
ejs/shrdlu-20240813
branch
from
August 14, 2024 00:36
dc3344b
to
ef2a198
Compare
larsbrinkhoff
approved these changes
Aug 15, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Now, we build SHRDLU;TS SHRDLU and SHRDLU;TS PLNR from compiled lisp code. This should allow it to run faster and have fewer issues with garbage collection and running out of LIST space.
Found lots of bugs going through the compilation effort. Also fixed several issues that were resulting in weird/incorrect responses to standard demo prompts.
Below is a list of prompts that I have tested:
pick up a big red block.
grasp the pyramid
find a block that is taller than the one you are holding and put it into the box. what does the box contain?
what is the pyramid supported by?
how many blocks are not in the box?
is the red cube supported?
can the table pick up blocks?
can a pyramid be supported by a block?
can a pyramid support a pyramid?
stack up two pyramids.
stack up two red blocks.
put the green pyramid on the red cube.
which cube is sitting on the table?
is there a large block behind a pyramid?
put a small one onto the green cube that supports a pyramid. put the littlest pyramid on top of it.
does the red cube support anything?
what color is the block that supports the green pyramid? how many things are on top of green cubes?
had you touched any pyramid before you put the green pyramid on the little cube? when did you pick it up?
why did you pick it up?
why did you clear off that cube?
how did you clean off the red cube?
how many objects did you touch while you were doing it? put the blue pyramid on the block in the box.
is there anything which is bigger than every pyramid but is not as wide as the thing that supports it? thank you.
Of course, lots more things are possible.
It is still best to do this without answering "Y" to the Type 340 display prompt at the start. There is either an issue with the GRAPHF module or the SLAVE module (or perhaps with the Type 340 simulator) that is causing crashes after several successful commands with graphic output.