Skip to content

feat(ui): phase 1 quick/advanced UI, improved numeric parsing, and offline bundle tooling#32

Merged
rad1092 merged 1 commit into
mainfrom
codex/review-executable-rate-and-analysis-features-jndyb1
Feb 15, 2026
Merged

feat(ui): phase 1 quick/advanced UI, improved numeric parsing, and offline bundle tooling#32
rad1092 merged 1 commit into
mainfrom
codex/review-executable-rate-and-analysis-features-jndyb1

Conversation

@rad1092
Copy link
Copy Markdown
Owner

@rad1092 rad1092 commented Feb 15, 2026

Motivation

  • Make the UI approachable for first-time users by exposing a minimal "빠른 시작" path and hiding advanced controls by default, as Phase 1 of the UI simplification plan.
  • Increase robustness of numeric detection to handle common human-entered formats (commas, currency symbols, percent signs, and parentheses for negatives).
  • Add tooling and documentation to support reproducible offline bundling and online resource collection for future phases and deployments.

Description

  • Implemented a simplified Phase 1 web UI with a quick/advanced mode split and unified status feedback in bitnet_tools/ui/index.html, bitnet_tools/ui/app.js, and bitnet_tools/ui/styles.css, with advanced panels hidden by default.
  • Hardened numeric parsing in bitnet_tools.analysis._to_float to normalize/remove commas, currency symbols, percent signs and interpret parentheses as negative values.
  • Added offline/online support artifacts and scripts including scripts/prepare_online_bundle.sh, scripts/collect_online_assets.py, resources/online_sources.json, and documentation files NEXT_STEPS_UI_AND_BACKLOG.md, BASELINE_PLAN.md, ONLINE_EXECUTION_LOG.md, ONLINE_RESOURCE_COLLECTION.md, and REVIEW_REPORT.md.
  • Added CSV fixtures and expanded tests in tests/test_analysis.py to validate the new numeric parsing and baseline summaries, and included UI smoke artifacts captured during validation.

Testing

  • Ran pytest -q and all tests passed (26 passed).

Codex Task

@rad1092 rad1092 merged commit bc59a36 into main Feb 15, 2026
4 checks passed
@rad1092 rad1092 deleted the codex/review-executable-rate-and-analysis-features-jndyb1 branch February 15, 2026 02:58
Copy link
Copy Markdown

@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: 61dd77bb62

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

Comment thread bitnet_tools/analysis.py
except ValueError:
return None

return -parsed if negative_by_parentheses else parsed
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Avoid double-negating parenthesized values

When _to_float sees parentheses it always negates the parsed value, so an input like (-99.1) is normalized to -99.1 and then flipped to +99.1. This silently corrupts numeric summaries for datasets that already include an explicit minus sign inside parentheses (a common export quirk), because those rows are now treated as positive instead of negative.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant