Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch data source to ta_path #183

Open
wants to merge 1 commit into
base: deploy
Choose a base branch
from
Open

Switch data source to ta_path #183

wants to merge 1 commit into from

Conversation

Nate-Wessel
Copy link
Contributor

Resolves #182

@Nate-Wessel
Copy link
Contributor Author

Want to now compare a bunch of random requests and make sure there's no systematic bias in either direction. I don't expect any.

@Nate-Wessel
Copy link
Contributor Author

Nate-Wessel commented Jan 16, 2025

To compare the results of the two branches:

WITH ta_path AS (
    SELECT
        uri_string,
        (results->'results'->'travel_time'->'seconds')::real AS seconds
    FROM nwessel.cached_travel_times
    WHERE commit_hash ~ '5b8f'
), 

ta AS (
    SELECT
        uri_string,
        (results->'results'->'travel_time'->'seconds')::real AS seconds
    FROM nwessel.cached_travel_times
    WHERE commit_hash ~ 'fd2a'
)

SELECT
    exp(avg(ln(ta.seconds / ta_path.seconds)))
FROM ta
JOIN ta_path USING (uri_string)

Average difference so far looking like ~1%. Can test this further by just running more of the same queries on each commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Switch from here.ta to here.ta_path
1 participant