Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Filter returns wrong results #22

Closed
fxstr opened this issue Nov 3, 2014 · 14 comments
Closed

Filter returns wrong results #22

fxstr opened this issue Nov 3, 2014 · 14 comments
Assignees
Milestone

Comments

@fxstr
Copy link
Contributor

fxstr commented Nov 3, 2014

Returns gram+ for gram-

@fxstr fxstr changed the title Filter doesn't work Filter returns wrong results Nov 3, 2014
@fxstr fxstr assigned fxstr and pascalmfrey and unassigned fxstr Feb 10, 2015
@fxstr
Copy link
Contributor Author

fxstr commented Feb 11, 2015

@Alpascal Should be fixed. Please provide some test cases to me (what to enter in the search field, what results to expect – especially with combinated search terms). Thanks a lot.

@fxstr fxstr modified the milestone: Prototype Feb 11, 2015
@pascalmfrey
Copy link
Contributor

test cases: "gram+" should return bacterial species with the name containing "Staphylococcus" or "Streptococcus", and none of "Klebsiella", "Serratia", "Neisseria", "Pseudomonas" (and others). "gram-" should return the opposite (all "Klebsiella", none "Staphylococcus").

Combination of "gram+" and "coccus" should return species of "Staphylococcus" and "Streptococcus", and none of the ones above that are returned by "gram -".
Other combinations: gram+ & rod -> Clostridium; gram- & coccus -> Neisseria; gram- & rod -> Klebsiella, E. coli.

Does this help? Unfortunately I have very limited IT infrastructure here in Sri Lanka, but will do what's possible from here. Thanks for the work!

@fxstr
Copy link
Contributor Author

fxstr commented Feb 19, 2015

gram+ looks good so far:
image

@fxstr
Copy link
Contributor Author

fxstr commented Feb 19, 2015

gram-
image

@fxstr
Copy link
Contributor Author

fxstr commented Feb 19, 2015

gram+ & coccus:
image

@fxstr
Copy link
Contributor Author

fxstr commented Feb 19, 2015

gram- / rod

image

@fxstr
Copy link
Contributor Author

fxstr commented Feb 19, 2015

Looks good, right? Thaks a lot for the cases, @Alpascal !

@pascalmfrey
Copy link
Contributor

Unfortunately it's not correct yet, wrong results are returned in all, gram+, gram- and shape filters.
The following species must not be returned when filter gram+ is active:
image

When filter gram- ist active, the following must not be returned:
image

Could it be a problem with labelling in different language versions, i.e. when the filter term "rod" is entered, it should show all bacteria not only labelled as "rod" in the english language version but also "bacillus" as the "db entry term"?

The above list examples are from the old .xls file located here: infect/docs/Antibiotika Spektrum.xls

Species in the first list are all gram-, while species from the second are all gram+; however, there are further species in our database not in any of these lists that also have a gram property (but I think it's easiest if I look at those as soon as the filter works with the examples given above, as the follow the same structural rules)

@pascalmfrey
Copy link
Contributor

As for the shapes:

  • Acinetobacter, Aeromonas, Shigella, Stenotrophomonas, and Yersinia are not cocci, while both species of Enterococcus are gram+ cocci and not returned
  • both Enterococcus and both Neisseria species are cocci, not rods, while Klebsiella, Pseudomonas and Proteus are both gram- rods but not returend

Thanks for looking into it!

@fxstr
Copy link
Contributor Author

fxstr commented Feb 20, 2015

Coccus AND Gram+ return:
screen shot 2015-02-20 at 19 01 06

@fxstr
Copy link
Contributor Author

fxstr commented Feb 20, 2015

@tobiaskneubuehler Will do a release, you'll be able to test online soon! Yeah.

fxstr pushed a commit that referenced this issue Feb 20, 2015
@pascalmfrey
Copy link
Contributor

Does the "coccus" and "gram+" also return the other species like
Streptococcus and Staphylococcus?
I will give it a thorough test as soon as it's online and my internet
connection is a bit faster than barely 28kbps...
On Feb 20, 2015 11:35 PM, "Felix Steiner" notifications@github.com wrote:

Coccus AND Gram+ return:
[image: screen shot 2015-02-20 at 19 01 06]
https://cloud.githubusercontent.com/assets/1211144/6306336/dd1ed354-b932-11e4-97fe-6675d8fc76bb.PNG


Reply to this email directly or view it on GitHub
#22 (comment).

@fxstr
Copy link
Contributor Author

fxstr commented Feb 21, 2015

Here's the full list – and yes, it does. Hope that's the expected behaviour.

screen shot 2015-02-21 at 10 33 04

@pascalmfrey
Copy link
Contributor

looks lovely! :-)
On Feb 21, 2015 3:05 PM, "Felix Steiner" notifications@github.com wrote:

Here's the full list – and yes, it does. Hope that's the expected
behaviour.

[image: screen shot 2015-02-21 at 10 33 04]
https://cloud.githubusercontent.com/assets/1211144/6313598/4e609a66-b9b5-11e4-9566-f08d9c80f9c8.png


Reply to this email directly or view it on GitHub
#22 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants