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

Refactor microscope modules #311

Merged
merged 5 commits into from
Mar 4, 2024
Merged

Conversation

itutu-tienday
Copy link
Collaborator

@itutu-tienday itutu-tienday commented Feb 29, 2024

対応内容

  • リファクタリング
    • テストコード リファクタリング
  • Inscopix Isx Reader
    • isxモジュール IDPS版とOSS版 との互換性確保
      • この対応により、基本的にOSS版のみでIsx Readerを利用可能となる
    • OSS版のテストコードで、 image save でエラー発生
      • 環境セットアップ(pillow)の問題の可能性もある。調査中
        • → isx module間で仕様差異があることを確認。差異を吸収する調整を適用し対処(画像データのdtypeの調整)
  • Olympus OIR Reader
    • framerate のメタデータを取得するよう調整(対応するmetadataからの取得)

@itutu-tienday
Copy link
Collaborator Author

itutu-tienday commented Mar 1, 2024

@ReiHashimoto
このPRを現在メンテナンス中です。
対応完了次第、optinist-for-server リポジトリへも同内容を反映予定です。 一旦ご連絡します。

Copy link

@ReiHashimoto ReiHashimoto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@itutu-tienday itutu-tienday merged commit f0a66ce into develop-main Mar 4, 2024
5 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.

2 participants