Dave MacFarlane
Dave MacFarlane
I still think it would be a bad idea to change the default in a backwards incompatible way without a strong reason for doing so, but I'm not opposed to...
There should already be tests for the parts that are testable (ie. css parsing, the parts that check if an element matches a style, etc), but the rendering is harder...
@jojo05 there was some benchmark tests which didn't have the arguments updated when a function changed its signature. It should be fixed now. I'm not sure why you'd get a...
What do you mean "does not use instrument tables while the stats module expects it"? Does the stats module not work with json data?
That doesn't make sense. The completion status is on the flag table, not the instrument table. It looks like it's just doing it to do `WHERE i.CommentID NOT LIKE 'DDE%'`...
actually the inner join is implicitly filtering out other instruments too, but that can be done with the test_name on the flag table.
How is this happening? If the database is referencing a file that doesn't exist that sounds like it *is* an internal server error?
the problem is already on the agenda for the next imaging meeting, it's because BrainBrowser uses the API (which requires access to the candidate) while the imaging_browser has its own...
The workaround (for now) is to give the user access to the sites, instead of the all_sites permission.
Sent as draft because this is a significant change for after the 26 release, but it should be good to go now.