Repository navigation
Try page: open on NLPFix's NLP analyzer, and offer every text - #24
Merged
Merged
Conversation
Corporate comes off the try page, and NLP from NLPFix (nlpfix-analyzers/nlp, until now listed as "Entities") takes its place at the top, so the page opens on it. Seven analyzers, 145 files, 0.65 MB; all seven run clean. It is called NLP now, as its folder is, rather than a name made up for it. An analyzer can ship several texts to run on -- corporate has four, in two folders. The Text header now has a picker listing every file under the analyzer's input/, by path, starting on the one the build chose; choosing one replaces the box, and Reset puts the chosen text back. With one text there is nothing to choose and the picker does not appear -- which, for the seven shipped now, is every one of them. Which files are texts, and which to start on, is src/try/inputs.ts, pure and tested. TRY-PAGE.md says what changed, and no longer claims the shipped set exercises both pass-numbering edge cases: corporate was the one with a stub. 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.
As asked:
input/, a picker in the Text header lists them by path. Choosing one replaces the box; Reset restores the chosen text. With one text the picker stays hidden.Note: none of the seven shipped now has more than one text — Corporate was the one with several (four). So the picker is built and tested but will not appear until an analyzer with several texts is on the list (
parse-en-ushas 3,tutorial-13-a2).src/try/inputs.ts(pure) decides which files are texts and which to start on, with tests.docs/TRY-PAGE.mdupdated, including a claim that no longer held (the shipped set no longer contains astubpass).🤖 Generated with Claude Code