<feed xmlns='http://www.w3.org/2005/Atom'>
<title>chouette-core/app/services, branch 5642-insee-code</title>
<subtitle>Chouette manage transport static data</subtitle>
<link rel='alternate' type='text/html' href='https://git.teddywing.com/fork/chouette-core/'/>
<entry>
<title>Merge pull request #553 from af83/6884-tomtom-matrix--handle-error-when-response-doesn,t-inclu</title>
<updated>2018-05-16T13:42:24+00:00</updated>
<author>
<name>Alban Peignier</name>
</author>
<published>2018-05-16T13:42:24+00:00</published>
<link rel='alternate' type='text/html' href='https://git.teddywing.com/fork/chouette-core/commit/?id=223d988ef59e562b201cac8391c397630c8953e5'/>
<id>223d988ef59e562b201cac8391c397630c8953e5</id>
<content type='text'>
Handle API error(s) in Tomtom matrix. Fixes #6884</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Handle API error(s) in Tomtom matrix. Fixes #6884</pre>
</div>
</content>
</entry>
<entry>
<title>refs #6923; small improvements</title>
<updated>2018-05-07T06:26:07+00:00</updated>
<author>
<name>Zog</name>
</author>
<published>2018-05-07T06:26:07+00:00</published>
<link rel='alternate' type='text/html' href='https://git.teddywing.com/fork/chouette-core/commit/?id=2c619e4aa072950681be04a603f91597dfcca088'/>
<id>2c619e4aa072950681be04a603f91597dfcca088</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Refs #6923; Implement a consolidated view</title>
<updated>2018-05-04T14:39:51+00:00</updated>
<author>
<name>Zog</name>
</author>
<published>2018-05-04T14:39:51+00:00</published>
<link rel='alternate' type='text/html' href='https://git.teddywing.com/fork/chouette-core/commit/?id=08aa082e941db7e23d91edb8e48bead2b72e9fe3'/>
<id>08aa082e941db7e23d91edb8e48bead2b72e9fe3</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Move `TomTom::Matrix::RemoteError` to`TomTom::Errors::MatrixRemoteError`</title>
<updated>2018-05-04T10:54:25+00:00</updated>
<author>
<name>Teddy Wing</name>
</author>
<published>2018-05-04T10:49:06+00:00</published>
<link rel='alternate' type='text/html' href='https://git.teddywing.com/fork/chouette-core/commit/?id=9c442cd66c7d12266ba53e85949f775f6b842d2d'/>
<id>9c442cd66c7d12266ba53e85949f775f6b842d2d</id>
<content type='text'>
I previously tried to correct a circular dependency problem in
a057276129b1f62b811743db3b8f867a05241ed3, but that didn't fix it (it was
intermittent, and came back).

After some wrangling, I've now deduced with some confidence that the
problem comes from `RouteWayCostCalculator`, which used
`TomTom::Matrix::RemoteError`. From the way it looks, this seems to mess
up the Rails autoloader since `tom_tom.rb` will try to load the `Matrix`
class from the `TomTom.matrix` call above. Or something.

In an attempt to fix the circular dependency error for real this time,
move the error class to a completely separate module from `Matrix`, and
refer to this when we need to use the error class.

Refs #6884
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
I previously tried to correct a circular dependency problem in
a057276129b1f62b811743db3b8f867a05241ed3, but that didn't fix it (it was
intermittent, and came back).

After some wrangling, I've now deduced with some confidence that the
problem comes from `RouteWayCostCalculator`, which used
`TomTom::Matrix::RemoteError`. From the way it looks, this seems to mess
up the Rails autoloader since `tom_tom.rb` will try to load the `Matrix`
class from the `TomTom.matrix` call above. Or something.

In an attempt to fix the circular dependency error for real this time,
move the error class to a completely separate module from `Matrix`, and
refer to this when we need to use the error class.

Refs #6884
</pre>
</div>
</content>
</entry>
<entry>
<title>RouteWayCostCalculator: Don't update `costs` if response errors</title>
<updated>2018-05-02T17:27:39+00:00</updated>
<author>
<name>Teddy Wing</name>
</author>
<published>2018-05-02T17:27:39+00:00</published>
<link rel='alternate' type='text/html' href='https://git.teddywing.com/fork/chouette-core/commit/?id=d53978539c71d598d7cb5f47ad9e3f30a83eb06a'/>
<id>d53978539c71d598d7cb5f47ad9e3f30a83eb06a</id>
<content type='text'>
If there's an API error, we shouldn't update the route's `costs` field.
Let's say we've already calculated some costs for a route A. We then edit
and re-save A, which triggers a recalculation of the costs. Now the
TomTom API responds with an error. We don't want to overwrite our
existing costs with an empty array because they could still be useful.
In this case, we should instead keep the existing costs we already had.

To achieve this, move the `RemoteError` rescue into
`RouteWayCostCalculator`, leaving the error unhandled in
`TomTom.matrix`.

Refs #6884
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
If there's an API error, we shouldn't update the route's `costs` field.
Let's say we've already calculated some costs for a route A. We then edit
and re-save A, which triggers a recalculation of the costs. Now the
TomTom API responds with an error. We don't want to overwrite our
existing costs with an empty array because they could still be useful.
In this case, we should instead keep the existing costs we already had.

To achieve this, move the `RemoteError` rescue into
`RouteWayCostCalculator`, leaving the error unhandled in
`TomTom.matrix`.

Refs #6884
</pre>
</div>
</content>
</entry>
<entry>
<title>:fire: log</title>
<updated>2018-04-09T10:16:07+00:00</updated>
<author>
<name>Zog</name>
</author>
<published>2018-04-09T10:16:07+00:00</published>
<link rel='alternate' type='text/html' href='https://git.teddywing.com/fork/chouette-core/commit/?id=cb3a2fecf94c658448a6886529efc417f8d8dd39'/>
<id>cb3a2fecf94c658448a6886529efc417f8d8dd39</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Refs #6360; Add checks on calendars during WorkbenchImport</title>
<updated>2018-04-09T09:22:40+00:00</updated>
<author>
<name>Zog</name>
</author>
<published>2018-04-09T09:20:56+00:00</published>
<link rel='alternate' type='text/html' href='https://git.teddywing.com/fork/chouette-core/commit/?id=27442a9ffe40d0f548b1dc99ab917a8d8b0a02a2'/>
<id>27442a9ffe40d0f548b1dc99ab917a8d8b0a02a2</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>RouteWayCostCalculator: Use `TomTom.matrix` instead of batch</title>
<updated>2018-03-27T14:46:18+00:00</updated>
<author>
<name>Teddy Wing</name>
</author>
<published>2018-03-27T09:18:17+00:00</published>
<link rel='alternate' type='text/html' href='https://git.teddywing.com/fork/chouette-core/commit/?id=f257a8b3a7ae4c59c6907b41f6e8d7dc74259917'/>
<id>f257a8b3a7ae4c59c6907b41f6e8d7dc74259917</id>
<content type='text'>
Calculate routes using the TomTom matrix API. Update the spec's HTTP
request stub accordingly (note that the stub is still fake).

Refs #6222
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Calculate routes using the TomTom matrix API. Update the spec's HTTP
request stub accordingly (note that the stub is still fake).

Refs #6222
</pre>
</div>
</content>
</entry>
<entry>
<title>Route: Add `#calculate_costs!` to populate `#costs` with `WayCost`s</title>
<updated>2018-03-14T15:42:02+00:00</updated>
<author>
<name>Teddy Wing</name>
</author>
<published>2018-03-13T14:37:55+00:00</published>
<link rel='alternate' type='text/html' href='https://git.teddywing.com/fork/chouette-core/commit/?id=dbcd099b3d38f7c584450edffc8bee0a2502b170'/>
<id>dbcd099b3d38f7c584450edffc8bee0a2502b170</id>
<content type='text'>
This new method will launch a worker that takes the route's `StopArea`s,
converts them to `WayCost`s, sends them to the TomTom API to calculate
distance and time costs, and saves those costs to the route's `costs`
field.

Refs #6095
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This new method will launch a worker that takes the route's `StopArea`s,
converts them to `WayCost`s, sends them to the TomTom API to calculate
distance and time costs, and saves those costs to the route's `costs`
field.

Refs #6095
</pre>
</div>
</content>
</entry>
<entry>
<title>Small improvements on referential overviews</title>
<updated>2018-03-01T19:51:10+00:00</updated>
<author>
<name>Zog</name>
</author>
<published>2018-02-28T15:09:05+00:00</published>
<link rel='alternate' type='text/html' href='https://git.teddywing.com/fork/chouette-core/commit/?id=2899e02957f87f20f4b12aaf9190acf9555a7042'/>
<id>2899e02957f87f20f4b12aaf9190acf9555a7042</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
</feed>
