Skip to content

Named Expression Section#128

Merged
konn merged 15 commits intodevelopfrom
konn/named-expressions
Mar 24, 2026
Merged

Named Expression Section#128
konn merged 15 commits intodevelopfrom
konn/named-expressions

Conversation

@konn
Copy link
Copy Markdown
Collaborator

@konn konn commented Mar 19, 2026

  • Updated 2.3.1 release notes
  • Adopts most recent
  • Shortens up
  • Named Expression section
  • Updates math

Description

@konn konn requested a review from NY57 March 19, 2026 04:41
Comment thread markdowns/ja/releases/jijmodeling-2.3.1.md Outdated
Comment thread markdowns/ja/releases/jijmodeling-2.3.1.md Outdated
Comment thread markdowns/ja/advanced/named_expr.md Outdated
Comment thread markdowns/ja/advanced/named_expr.md Outdated
konn and others added 8 commits March 19, 2026 15:53
Co-authored-by: NY57 <121103446+NY57@users.noreply.github.com>
# 変更点
- [NamedExprの概要 / 2つの利用方法の提示] -> [式の命名] -> [インスタンスへの保存]
という説明順序になるように構成を見直しました
- 合わせて、`jijmodeling.Problem.named_exprs`,
`jijmodeling.Compiler.get_named_function_id_by_name`
の説明を簡略化し、TIPSとして記載する形に変更
     - 理由: 上記の説明順序の”本筋”から外れている部分であり、TIPSとしてその存在が認知できる程度の説明で十分であると考えたため
- 「インスタンスへの保存」 におけるOMMXインスタンスに保存できる条件について、スカラー式という言葉を使わない形に修正
   - 合わせて、保存できる場合の具体例も列挙する形に変更 
   - 理由: スカラー式という"言葉だけ"では伝わりにくいと考えたため
- 「インスタンスへの保存」 におけるナップサック問題の例について、評価する式を総重量だけに変更
- 理由:
アイテムの重さあたりの価値の寄与を評価する意味とは?という疑問を読者に抱かせないため。元々、このケースを含めたのは”式の配列”の例示をしたかったからだと思われるが、既にOMMXインスタンスに保存できる条件で具体例を示す形に変更したので、削除しても問題ないと考えている

# 注意点
`docs/en/` 側の修正は行っていません。
@konn konn enabled auto-merge (squash) March 23, 2026 07:41
@konn konn disabled auto-merge March 23, 2026 07:56
Copy link
Copy Markdown
Member

@NY57 NY57 left a comment

Choose a reason for hiding this comment

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

LGTM!

@konn konn merged commit e842a7e into develop Mar 24, 2026
6 of 24 checks passed
@konn konn deleted the konn/named-expressions branch March 24, 2026 00:52
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.

2 participants