feat: Epiplexity Demo 2 - Trajectory Ordering Matters#11
Conversation
|
You have run out of free Bugbot PR reviews for this billing cycle. This will reset on March 7. To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5ae1597e5d
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| return {a: (confidence if a == pred else spread) for a in actions} | ||
|
|
||
| def _chat_api(self, system: str, prompt: str) -> str: | ||
| provider = self.config.get("provider", "openai") |
There was a problem hiding this comment.
Respect --mode when selecting chat provider
LLMAdapter._chat_api pulls provider only from config (openai by default) and does not use self.mode, even though the CLI exposes --mode openai|ollama and passes that into the adapter. This makes the documented --mode ollama flow fail unless provider=ollama is redundantly set in config, because the code falls back to OpenAI behavior and throws Missing api_key in config for non-ollama provider before any Ollama request is attempted.
Useful? React with 👍 / 👎.
Code Review: PR #11 - Ordering Matters (Trajectory Analysis)What Looks Good
Issues Found
Test ResultsAll thresholds met:
Suggestions
ApprovalApproved. Correctly validates Paradox 2: information is NOT order-independent. Same trajectories yield different extractable structure based on presentation order. |
5ae1597 to
04c3202
Compare
Epiplexity Demo 2: Ordering Matters
Validates Paradox 2 from From Entropy to Epiplexity (Finzi et al., 2026): arXiv:2601.03220
What This Does
Results
Validates Paradox 2: Information is NOT order-independent. Same data, different orderings → different extractable structure.
Files
demos/epiplexity-02-ordering-matters/trajectory_generator.nlogo- NetLogo modeldemos/epiplexity-02-ordering-matters/trajectory_analysis.py- Python analysis harnessdemos/epiplexity-02-ordering-matters/results/- CSV and plotsdemos/epiplexity-02-ordering-matters/tests/- Unit tests with offline mock modeHow to Run
trajectory_generator.nlogo→ generatesdata/trajectory-raw.txtpython3 trajectory_analysis.py --mode mockresults/for output CSV and plots