WIP: Loop fixes and other parametric setup for dependencies. #58
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
Also added debugging information and flags per plan tree dependencies.
|
@ukuter's branch. I think it is intended to merged back into master, but am unsure if it is ready. There are some debugging prints still in this branch, for instance. |
Xach has added SHOP3 to the Quicklisp distribution, so it is now may be installed simply by using `quickload`.
Made a mistake in the Markdown
The (f)lex command was incorrect. Syntactically it was wrong, the pathname was wrong, *and* errors were masked by using `;` instead of `&&`.
@ko56 (Kostas Oikonomou) has kindly done substantial edits to the documentation, notably fixing up a lot of the typography, and also added a simple example for novices.
Addresses issue #75
`shop-trace-info` wasn't exported. Fix some stylistic issues in the trace code.
* Manual fixes from Kostas (@ko56) * Minor edits by me (@rpgoldman) * Fix some crossreferences not handled properly in the translation from Microsoft Word. * Make citations into cross-references. * Swap chapters per Kostas's suggestion: Put "General Notes on Shop3" ahead of "Internal Technical Information."
Check build more stringently and fix some problems with test evaluation.
Also added debugging information and flags per plan tree dependencies.
Also added the capability to interleave :FOR loops with a :COND in them. It's limited to only this kind of nesting right now and needs to be testing more before a merge.
|
@matthewdehaven How would you feel about rebasing this on |
|
Please see #142 which resolves the conflicts. Also, if it takes a while to get this merged, let's regularly rebase it on the current |
|
Should we kill this MR, now that we have the rebased one: #142 ? |
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.
Also added debugging information and flags per plan tree dependencies.