Decide how to apply qe-fig-003 (no matplotlib titles) across the lectures

オープン
  • 何か問題がありますか？
#398 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

難易度
5/5
見積もり時間
1週間以上
初心者へのやさしさ
25/100
issue の種類
機能追加
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
matplotlib, python
領域
documentation

調査の方向性

Read qe-fig-003 in the action-style-guide and review the counts by lecture in the issue. Compare the two proposed policies, including how they handle panel descriptions and single-panel figures. Done means the project has decided whether to add a multi-panel exception or enforce the existing rule across the listed lectures.

索引モデルが issue の本文から書いたものです。

説明

style-guide

The QuantEcon style guide's rule qe-fig-003 says figures should not embed titles with ax.set_title() or fig.suptitle() outside exercises and solutions; the mystnb caption carries the title instead. The review of #396 found four figures in the two new lectures that use panel titles, but the pattern is repo-wide, so it needs a decision before any changes.

Scope

Counted on the oct_seoul branch (main plus the two new lectures). Each count is the number of lines calling set_title or suptitle outside exercise and solution blocks, so a loop that titles several panels counts once.

  • 33 of 75 lectures, 148 lines.
  • Most hits title the panels of multi-panel figures, where one caption can't label every panel. The counts don't yet separate single-panel figures from multi-panel ones.

The four figures from #396:

  • gradual_trade_agreements: fig-gta-frontier (a title per parameter case) and fig-gta-smoothing (three panel titles)
  • organizational_equilibrium: fig-oe-growth-paths (four panel titles) and fig-oe-chi-G (two panel titles)
Counts by lecture
Lecture Lines
induced_priors 15
additive_functionals 11
tax_smoothing_2 9
dovis_accounting_mf 9
BCG_incomplete_mkts 9
policy_counterfactuals_nk 7
policy_counterfactuals 7
tsyrennikov_2013 6
hs_invertibility_example 6
smoothing_tax 5
five_preferences 5
cattle_cycles 5
tax_smoothing_3 4
supply_demand_var 4
smoothing 4
rosen_schooling_model 4
markov_jump_lq 4
black_litterman 4
policy_regimes 3
organizational_equilibrium 3
muth_kalman 3
dyn_stack 3
policy_invertibility 2
gradual_trade_agreements 2
gorman_heterogeneous_households 2
chang_ramsey 2
calvo 2
BCG_complete_mkts 2
atkeson_1991 2
opt_tax_recur 1
estspec 1
discrete_dp 1
cagan_rational_expectations 1
Options
  1. Enforce the rule as written. Move each panel's description into its y-axis label, and use a legend title or annotation where panels differ by parameter case. Sweep all 33 lectures.
  2. Allow panel titles in multi-panel figures. This needs an exception added to qe-fig-003 in action-style-guide. Only single-panel figures would then need changing, and the style checker would stop flagging multi-panel figures.

Option 2 matches current practice and is much smaller. Option 1 keeps the rule simple but touches nearly half the lectures.

主要言語
TeX
スター
38
フォーク
27
平均マージ
7時間 38分
マージ済み PR（30日）
9

環境構築

このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

QuantEcon/lecture-python-advanced.myst のほかの issue

QuantEcon/lecture-python-advanced.myst の issue をすべて見る

似ている issue

Documentation の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。