This repository (optuna-dashboard) and Goptuna share common code.
This pull request may therefore be ported to Goptuna.
Make sure that you understand the consequences concerning licenses and check the box below if you accept the term before creating this pull request.
[x] I agree this patch may be ported to Goptuna by other Goptuna contributors.
Reference Issues/PRs
NA
What does this implement/fix? Explain your changes.
When deleting a study, all artifacts associated with that study were previously deleted as well. However, the problem occurred when a study was copied, resulting in multiple IDs pointing to the same artifact. In such cases, after deleting the artifact and one ID, the other IDs were pointing to a non-existent reference. This PR addresses the problem by introducing a feature that allows users to choose whether or not to delete the associated artifacts.
Dialog Design
To align with the design changes mentioned above, I have also modified DeleteArtifactDialog:
Contributor License Agreement
This repository (
optuna-dashboard
) and Goptuna share common code. This pull request may therefore be ported to Goptuna. Make sure that you understand the consequences concerning licenses and check the box below if you accept the term before creating this pull request.Reference Issues/PRs
NA
What does this implement/fix? Explain your changes.
When deleting a study, all artifacts associated with that study were previously deleted as well. However, the problem occurred when a study was copied, resulting in multiple IDs pointing to the same artifact. In such cases, after deleting the artifact and one ID, the other IDs were pointing to a non-existent reference. This PR addresses the problem by introducing a feature that allows users to choose whether or not to delete the associated artifacts.
Dialog Design
To align with the design changes mentioned above, I have also modified
DeleteArtifactDialog
: