Skip to content

ComputeStatsThread parallelism - #386

Merged
jteresco merged 4 commits into
TravelMapping:masterfrom
yakra:cs2_merge
Dec 17, 2020
Merged

ComputeStatsThread parallelism#386
jteresco merged 4 commits into
TravelMapping:masterfrom
yakra:cs2_merge

Conversation

@yakra

@yakrayakra commented Dec 16, 2020

Copy link
Copy Markdown
Contributor

Closes#366.
A little increased overhead Augmenting travelers for detected concurrent segments pays off with more time savings when Computing stats.


stat2019
Same commits used in all the other speed graphs.


stat2020
Data from like a week ago. Wow, we've added a fair amount of data in just this one year. :D


Some minor changes also made it into siteupdate.py -- removing a redundant safety net when computing stats saves us about 4s.

@yakra

Copy link
Copy Markdown
ContributorAuthor

This branch is fine in theory, with no diffs in output from master.
The CodeOrg branch where I'm working on #378 is fine in theory, with no diffs in output from master.
When I merge them, something breaks and the output differs.
I must be doing the merge wrong; closing until I can figure this out.

@yakrayakra closed this Dec 16, 2020
@yakra

Copy link
Copy Markdown
ContributorAuthor

Whatever the issue is has cleared up. Must have resolved the merge conflict differently the 1st time; it was a tricky one.

@yakrayakra reopened this Dec 16, 2020
@jteresco
jteresco merged commit c064c0e into TravelMapping:masterDec 17, 2020
@yakra
yakra deleted the cs2_merge branch December 18, 2020 03:38
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ComputeStatsThread parallelism

2 participants

@yakra@jteresco