Skip to content

Commit

Permalink
Delete unnecessary dashboard_artifact system_attr deletion
Browse files Browse the repository at this point in the history
  • Loading branch information
gen740 committed Nov 17, 2023
1 parent 36ca4f7 commit 68e181e
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions optuna_dashboard/artifact/_backend.py
Original file line number Diff line number Diff line change
Expand Up @@ -204,10 +204,6 @@ def delete_study_artifact(study_id: int, artifact_id: str) -> dict[str, Any]:
return {"reason": "Cannot access to the artifacts."}
artifact_store.remove(artifact_id)

Check warning on line 205 in optuna_dashboard/artifact/_backend.py

View check run for this annotation

Codecov / codecov/patch

optuna_dashboard/artifact/_backend.py#L202-L205

Added lines #L202 - L205 were not covered by tests

# The artifact's metadata is stored in one of the following two locations:
storage.set_study_system_attr(
study_id, _dashboard_artifact_prefix(study_id) + artifact_id, json.dumps(None)
)
storage.set_study_system_attr(

Check warning on line 207 in optuna_dashboard/artifact/_backend.py

View check run for this annotation

Codecov / codecov/patch

optuna_dashboard/artifact/_backend.py#L207

Added line #L207 was not covered by tests
study_id, ARTIFACTS_ATTR_PREFIX + artifact_id, json.dumps(None)
)
Expand Down

0 comments on commit 68e181e

Please sign in to comment.