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

Duplicate Documentation #41

Open
moschmdt opened this issue Sep 13, 2024 · 3 comments
Open

Duplicate Documentation #41

moschmdt opened this issue Sep 13, 2024 · 3 comments

Comments

@moschmdt
Copy link
Contributor

I noticed that there are three (or more?) versions of very similar documentation:

  1. Soar debugger help command for the explain and chunk commands. I assume this must be located either in the Soar CLI or Soar Java Debugger code?
  2. Usage instructions in chapter procedural knowledge learning
  3. Soar CLI reference for explain and chunk commands on the website.

Points 1. and 3. probably apply to all CLI reference commands.

@garfieldnate
Copy link
Collaborator

1: yes, it's located directly in the Soar code.

2: I think we can safely replace that with a link to the reference page. I don't know how that will look when we generate the manual from the website, though.

There is also a number 4! The wiki: https://github.com/SoarGroup/Soar/wiki. We actually download from there to write the manual sections currently: https://github.com/SoarGroup/Release-Support/tree/master/ManualSource/wikicmd.

The documentation website is essentially a GitHub wiki, so I want to get rid of 4 and replace it with 3.

In turn I would also like to unify 1 and 3, with the website being the canonical version. I'm thinking of some system where we can format the CLI references properly on the website, and then we would periodically run a script in the Soar repo to pull the latest text, format it as needed, and place it at the correct locations in the code. Would also be nice to print something like "view on the website at ..." when running in Soar, as the long texts can be hard to follow in the terminal.

@moschmdt
Copy link
Contributor Author

2: I think we can safely replace that with a link to the reference page. I don't know how that will look when we generate the manual from the website, though.

I will have a look at it and add this to #40. I think keeping the image in the section or adding the image to the reference page is reasonable?

In turn I would also like to unify 1 and 3, with ...

That sounds reasonable, but is the full text even necessary in the CLI or is a reference to the canonical version sufficient? As you already mentioned, the web version is probably easier to read, so in my opinion a link/ reference would be sufficient. In general, this sounds like the man pages project in unix systems - maybe that or a cpp CLI library is a feasible approach?

@garfieldnate
Copy link
Collaborator

I think keeping the image in the section or adding the image to the reference page is reasonable?

I'm not sure what image you're referring to, but if an image works well in the reference section, then yes, I think moving it over and linking to it makes sense.

is the full text even necessary in the CLI or is a reference to the canonical version sufficient?

I do think having the full text available there is important. Many users use Windows, so a Unix man page would not suffice, and Soar is also used in contexts where general internet access might not be available (think defense industry). Also, the website doesn't have versioning yet, so its contents are always newer than whatever version of Soar was last released, which could be confusing for users unless we start adding notes about what Soar version a particular option was added. I just don't want to make waves here. The formatting or where exactly in the distribution we store the data could definitely be iterated on, though.

moschmdt added a commit to moschmdt/documentation that referenced this issue Sep 16, 2024
Some parts of the usage section in the procedural knowledge learning chapter of the Soar manual were not included in the cli reference, thus they were moved.

Discussion see SoarGroup#41
garfieldnate pushed a commit that referenced this issue Sep 17, 2024
Some parts of the usage section in the procedural knowledge learning chapter of the Soar manual were not included in the cli reference, thus they were moved.

Discussion see #41
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

No branches or pull requests

2 participants