Repository navigation
The README says what phase 2 is now - #21
Merged
Merged
Conversation
Three things had gone stale while the work moved:
* phase 2 read "started -- the editor, and running analyzers through a local
run server". It is deployed at studio.visualtext.org/studio/, opens
analyzers from GitHub and commits them back as pull requests.
* the shared package was described as "the pass sequence and knowledge base".
It is seven views now -- output files, parse trees, code, log and values as
well -- and it is released for other pages to install.
* the values a run found, which is the first thing the Output tab shows, were
not mentioned at all. A person reading the quick start would not know the
studio answers "what did the analyzer find" and not only "what did it write".
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
"NLP Studio uses it today, and the admin console's Browse and run is next" -- which has shipped, from a release, so the sentence was describing a plan that had already happened. Named with it are the three rules that made one package worth having: the .kb files nobody edits are never listed, a switched-off pass keeps the number the engine gives it, and a -DEV run's per-pass trees and rule matches are read the same way in both pages. No customer, repository or host is named: the line says "the admin console", as it did before. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or 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
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.
Three stale spots, found by reading the README against what the app actually does:
studio.visualtext.org/studio/(401s for anyone not invited, so it is really there), opens analyzers from GitHub and commits them back as pull requestsThe
-DEVper-pass trees and rule matches went in with #20 and are already described.Also in the package's own README: "NLP Studio uses it today, and the admin console's Browse and run is next" — which has shipped, from a release, so the line described a plan that had already happened. It now names the three rules that made one package worth having: no
.kbfiles in the knowledge base, a switched-off pass keeping the number the engine gives it, and a-DEVrun's per-pass trees and rule matches read the same way in both pages. No customer, repository or host is named — it still says "the admin console", as it did before.🤖 Generated with Claude Code