Skip to content

plots: add smooth template#4046

Merged
efiop merged 1 commit into
treeverse:masterfrom
pared:plot_smooth
Jun 15, 2020
Merged

plots: add smooth template#4046
efiop merged 1 commit into
treeverse:masterfrom
pared:plot_smooth

Conversation

@pared
Copy link
Copy Markdown
Contributor

@pared pared commented Jun 15, 2020

  • ❗ I have followed the Contributing to DVC checklist.

  • 📖 If this PR requires documentation updates, I have created a separate PR (or issue, at least) in dvc.org and linked it here. If the CLI API is changed, I have updated tab completion scripts.

  • ❌ I will check DeepSource, CodeClimate, and other sanity checks below. (We consider them recommendatory and don't expect everything to be addressed. Please fix things that actually improve code or fix bugs.)

Thank you for the contribution - we'll try to review it as soon as possible. 🙏
Fixes #3906

@pared pared requested review from dmpetrov and efiop June 15, 2020 18:44
"loess": Template.Y_ANCHOR,
"on": Template.X_ANCHOR,
"groupby": ["rev"],
"bandwidth": 0.3,
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Added so that users will not have to search for parameter name

@efiop efiop merged commit d2dbe96 into treeverse:master Jun 15, 2020
@efiop
Copy link
Copy Markdown
Contributor

efiop commented Jun 15, 2020

Wonder if it is worth adding a smooth example to https://dvc.org/doc/command-reference/plots ?

@dmpetrov
Copy link
Copy Markdown
Contributor

@efiop for sure, the example is needed

@efiop
Copy link
Copy Markdown
Contributor

efiop commented Jun 15, 2020

@dmpetrov Looks like smoothing doesn't change anything in the examples we have in https://dvc.org/doc/command-reference/plots . @pared @dmpetrov could you guys provide an example for https://dvc.org/doc/command-reference/plots , please?

@jorgeorpinel
Copy link
Copy Markdown
Contributor

jorgeorpinel commented Jun 15, 2020

Hi. Docs is checked but I don't see any ref. to an issue or PR in dvc.org 😢

An example would be greatly appreciated indeed but this requires updates to the plots description(s) as well.

@efiop
Copy link
Copy Markdown
Contributor

efiop commented Jun 16, 2020

@jorgeorpinel Created treeverse/dvc.org#1439 so we don't forget about it.

@jorgeorpinel
Copy link
Copy Markdown
Contributor

Much appreciated!

@pared pared deleted the plot_smooth branch January 4, 2022 10:13
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.

plots smoothing

4 participants