Add Hacker News Challenge #164
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: radha <rads_venkatesan@yahoo.com>
zeibura
requested changes
Jul 13, 2020
COBOL Programming Course #2 - Advanced Topics/COBOL Programming Course #2 - Advanced Topics.md
Outdated
Show resolved
Hide resolved
COBOL Programming Course #2 - Advanced Topics/COBOL Programming Course #2 - Advanced Topics.md
Outdated
Show resolved
Hide resolved
COBOL Programming Course #2 - Advanced Topics/COBOL Programming Course #2 - Advanced Topics.md
Outdated
Show resolved
Hide resolved
COBOL Programming Course #2 - Advanced Topics/COBOL Programming Course #2 - Advanced Topics.md
Outdated
Show resolved
Hide resolved
Signed-off-by: radha <rads_venkatesan@yahoo.com>
jellypuno
approved these changes
Jul 15, 2020
|
Hi @MikeBauerCA @zeibura , Can you please review this PR again? I would really like this to be merged. Thank you so much! |
MikeBauerCA
reviewed
Jul 16, 2020
| We will be working on a Hacker News 2015-2016 dataset from Kaggle with a full year’s worth of stories: Our goal is to extract only the Mainframe/COBOL related stories and assign ranking scores to them based on (a simplified version) the published Hacker News ranking algorithm. We will create a front page report that reflects this ranking order. The algorithm works in a way that nothing stays on the front page for too long, so a story’s score will eventually drop to zero over time (the gravity effect). Since our posts are spread out over a year and as older posts will always have a lower (or zero) ranking, we will distort the data so all our stories have the same date and and consider only the times in the ranking score calculation. This will give all our posts a fair chance of landing the front page. Our front page report is published at 11:59pm. [Here's some additional information on the ranking.](http://www.righto.com/2013/11/how-hacker-news-ranking-really-works.html) | ||
|
|
||
| ### The Plan | ||
| [] There are different creative ways of accomplishing this but here’s our plan: We will have a COBOL program that reads the input CSV file and retrieves only the ***Mainframe/COBOL*** stories. It then calculates the ranking score for the stories by factoring in the time they were posted and the number of votes they received. Each of the records is then written to an output dataset along with the ranking score. |
MikeBauerCA
reviewed
Jul 16, 2020
COBOL Programming Course #2 - Advanced Topics/COBOL Programming Course #2 - Advanced Topics.md
Outdated
Show resolved
Hide resolved
MikeBauerCA
reviewed
Jul 16, 2020
|
@raven300 I'll mark as approved. Thanks for making the change. The DCO signoff is failing because it wants you to put raven300 as your name in the signoff. You will need to head to your local branch on your machine to resolve the DCO signoff issue. Be sure to pull first. |
MikeBauerCA
approved these changes
Jul 16, 2020
Signed-off-by: Radha Venkatesan <rads_venkatesan@yahoo.com> Signed-off-by: radha <rads_venkatesan@yahoo.com>
|
Done. Thank you! |
|
@zeibura once you are satisfied with your requested changes, please approve and merge :) |
zeibura
approved these changes
Jul 20, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Signed-off-by: radha rads_venkatesan@yahoo.com