Consolidate everything from the archived transaction-forensics repo - #8
Merged
Merged
Conversation
The predecessor repo (chrbailey/transaction-forensics) is archived and names THIS repo as its successor — but several artifacts never actually made it over. This finishes the consolidation so nothing is lost and the "contains everything" claim is finally true: - herb-nlp/: the unstructured-text NLP pipeline (analyze.py + BERTopic, network, and temporal modules) behind the HERB 37,064-document analysis, with a README. - demo/portfolio/: the 6-lens portfolio dashboard (Overview, CRM Pipeline, BPI Challenge, IDES Compliance, Client Cases, NLP Patterns) over real datasets, plus its pattern_cards.json. Deploys at /portfolio/ on GitHub Pages. - demo/index.html: "Full Analysis ↗" nav link to the portfolio dashboard. - README: documents the 6 lenses and the herb-nlp pipeline. Source repo untouched (it stays archived and feeds its existing Vercel site). https://claude.ai/code/session_0194vyNiy4Nn95aSHdA7ENgJ
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
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.
Why
You weren't sure whether you'd accidentally created two repos. Here's the answer: the other repo (
chrbailey/transaction-forensics) is already archived, and its ownARCHIVED.mdexplicitly names this repo as its successor — "contains everything in this repo plus...". So this repo is canonical. But that "contains everything" claim wasn't actually true: several artifacts never made it over. This PR finishes the consolidation so nothing is lost.I did not touch the source repo — it stays archived and keeps feeding its existing Vercel site. This is purely additive: everything unique comes into this repo.
Brought over
herb-nlp/— the unstructured-text NLP pipeline that was only in the archived repo:analyze.py(TF-IDF + KMeans pattern engine over 37,064 HERB documents)bertopic_cluster.py,network_analysis.py,temporal_analysis.pyrequirements.txt+ a README explaining how it relates to the forensic enginedemo/portfolio/— the 6-lens portfolio dashboard (Overview, CRM Pipeline, BPI Challenge, IDES Compliance, Client Cases, NLP Patterns) over real datasets (251,734 BPI POs, 7 IDES compliance violations, 3M+ ERP records, 37,064 HERB docs), plus itspattern_cards.json. It deploys at/portfolio/on GitHub Pages.demo/index.html— a "Full Analysis ↗" nav link from the animated landing page to the portfolio dashboard.README.md— a table documenting the six lenses and theherb-nlp/pipeline.Verification
py_compile.pattern_cards.jsonis valid JSON.Suggested follow-up (your call, not in this PR)
Now that this repo truly contains everything, you could point the existing
transaction-forensics.vercel.appproject at this repo (Vercel → Project → Settings → Git) so your established URL serves the consolidated site. I can't change another repo's Vercel binding from here, but I leftvercel.jsonin place so it works with zero config.https://claude.ai/code/session_0194vyNiy4Nn95aSHdA7ENgJ
Generated by Claude Code