joinbox / infect

INterface For Empirical antimicrobial ChemoTherapy. An easy-to-use online tool to bring the latest resistance data from bench to bedside
http://infect.in/
MIT License
1 stars 0 forks source link

Filter returns wrong results #22

Closed fxstr closed 9 years ago

fxstr commented 9 years ago

Returns gram+ for gram-

fxstr commented 9 years ago

@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.

pascalmfrey commented 9 years ago

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 commented 9 years ago

gram+ looks good so far: image

fxstr commented 9 years ago

gram- image

fxstr commented 9 years ago

gram+ & coccus: image

fxstr commented 9 years ago

gram- / rod

image

fxstr commented 9 years ago

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

pascalmfrey commented 9 years ago

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 commented 9 years ago

As for the shapes:

Thanks for looking into it!

fxstr commented 9 years ago

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

fxstr commented 9 years ago

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

pascalmfrey commented 9 years ago

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 https://github.com/joinbox/infect/issues/22#issuecomment-75288994.

fxstr commented 9 years ago

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 commented 9 years ago

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 https://github.com/joinbox/infect/issues/22#issuecomment-75363953.