OneBusAway/onebusaway-application-modules — pull request #1 by jeffmaki (2012-03-06 19:13:38+00:00)
Title: Patch for eventual merge with OBA-app-mods trunk for OBA-NYC

Repasting the developer list post here--to get the conversation started--

Hello OBA-ers,

Brian--this is mostly directed to you: I'm writing to submit a series
of smallish patches we made as part of OBA's integration with MTA Bus
Time to OBA trunk, hopefully to eventually merge these two codebases
back together. Patch is attached. I'm also happy to commit these if I
receive the go-ahead, otherwise feel free to merge them yourself.

Rough summary of patch contents are:

ServiceAlertsServiceImplTest: unit test for service alert changes

RoutesBeanServiceImpl: change to eliminate internal cached data
structures in favor of a "cacheable" method that does the same;
queryBean configurable minimum lucene score to keep

VehicleStatusBeanServiceImpl: informative debugging message only

RefreshableResources: new refreshable resource key for
WhereGeospatialServiceImpl

SiriService: change to enable unit testing

VehicleStatusServiceImpl: change to reset block state in
BlockVehicleLocationService if a record is received by the
VehicleStatusServiceImpl that has no trip/block assigned to it (e.g.
inference drops out) for the same vehicle ID

WhereGeospatialServiceImpl: made "refreshable", plus informative debug message

ServiceAlertsServiceImpl: switch to turn bundle persistence of service
alerts on/off; new time parameter to getServiceAlertIdsAsObject

BlockStatusServiceImpl: enlargement of search window for block list
that backs getBlocksForRoute(), to better handle vehicles assigned to
"late" trips (that weren't within the previously super tight bound
(instant?) specified as "time").

BlockGeospatialServiceImpl: change made to, when "refreshed", clear
the blockSequenceIndices so the old ones are not left in the internal
cache state.

ServiceAreaServiceImpl: turn off
_calculateDefaultBoundsFromAgencyCoverage by default because the call
was failing when required to start the TDS webapp; TDS service was
calling itself, and could not start until itself was available (if
that makes sense).

SearchQueryBean: moved minimum lucene score to keep to this bean so
that it is configurable per query by the developer.

Again, patch is attached for your review/acceptance.

Thanks--let me know what you think, or fire questions my way for
forwarding on to any other relevant team members--

-Jeff