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.

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

Added Road accident analysis project using Power BI #638

Closed
wants to merge 12 commits into from
Closed

Added Road accident analysis project using Power BI #638

wants to merge 12 commits into from

Conversation

jeet-Abhi123
Copy link
Contributor

I have successfully added the project as a new folder in the repository.

First initial step
Adding dashboard image
Adding images of the dashboard
Dataset and .pbit file has been uploaded
Updating the data cleaning and filling techniques
one hot encoding
Added one hot encoded
@invigorzz313
Copy link
Contributor

Please tag the corresponding issue.

@jeet-Abhi123
Copy link
Contributor Author

jeet-Abhi123 commented May 20, 2024

#622

@invigorzz313
Copy link
Contributor

@jeet-Abhi123 You seem to have created this PR for 2 issues of different topics.
It is suggested that a single PR fixes single issue. So, please create separate ones.

@jeet-Abhi123
Copy link
Contributor Author

How can I create different PR? When I clicked the PR of issue #623 it is getting adding to this PR only.

@invigorzz313
Copy link
Contributor

@jeet-Abhi123 That is because you mentioned 2 issue numbers above (622 and 623).
Also the files you uploaded in the commit in this PR have both Road accident analysis and Data Cleaning Techniques.
Since this PR is for Road Accident Analysis, retain these files and delete others from this PR.
You can make another PR with Data Cleaning files and tag issue 623 there.

@invigorzz313
Copy link
Contributor

For creating another PR, you must first create another branch in your local repo. And commit the relevant files there.
(Separate issue <-> separate PR <-> separate branch)

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.

2 participants