ML-For-Beginners
ML-For-Beginners copied to clipboard
updated lineplot() arguments
These are just small changes for updated packages:
In commit 1d9d018:
- Latest version of
seabornpackage where it is supported to passxandyas arguments forlineplot()(positional) is v0.10 (link) . Since version 0.11 (latest is 0.12)lineplot()takesxandyonly as keyword argument (and also the new parameterdata) is added
In commit d01ebf4:
- There is an unused
enumerate()which is removed to make code cleaner and easier to read alson_classifiersvariable has no use and it's unnecessary to declare/define.
In commit f471710:
- as in Seaborn docs FacetGrid, Does not have any argument named
size. Instead ofsizeyou should useheightandaspect
In commit 327afbc:
- The same issue with lineplot() in 1d9d018
- Small fixes to text and codes
In commit 8c1ef29:
- Using f-strings instead of using
+operator - using name of variables in output instead of the whole string content, so it'll be easier to read
جميل
This PR has not seen any action for a while! Closing for now, but it can be reopened at a later date.
@carlotta94c @gdequeiroz I think it would be good if you guys take a look at this after 4 month
This PR has not seen any action for a while! Closing for now, but it can be reopened at a later date.