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

NL_O5_OddsSanrenInfoとNL_O6_OddsSanrentanInfoのレコード数が正しくないことがある #1

Closed
urasandesu opened this issue Apr 18, 2024 · 3 comments

Comments

@urasandesu
Copy link
Owner

メールにて以下のご報告:

杉浦様

突然の連絡失礼いたします。
JRA-VANデータラボより JVLinkToSQLite をダウンロードさせていただき試用させていただいているものです。一部の動作に関しまして気づいた点がございましたので報告させていただきます。

NL_O5_OddsSanrenInfoとNL_O6_OddsSanrentanInfo にてレコード数が正しく記録されない場合があるようです。
発生条件確認してみますと,出走頭数が18頭以下の場合に発生しているようです。(出走18頭の場合は正しく出力されています。)
憶測ですが,18頭未満の出走の場合にJVDataフォーマットのレコードに含まれるスペースの部分でSQLへの変換が終了してしまっているのではないかと思っています。

C#,githubともに素人のためプルリクエスト等できず恐縮ですが,お手すきの際に修正いただけますと幸いです。
以上ご報告まで。

@urasandesu
Copy link
Owner Author

urasandesu commented Apr 18, 2024

JVData の仕様を勘違いしていた。

NL_O5_OddsSanrenInfo の元になっている3連複オッズは、組番の昇順(01-02-03~16-17-18)でデータが提供されるが、18 頭より頭数が少ないレースでは、昇順にした際に不要となる組を除外するのではなく、空白として提供される。例えば、11 頭立てのレースでは、以下のようなデータが提供されることとなる:

# オッズ
1 010203 014030
2 010204 001521
3 010205 010333
4 010206 004875
5 010207 009287
6 010208 049869
7 010209 016563
8 010210 051550
9 010211 039895
10
11
12
13
14
15
16
17 010304 002204
18 010305 013298
19 010306 007236
20 010307 012887
21 010308 048808
22 010309 011470
23 010310 058820
24 010311 041333

現状の処理では、10 個目の組でデータが存在しないと判断し、以降のデータを切り捨ててしまっていた。そのため、報告にあるような振る舞いになっていた。

urasandesu added a commit that referenced this issue Apr 18, 2024
* JVPastDataServiceOperatorTest.cs にパラメータ間違いの問題があったため合わせて修正。
* 子テーブルのレコードの絞り込みについて、「空を検出したら残りを切り捨てる」ではなく「空のものをスキップする」処理に変更。
@urasandesu
Copy link
Owner Author

同様の問題が以下の子テーブルにもある:

  • NL_H1_**
  • NL_H6_**

合わせて修正する

urasandesu added a commit that referenced this issue Apr 19, 2024
* O5_ODDS_SANREN の子テーブルのテストを問題の修正に合わせて修正
urasandesu added a commit that referenced this issue Apr 19, 2024
* O6_ODDS_SANRENTAN の子テーブルのテストを問題の修正に合わせて修正
urasandesu added a commit that referenced this issue Apr 19, 2024
* H1_HYOSU_ZENKAKE の子テーブルのテストを問題の修正に合わせて修正
urasandesu added a commit that referenced this issue Apr 19, 2024
* H6_HYOSU_SANRENTAN の子テーブルのテストを問題の修正に合わせて修正
urasandesu added a commit that referenced this issue Apr 20, 2024
* バージョン更新
@urasandesu
Copy link
Owner Author

以下の通り修正版がリリースされたため、クローズ

JVLinkToSQLite|競馬ソフト使い放題の会員サービス DataLab.(データラボ)|競馬情報ならJRA-VAN
https://jra-van.jp/dlb/sft/lib/jvl_tosqlite.html

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

No branches or pull requests

1 participant