Skip to content

feat(factorio-cycle-calculator): Introduce the Project#114

Merged
hcoona merged 5 commits intomainfrom
dev/shuaizhang/factorio-calculator
Feb 12, 2026
Merged

feat(factorio-cycle-calculator): Introduce the Project#114
hcoona merged 5 commits intomainfrom
dev/shuaizhang/factorio-calculator

Conversation

@hcoona
Copy link
Owner

@hcoona hcoona commented Feb 12, 2026

No description provided.

Copilot AI review requested due to automatic review settings February 12, 2026 07:11
@hcoona hcoona enabled auto-merge (rebase) February 12, 2026 07:11
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Introduces a new private UV workspace member, factorio-cycle-calculator, including a Streamlit prototype app that models Factorio’s advanced oil processing chain and solves steady-state machine counts via OR-Tools.

Changes:

  • Adds the new src/private/app/factorio-cycle-calculator project (pyproject + README + Streamlit app).
  • Updates workspace configuration (pyproject.toml) and dependency lockfile (uv.lock) to include the new member and its dependencies (notably OR-Tools).
  • Adds internal .AGENT analysis docs and a helper script to validate Factorio icon paths/localization.

Reviewed changes

Copilot reviewed 8 out of 9 changed files in this pull request and generated 8 comments.

Show a summary per file
File Description
uv.lock Adds the new workspace member and resolves new dependencies (e.g., ortools, absl-py, immutabledict), plus protobuf bump.
pyproject.toml Registers the new project as a UV workspace member and includes it in pyrefly scanning.
src/private/app/factorio-cycle-calculator/pyproject.toml Defines the new project metadata and dependencies.
src/private/app/factorio-cycle-calculator/app.py Implements the Streamlit UI + parsing + OR-Tools solve flow for the oil chain example.
src/private/app/factorio-cycle-calculator/README.md Documents project intent and how to run the Streamlit prototype.
src/private/app/factorio-cycle-calculator/.AGENT/scripts/check_icons_and_locale.py Adds a helper script to validate icons/localization via jq + locale parsing.
src/private/app/factorio-cycle-calculator/.AGENT/*.md Adds internal research/notes describing data sources, modeling assumptions, and tooling.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0e23c8b62c

ℹ️ 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".

@hcoona hcoona merged commit cc0d22b into main Feb 12, 2026
17 checks passed
@hcoona hcoona deleted the dev/shuaizhang/factorio-calculator branch February 12, 2026 18:07
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

Successfully merging this pull request may close these issues.

1 participant