Added 2021 and 2022 data with aanalysis and updated streamlit app #316
Conversation
Co-authored-by: Nikita [email protected] Co-authored-by: Leena [email protected]
There was a problem hiding this comment.
Awesome @NIKITA320495
some suggestions for you would be : adding description on the changes made in the PR.
incoperating bit more coding comments. so it will improve readbility and understanding for future developments.
|
🎉 Your pull request has been successfully merged! 🎉 Thank you for your valuable contribution to our project. Your efforts are greatly appreciated. Feel free to reach out if you have any more contributions or if there's anything else we can assist you with. Keep up the fantastic work! 🚀 |
|
Could you check and reply here regarding the Streamlit app and Streamlit on Snowflakes are same thing? @NIKITA320495 |
|
hello @sanjay-kv sir, please assign me the PR as well. As Nikita and I collaborated on this one as well. And yes, we'll make sure to add proper comments for readability from now on. As for Snowflakes, we never worked on it, so not sure. |
|
leena just raise one more PR and link the PR to there. and I will assign same level. it has been noted leaderboard is not fetching two people on co-authored commits.
|
|
Ok @sanjay-kv sir ! |
This is a sample PR for the issue recodehive#316 (recodehive#316) that I collaborated on. Due to some technical issues, my PR couldn't be merged, hence a sample merge is being created for fetching updated information on leaderboard.
No description provided.