Skip to content

Conversation

ayush-raj8
Copy link
Contributor

@ayush-raj8 ayush-raj8 commented Apr 8, 2021

Description

Added a folder 'corona cases forecasting' having python script , result images and requirements.txt.

Fixes #828

Have you read the Contributing Guidelines on Pull Requests?

  • [] Yes

Type of change

Please delete options that are not relevant.

  • [] New feature (non-breaking change which adds functionality)

Checklist:

  • [] My code follows the style guidelines(Clean Code) of this project
  • [] I have performed a self-review of my own code
  • [] I have commented my code, particularly in hard-to-understand areas
  • [] I have created a helpful and easy to understand README.md
  • [] My documentation follows Template for README.md
  • [] My changes generate no new warnings .
  • [] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Congratulations!! for making your first PR at Amazing-Python-Scripts, our mentors will review it soon.

@@ -0,0 +1,17 @@
# corona Cases Forecasting
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
# corona Cases Forecasting
# Corona Cases Forecasting




def recovered_cases():
Copy link
Contributor

Choose a reason for hiding this comment

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

Here is the error, you previously defined the "recovered_cases" as a pandas data frame and now defining it as a function. This may not have any effect in Jupyter notebook as they are cell-based but in a Py script, it will give an error. Change this function name and at other instances where it is used

@kaustubhgupta kaustubhgupta added bug Something isn't working test fail The code failed the initial GitHub tests labels Apr 9, 2021
plt.title('New active Cases Forecast')
plt.xticks(rotation=90)
# plt.figure(figsize=(22,22))
plt.savefig("plot1.png",bbox_inches='tight')
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
plt.savefig("plot1.png",bbox_inches='tight')
plt.savefig("./corona cases forecasting/Results/plot1.png",bbox_inches='tight')

plt.title('New death Cases Forecast')
plt.xticks(rotation=90)
# plt.figure(figsize=(22,22))
plt.savefig("plot2.png",bbox_inches='tight')
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
plt.savefig("plot2.png",bbox_inches='tight')
plt.savefig("./corona cases forecasting/Results/plot2.png",bbox_inches='tight')

plt.title('New recovered Cases Forecast')
plt.xticks(rotation=90)
# plt.figure(figsize=(22,22))
plt.savefig("plot3.png",bbox_inches='tight')
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
plt.savefig("plot3.png",bbox_inches='tight')
plt.savefig("./corona cases forecasting/Results/plot3.png",bbox_inches='tight')

plt.show()


choice=input("F for fresh cases,D for death cases,R for recovered cases prediction")
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
choice=input("F for fresh cases,D for death cases,R for recovered cases prediction")
choice=input("F for fresh cases,D for death cases,R for recovered cases prediction: ")

recovered_cases_daily()
else:
print("Enter a valid choice")

Copy link
Contributor

Choose a reason for hiding this comment

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

Remove these long trailing blank lines

@kaustubhgupta kaustubhgupta added minor-change-not-bug Suggested a minor change in code, not a bug and removed bug Something isn't working test fail The code failed the initial GitHub tests labels Apr 10, 2021
Copy link
Contributor Author

@ayush-raj8 ayush-raj8 left a comment

Choose a reason for hiding this comment

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

Ok

Copy link
Contributor Author

@ayush-raj8 ayush-raj8 left a comment

Choose a reason for hiding this comment

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

Done

Copy link
Contributor

@kaustubhgupta kaustubhgupta left a comment

Choose a reason for hiding this comment

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

👍🏼

@kaustubhgupta kaustubhgupta added next review needed Approved by some mentors, more approvals needed and removed minor-change-not-bug Suggested a minor change in code, not a bug labels Apr 11, 2021
@kaustubhgupta kaustubhgupta requested review from antrikshmisri and removed request for antrikshmisri April 13, 2021 04:52
@santushtisharma10 santushtisharma10 added Approved PR Approved and Ready to Merge and removed next review needed Approved by some mentors, more approvals needed labels Apr 13, 2021
@santushtisharma10 santushtisharma10 added gssoc23 Issues created for/by the GirlScript Summer of Code'23 Participants level3 New features, Major bug fixing labels Apr 13, 2021
Copy link
Owner

@avinashkranjan avinashkranjan left a comment

Choose a reason for hiding this comment

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

LGTM

@avinashkranjan avinashkranjan merged commit 521e1c4 into avinashkranjan:master Apr 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved PR Approved and Ready to Merge gssoc23 Issues created for/by the GirlScript Summer of Code'23 Participants level3 New features, Major bug fixing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Time series forecasting covid 19 case Notebook
5 participants