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

Nifty Close Price Prediction using Neural Networks #274

Merged
merged 7 commits into from
Jun 5, 2024

Conversation

Srijansarkar17
Copy link
Contributor

πŸ› οΈ Related Issue

  • Closes #[270]
  • Issue already closed, just had to add the model inside the nifty folder

πŸ“ Description

Added the model inside the Nifty folder.

πŸ” Type of PR

  • πŸ› Bug fix
  • ✨ Feature enhancement
  • πŸ“š Documentation update
  • πŸ›‘ Other (specify): _______________

πŸ“Έ Screenshots / πŸŽ₯ Videos (if applicable)

[Attach screenshots or videos]

βœ… Checklist

  • I have performed a self-review of my code.
  • I have read and followed the Contribution Guidelines.
  • I have tested the changes thoroughly before submitting this pull request.
  • I have provided relevant issue numbers, screenshots, and videos after making the changes.
  • I have commented my code, particularly in hard-to-understand areas.

ℹ️ Additional Context

I had already submitted my code, you just to told create a folder inside the nifty folder named as niftyclosepriceprediction and inside that store the model. So i did that, and thank you for reviewing

Copy link
Owner

@Akshat111111 Akshat111111 left a comment

Choose a reason for hiding this comment

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

now its fine

@Akshat111111 Akshat111111 merged commit 61ce51b into Akshat111111:main Jun 5, 2024
1 check passed
Copy link

github-actions bot commented Jun 5, 2024

πŸŽ‰ Your pull request has been successfully merged! πŸŽ‰ Thank you for your contribution to our project. Your efforts are greatly appreciated. Keep up the fantastic work! πŸš€

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants