Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Feature]: log transformation #9

Merged
merged 2 commits into from
Mar 27, 2024
Merged

[Feature]: log transformation #9

merged 2 commits into from
Mar 27, 2024

Conversation

xTwo56
Copy link
Contributor

@xTwo56 xTwo56 commented Mar 27, 2024 •
edited by Dev7g
Loading

Description

  • changed method to plot graph of normal distribution of price from displot to kdeplot as it(displot) is deprecated
  • plotted KDE of log transformation of price and compared it with its normal distribution
  • analysed data and added observations

Semver Changes

  • Minor (new features, no breaking changes)

Issues

#2

Checklist

xTwo56 added 2 commits March 27, 2024 17:29
@xTwo56
feat: price distribution plot
724b3af 
distplot-->kdeplot (due to deprecation), observation added
@xTwo56
feat: price_log kde plotted and observation added
0bf56da
@Dev7g
Copy link
Collaborator

Dev7g commented Mar 27, 2024

LGTM

@Dev7g Dev7g added PR:Accept Pr has been accepted and is merged Type:Easy Pr is accepted with difficulty level as easy labels Mar 27, 2024
@Dev7g Dev7g merged commit 1e8c687 into OPCODE-Open-Spring-Fest:main Mar 27, 2024
1 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
PR:Accept Pr has been accepted and is merged Type:Easy Pr is accepted with difficulty level as easy
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@xTwo56 @Dev7g