Skip to content
This repository was archived by the owner on Apr 23, 2025. It is now read-only.

Change name of PNG to be saved #177

Closed
wants to merge 1 commit into from
Closed

Change name of PNG to be saved #177

wants to merge 1 commit into from

Conversation

RCdeWit
Copy link
Contributor

@RCdeWit RCdeWit commented Dec 28, 2021

The description refers to the basic use case, in which the output image is called confusion_matrix.png rather than plot.png. Assuming users build off the basic example, I think it'd help to be consistent.

@julieg18 julieg18 temporarily deployed to cml-dev-patch-2-ihxlsz963px0j0 December 28, 2021 15:38 Inactive
@casperdcl casperdcl self-requested a review December 30, 2021 11:22
casperdcl added a commit to iterative/cml_base_case that referenced this pull request Dec 30, 2021
casperdcl added a commit to iterative/example_cml that referenced this pull request Dec 30, 2021
@casperdcl casperdcl added documentation Markdown files p0-critical Max priority (ASAP) labels Dec 30, 2021
casperdcl added a commit that referenced this pull request Dec 30, 2021
@casperdcl casperdcl mentioned this pull request Dec 30, 2021
@casperdcl
Copy link
Contributor

Thanks for pointing this out! We've renamed to plot.png everywhere for consistency so fixed upstream repos now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Markdown files p0-critical Max priority (ASAP)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants