CIIR / Proteus

Million Book Project
8 stars 5 forks source link

Bug in duplication #107

Closed hamed-zamani closed 8 years ago

hamed-zamani commented 8 years ago

When I selected "All" to search and issue the "states" query. The system retrieve just one result, with 9 duplications. Issue 1: I am not sure it's a bug or not, but there are lots of "transmitting states" in the description of the search result. I believe it doesn't make sense to repeat such a thing a million times!

Issue 2: Do this: click on duplications to expand them. Then scroll down to load more duplications. Now go up and hide duplications. Now the results will change from 1 to much more! and all of them seem to be duplicate!

mzarozinski commented 8 years ago

This is the "correct" behavior - for now - n the future we'd like to filter out these "junk" pages.

j-allan commented 8 years ago

I disagree that this behavior is correct.

When I run the query “states” I get one result and a claim that there are 9 duplicates. It also maintains the “Search request sent to server!” notification on the bottom of the list.

When I click “show duplicate” and then scroll down, only the first eight are showed. Then it expands and adds more. Far more than just one more. It also adds a number of non-duplicates in the list (though they look very similar). When I scroll down to the bottom of that list it loads more. Recall that ONE item was shown to start with; now there are many.

At this point, if I scroll back up to the top and click “hide duplicates” it collapses them and says there are 24.

The numbers are changing and are not consistent. This behavior is not correct.

From: Michael Zarozinski [mailto:notifications@github.com] Sent: Thursday, May 5, 2016 8:20 AM To: CIIR/Proteus Proteus@noreply.github.com Subject: Re: [CIIR/Proteus] Bug in duplication (#107)

This is the "correct" behavior - for now - n the future we'd like to filter out these "junk" pages.

— You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHubhttps://github.com/CIIR/Proteus/issues/107#issuecomment-217139282