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

Netflix Stock Prediction Using Regression #492

Closed
wants to merge 1 commit into from

Conversation

ashis2004
Copy link
Contributor

🛠️ Related Issue

📝 Description

[Provide a concise description of the changes. Include the purpose and any relevant context or details.]

🔍 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

[Include any additional context or information that might assist reviewers in understanding the changes]

@ashis2004
Copy link
Contributor Author

@Akshat111111 pls check it issue #480

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.

it should be inside NYSE

@ashis2004
Copy link
Contributor Author

I didn't get @Akshat111111

@ashis2004
Copy link
Contributor Author

i've to make separate folder "NYSE" inside that i've to make PR @Akshat111111

@ashis2004
Copy link
Contributor Author

while i was cloning the repo i didn't get all file and folders only these are
image

@ashis2004
Copy link
Contributor Author

pls review it @Akshat111111

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.

it should be inside NYSE.Make a new pr if you are facing any difficulty

@ashis2004
Copy link
Contributor Author

it should be inside NYSE.Make a new pr if you are facing any difficulty

I'm not getting all files while cloning pls merge it @Akshat111111

@Akshat111111
Copy link
Owner

make a new branch and try with it

@ashis2004
Copy link
Contributor Author

make a new branch and try with it

Okay i'll do

@ashis2004 ashis2004 closed this Jul 22, 2024
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