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

Fix loading image on VISUALIZE plot #242

Closed
ReiHashimoto opened this issue Dec 12, 2023 · 0 comments · Fixed by #243
Closed

Fix loading image on VISUALIZE plot #242

ReiHashimoto opened this issue Dec 12, 2023 · 0 comments · Fixed by #243
Assignees
Labels
backend bug Something isn't working
Milestone

Comments

@ReiHashimoto
Copy link

VISUALIZE画面でのImagePlotで表示するデータの改修。

  1. Start/End Indexを指定してLOADをクリック
  2. /image/{filepath:path}?start_index={start_index}&end_index={end_index}のAPI call
  3. OPTINIST_DIR/input/{workspace_id}/{image_file_basename}/{image_file_basename}_{start_index}_{end_index}.jsonが生成され、レスポンスとして返される
  4. Plot画面でLOADしたindexの範囲で動画再生が可能になる

上記の操作をした際に、
Inputで使用しているtiffファイルの場合は期待通りに動作している。

Screenshot 2023-12-12 at 17 32 38


outputで生成されたデータの場合、end_indexを100などにしても、10までしかデータが取得できない。
3次元のoutputの画像の場合、inputのデータ同様にindex指定で再生できるようにしたい。

Screenshot 2023-12-12 at 17 41 44

主な原因は、以下の2箇所

  1. 3次元の配列の場合、ImageDataのsave_jsonの処理の際にindex [:10]のデータしかjsonに保存していない

    JsonWriter.write_as_split(self.json_path, create_images_list(self.data))

    def create_images_list(data):
    if len(data.shape) == 2:
    save_data = copy.deepcopy(data)
    elif len(data.shape) == 3:
    save_data = copy.deepcopy(data[:10])
    else:
    assert False, "data is error"

  2. image_fileの拡張子がjsonの場合は、start_index, end_indexの設定が反映されていない。

    if ext in ACCEPT_TIFF_EXT:
    filepath = join_filepath([DIRPATH.INPUT_DIR, workspace_id, filepath])
    save_dirpath = join_filepath(
    [
    os.path.dirname(filepath),
    filename,
    ]
    )
    json_filepath = join_filepath(
    [save_dirpath, f"{filename}_{str(start_index)}_{str(end_index)}.json"]
    )
    if not os.path.exists(json_filepath):
    save_tiff2json(filepath, save_dirpath, start_index, end_index)
    else:
    json_filepath = filepath

対応方針(案)

以下のような対応が予測されるが、betterな実装方法があればそちらを採用。

  • ImageDataにおいて、3次元データの場合は
    • save_json→jsonファイルを保存しない
    • output_path→tiffファイルのパスにする
@ReiHashimoto ReiHashimoto added bug Something isn't working backend labels Dec 12, 2023
@ReiHashimoto ReiHashimoto self-assigned this Dec 13, 2023
@ReiHashimoto ReiHashimoto added this to the v1.1.0 milestone Dec 13, 2023
ReiHashimoto added a commit that referenced this issue Dec 15, 2023
[#242] outputの動画のrange指定機能修正
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant