-
Notifications
You must be signed in to change notification settings - Fork 3
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
Feature/cloud/improve post process #384
Feature/cloud/improve post process #384
Conversation
…mprove-post-process
- Cleanup of pkl at start of workflow - Forced pkl save process in case of error in post_process.py
- update ProcessType define - minor improvement
- minor improvement
- BE) update WorkflowResult for support PostProcessResult - FE) update post process result check request
# Operate remote storage. | ||
if RemoteStorageController.use_remote_storage(): | ||
# Get workspace_id, unique_id from output file path | ||
workspace_id, unique_id = __rule.output.split("/")[-4:-2] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
他の部分でも出てくるようなら専用のパーサ関数を書いてもいいかもしれません
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
現行コードの類似コードを踏襲していましたが(output_pathよりid情報を取得)、ご指摘のように関数化が適切なロジックのため、他の箇所と合わせて整理を行います。(別途 refactoring PR を発行予定)
- 現行の類似コード例
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@emuemuJP
以下で、上記のreview指摘に関する修正PRを作成しています。
上記PRがOKであれば、同内容を こちらにも反映します。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
以下にて修正を適用
…rser Add a parser for each ID in the experiment output data path
- add note comment - added use of ExptOutputPathIds
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
各コメント事項への対応を実施しています。
再reviewを お願いいたします。
# Operate remote storage. | ||
if RemoteStorageController.use_remote_storage(): | ||
# Get workspace_id, unique_id from output file path | ||
workspace_id, unique_id = __rule.output.split("/")[-4:-2] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
以下にて修正を適用
…mprove-post-process
Contents
GUI Changes
post_process
processing results display to the Experiment Table on the RECORD screen