Replace old travel times with OpenTimes #50
Conversation
|
Confirming that we've been working w Dan and would like to update this. Thx
…On Tue, May 13, 2025, 3:59 PM Dan Snow ***@***.***> wrote:
Hi @sjsrey <https://github.com/sjsrey>. I'm Dan Snow, the research
assistant that created the original pre-calculated travel times listed on
the access Resources page
<https://pysal.org/access/resources.html#explore-and-download-pre-computed-travel-times>
.
I've since graduated and moved on to different work, but travel times have
stayed in my heart, so I created OpenTimes <https://opentimes.org/>, a
free database of travel times between all major Census geographies and
years.
I'd love to replace these old pre-calculated times with ones from
OpenTimes. The OpenTimes times are easier to produce, easier to access,
more accurate, and have much better coverage. They'll also be free in
perpetuity. They're just a fun side project for me and a way to keep
tinkering with travel times.
Thanks!
------------------------------
You can view, comment on, or merge this pull request online at:
#50
Commit Summary
- fcdeb36
<fcdeb36>
Replace old travel times with OpenTimes
File Changes
(1 file <https://github.com/pysal/access/pull/50/files>)
- *M* docs/resources.rst
<https://github.com/pysal/access/pull/50/files#diff-93070006f8b532829aafa371f70ab677c01dc2f7e06731b05cefe96c421a5cd0>
(500)
Patch Links:
- https://github.com/pysal/access/pull/50.patch
- https://github.com/pysal/access/pull/50.diff
—
Reply to this email directly, view it on GitHub
<#50>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADILY7ZWP2NK33C6WDJIYVD26JMMDAVCNFSM6AAAAAB5BUTPYGVHI2DSMVQWIX3LMV43ASLTON2WKOZTGA3DCMRRHA4TSOI>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
|
This change looks fine to me, though I might drop the "research assistant" part --
since Dan has not been a research assistant in quite some time! However, it looks like the CI has also completely died in the interim. If @jGaboardi can point to how to get your current CI setup going (#48), I'd appreciate it. I owe another update here for #51, but have very little time to dedicate to this right now. |
|
This hitch to this will be first finishing out the modernizing efforts that have languished these past year12. Also, since Unless I am overthinking this? Thoughts @martinfleis @knaaptime @gegen07 Footnotes |
|
@jGaboardi fair observations imho. |
|
Give a thumbs up here and in my free time I will start creating tickets & pushing PRs to move towards resolve the stuff above. In order for this be as efficient as possible, I'd ask that if you don't have the bandwidth to review PRs then any other PySAL core dev's approval is sufficient. |
jGaboardi
left a comment
There was a problem hiding this comment.
approved pending @JamesSaxon 's re-review and input here.
|
Thanks @jGaboardi,
In general, but also especially for infra updates, I trust any pysal owner to approve PRs. That said, I'm generally happy to do PRs and get to them promptly, I just don't have the bandwidth to get this CI going again. Once the CI is running reliably, I'd be happy to make a few quick fixes and clean-ups. |
|
Barring any objections, who is squash/merging here? @dfsnow, do you want to do the honors? |
Co-authored-by: James Gaboardi <[email protected]>
|
I don't think @dfsnow has the merge rights :). |
Hi @sjsrey. I'm Dan Snow, the research assistant that created the original pre-calculated travel times listed on the access Resources page.
I've since graduated and moved on to different work, but travel times have stayed in my heart, so I created OpenTimes, a free database of travel times between all major Census geographies and years.
I'd love to replace these old pre-calculated times with ones from OpenTimes. The OpenTimes times are easier to produce, easier to access, more accurate, and have much better coverage. They'll also be free in perpetuity. They're just a fun side project for me and a way to keep tinkering with travel times.
If nothing else, we should remove the defunct interactive map from the docs on this page.
Thanks!