obnoxious behavior if > 500 results

[bradfordcondon]

A user was confused because their blast results had over 500 results, and therefore no results were displayed.

statonlab/hardwoods_site#199

After some fiddling I discovered that you can set the max target sequences in the advanced options to return 500 or fewer sequences (see below).

Rather than return no results, why not return the first 500 results and display a warning that there are more results?

default blast parameters: error, 500+ sequences

max target = 50

edit:
Also, the error message reads

We have provided the result files for Download at the top of this page; however, we suggest you re-submit your query using a more stringent e-value (i.e. a smaller number).

However, no download link is provided. Note that the link is provided if < 500 entries

[ekcannon]

It's great to have more people using this module so that annoyances like this can be identified and addressed. As I wrote the code to bail if >500 results (a quick solution to a problem that was holding up a website release), I'll tackle improved handling of large results sets.

[bradfordcondon]

As I wrote the code to bail if >500 results (a quick solution to a problem that was holding up a website release), I'll tackle improved handling of large results sets.

Great and thank you! IMO This module works great and is easy to administer so I'm a big fan. I'd be happy to contribute as well.

Cheers

[bradfordcondon]

Oh i forgot the second part of this issue:

The error reads

We have provided the result files for Download at the top of this page; however, we suggest you re-submit your query using a more stringent e-value (i.e. a smaller number).

However, on my site at least, no download link is provided.

[laceysanderson]

Part of the issue with >500 results is the XML reader this module is using. It reads the entire file into memory resulting in a WSOD for large files. The solution is to switch to a stream-based XML reader as suggested in Issue #12.

The lack of download files is a simple bug however that should be easy to fix :-)

[bradfordcondon]

woops sorry for creating an issue largely duplicate with #12

embarrassing considering theres only 2 other issues lol