Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chimei-ruiju.orgへの対応をrelease/v0.1.22にマージ #495

Merged
merged 56 commits into from
Nov 2, 2024

Conversation

YuukiToriyama
Copy link
Owner

変更点

  • chimei-ruiju.orgへの対応 #426
  • experimentalモジュールのParserにおいて、chimei-ruiju.orgのデータを使用できるようになりました。
  • また、chimei-ruiju.orgを使用して住所のパースに成功した場合、その地点の大まかな緯度経度も知ることができるようになったため、住所のパーサーとしてだけではなく、簡易的なジオコーダーとしても使用することができるようになりました。

備考

  • 現時点ではフィーチャフラグexperimentalを有効にすることで使用できる試験的機能ですが、v0.2での安定化を予定しています。

…n-and-service

chimei-ruiju.orgへの対応: `domain`と`service`にchimei-ruiju用の構造体を定義
当面は使用されないので`#[allow(dead_code)]`を付与しておく
…itory

chimei-ruiju.orgへの対応: `repository`にchimei-ruiju用の構造体を定義
`ChimeiRuijuApiService`は引数で受け取るように変更
`ChimeiRuijuApiService`は引数で受け取るように変更
`ChimeiRuijuApiService`は引数で受け取るように変更
`ChimeiRuijuApiService`は引数で受け取るように変更
`ChimeiRuijuApiService`は引数で受け取るように変更
`ChimeiRuijuApiService`は引数で受け取るように変更
…tor-repository

chimei-ruiju.orgへの対応: repository層のリファクタ
…/v0.1.17

chimei-ruiju.orgへの対応: main(v0.1.17)をマージ
…/v0.1.18

chimei-ruiju.orgへの対応: main(v0.1.18)の差分を取り込み
`parser`と`repository`の間を取り持つモジュール
…actor

chimei-ruiju.orgへの対応: `ChimeiRuijuInteractor`を定義
`prefecture_name: &str` -> `prefecture: &jisx0401::Prefecture`
`prefecture_name: &str` -> `prefecture: &jisx0401::Prefecture`
…nteractor

chimei-ruiju.orgへの対応: `ChimeiRujiuInteractor`のメソッドの引数を変更
chimei-ruiju.orgへの対応: main(v0.1.21)との差分を取り込み
…-with-chimeiruiju

chimei-ruiju.orgへの対応: `experimental`モジュールの`Parser`でChimeiRuijuを使ってパースできるようにした
…y-interactor

chimei-ruiju.orgへの対応: `ChimeiRuijuInteractor`の定義場所を移動
clippyの警告を抑制したいため。現時点では使用されていないが将来的には使用される予定。
…れるように設定

現時点では`experimental::Parser`以外からは使用されていないため
…lippy-warnings

chimei-ruiju.org: Clippyの警告を抑制
…i-ruiju-for-wasm

chimei-ruiju.orgへの対応: wasmモジュールにおいてもChimeiRuijuを使用できるようにする
ChimeiRuijuではなくGeoloniaのデータを使うテストになっていたため
…est-code

chimei-ruiju.orgへの対応: `Parser#parse_with_chimeiruiju`に対するテストコードを修正
`Vec<Token>`と共に緯度経度情報`Option<LatLng>`を返すようにした
…de-latlng-data

chimei-ruiju.orgへの対応: 住所のパースに成功した場合に位置情報を`ParsedAddress`に含めるようにする
@YuukiToriyama YuukiToriyama self-assigned this Nov 2, 2024
Copy link
Contributor

github-actions bot commented Nov 2, 2024

Benchmark for 705db6e

Click to view benchmark
Test Base PR %
orthographical_variant_adapter/松ケ崎東池ノ内町/松が崎東池ノ内町 418.8±9.42ns 410.2±10.67ns -2.05%
orthographical_variant_adapter/松ケ崎東池ノ内町/松ガ﨑東池の内町 2.2±0.05µs 2.2±0.05µs 0.00%
orthographical_variant_adapter/松ケ崎東池ノ内町/松ケ﨑東池の内町 2.3±0.08µs 2.3±0.08µs 0.00%
orthographical_variant_adapter/松ケ崎東池ノ内町/松ケ﨑東池ノ内町 1481.6±31.87ns 1454.4±47.05ns -1.84%
orthographical_variant_adapter/松ケ崎東池ノ内町/松ヶ崎東池ノ内町 278.1±9.50ns 276.8±9.42ns -0.47%

@YuukiToriyama YuukiToriyama marked this pull request as ready for review November 2, 2024 12:31
@YuukiToriyama YuukiToriyama merged commit 5f88631 into release/v0.1.22 Nov 2, 2024
7 checks passed
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