MIT-LCP / MIT-LCP/wfdb-python

Add validation of segments when writing a `MultiRecord`

オープン
#557 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

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

主要言語
Jupyter Notebook
スター
853
フォーク
322
PR マージ指標
30日以内にマージされた PR はありません

説明

As mentioned in #555, when writing multisegment records we should validate the segments. A good initial step would be to simply add the already developed _check_segment_cohesion validation to the workflow.

There are likely a number of other validations that should be performed. For example, we may want to confirm that samps_per_frame is set for segments with expanded signals and that their length is as expected.

Please add any additional suggestions for validating the segments below.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

はじめの一歩

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

調査の方向性

Issue にはファイルもテストも記載されていないため、まず MultiRecord の書き込みワークフローを追跡し、既存の _check_segment_cohesion バリデーションの場所を特定します。#555 と現在のバリデーションをコンテキストとして使用し、どの追加のセグメントチェックがスコープに含まれるのか、またどの完了基準を適用すべきかを確認してください。

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

評価

技術スタック
python
領域
data
issue の種類
機能追加
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
48/100

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

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