duckduckgo / zeroclickinfo-fathead

DuckDuckGo Instant Answers based on keyword data files
https://duckduckhack.com/
Other
318 stars 365 forks source link

NodeJS: fix duplicates #628

Closed bfmags closed 7 years ago

bfmags commented 7 years ago

Description

NodeJS output.txt file has a lot of duplicates that need to be removed. One way to solve this issue is to introduce a Dictionary to keep track of Articles and avoid duplicates.

Steps to recreate

Run tests in duckpan: DDG_TEST_FATHEAD=node_js prove --quiet t/validate_fathead.t 1>/dev/null

https://gist.github.com/bfmags/86f81caf513c0078b1d3be3918c22526

Refer to documentation : https://docs.duckduckhack.com/programming-mission/creating-effective-fatheads.html

People to notify

@MariagraziaAlastra @tagawa @moollaza

Get Started

Resources


Instant Answer Page: https://duck.co/ia/view/node_js

bfmags commented 7 years ago

@ahetawal-p should I assign you to this one? :) Thanks

ahetawal-p commented 7 years ago

Sure @bfmags I can take a look, although might be a little late getting to this. As I might be away for Holidays soon :)

bfmags commented 7 years ago

@ahetawal-p Cheers! I'll assign you but no pressure on timing! 👍

ahetawal-p commented 7 years ago

@bfmags added a fix, take a look when you get some time.