-
-
Notifications
You must be signed in to change notification settings - Fork 213
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
Online Course Engagement #715
Online Course Engagement #715
Conversation
Our team will soon review your PR. Thanks @siddhant4ds :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It will be better if you combine all the models in a single Jupyter Notebook.
Sure, I can do that. I had just separated them based on model complexity. |
…c3-ML-Crate into online-course-engagement
@abhisheks008 I have made the requested changes. Let me know if any further changes are required. Thanks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved ✅ @siddhant4ds
Pull Request for ML-Crate 💡
Issue Title: Online Course Engagement
Closes: #696
Describe the add-ons or changes you've made 📃
Type of change ☑️
What sort of change have you made:
How Has This Been Tested? ⚙️
Reproducibility: All experiments run with fixed random seed. Multiple runs of each notebook to produce same results.
Dependencies: Code run on local setup as well as cloud-hosted notebooks (Kaggle).
Checklist: ☑️