make ipa-auditが旧IPA関数名の参照でコンパイルできない問題を修正 - #1625
Merged
Merged
Conversation
tools/ipa_audit.rs は stationapi/src/domain/ipa.rs を include! して単独の
rustc でビルドするため cargo のターゲットに含まれず、ipa.rs が TTS セグメント
方式へリファクタされた際の関数リネームに追従できていなかった。
- romanized_name_to_ipa() は廃止済みのため station_name_to_ipa("", Some(name))
に置換する。カタカナ引数を空にすることでカタカナへのフォールバックを抑止し、
ローマ字表記だけで IPA を解決できるかという元の判定意図を保つ。
- word_to_ipa() は後継の word_to_tts_segments() に置換する。
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Px4SXdeBULB87KqD1TzKfy
Contributor
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughローマ字名のIPA解決判定で、カタカナ名へのフォールバックを停止しました。トークン単位の判定には ChangesIPA解決判定
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
Suggested labels: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
10 tasks
10 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
make ipa-auditが旧 IPA 関数名を参照したままでコンパイルエラーになり実行できなくなっていたため、現行 API に追従させる。変更の種類
変更内容
tools/ipa_audit.rsのromanized_name_to_ipa()をstation_name_to_ipa("", Some(name_roman))に置換。カタカナ引数を空にすることでカタカナへのフォールバックを抑止し、「ローマ字表記だけで IPA を解決できるか」という元の判定意図を保つ。word_to_ipa()を後継のword_to_tts_segments()に置換。tools/ipa_audit.rsはstationapi/src/domain/ipa.rsをinclude!して単独のrustcでビルドする構成のため cargo のターゲットに含まれず、ipa.rsが TTS セグメント方式へリファクタされた際の関数リネーム(romanized_name_to_ipa廃止 /word_to_ipa→word_to_tts_segments)に追従できていなかった。CSV のカラム位置(line_name_r=5、station_name_r=4、type_name_r=4)は現行ヘッダと一致していたため変更していない。修正後の
make ipa-audit実行結果(読み取り専用レポート、これまで通り検証は失敗させない):テスト
make ipa-auditがビルド・実行され、3 データセット分のレポートを出力することを確認。cargo fmt --all -- --checkが通ることcargo clippy -- -D warningsが通ることcargo test(SQLX_OFFLINE=true)が通ること関連Issue
スクリーンショット(任意)
Summary by CodeRabbit