Fix ps_point reserve capacity#3392
Conversation
Jenkins Console Log Machine informationNo LSB modules are available. Distributor ID: Ubuntu Description: Ubuntu 20.04.3 LTS Release: 20.04 Codename: focalCPU: G++: Clang: |
|
I know we can't trust one run of Jenkins for benchmarks. I am also aware
that small improvements in C++ language can yield great performance
improvements if we're using composing FLOPs and these C++ typing/language
improvements can reduce technical debt. Especially, since MCMC uses
repeated operations in a for loop, over thousands of iterations.
I'm not the main C++ guy, and long term this could be helpful WRT techincal
debt, but the huge increase in gp_regre.stan is a huge red flag, since the
largest bottleneck in computing GP models is the cholesky decomposition,
and the problem with Cholesky decomposition is repeated matrix
algebra/FLOPs.
So perhaps, somehow, the previous change added some computational
complexity?
More generally, is there any codified/vetted documentation of benchmarking
C++/C implementations and C from a computational perspective, not just
theoretical? I'm on hiatus but I am watching.
Best,
Andre Zapico
linkedin.com/in/andre-zapico
gitub.com/drezap
ME Information and Communication Engineering
University of Electronic Science and Technology of China
Consultant, Owner
likely llc
likelyllc.com
Stan Developer
mc-stan.org
BS Mathematical Sciences: Probabilistic Methods
BS Statistics
University of Michigan, Ann Arbor 2017
…On Thu, May 14, 2026 at 2:18 PM Brian Ward ***@***.***> wrote:
***@***.**** approved this pull request.
—
Reply to this email directly, view it on GitHub
<#3392 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACY543FYNK3XU2HNGMZDPX342YEWPAVCNFSM6AAAAACY3GSXY2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHM2DEOJSGM2TQNJZHE>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Jenkins Console Log Machine informationNo LSB modules are available. Distributor ID: Ubuntu Description: Ubuntu 20.04.3 LTS Release: 20.04 Codename: focalCPU: G++: Clang: |
|
@WardBrian reran this and the jenkins probably just had a hiccup because this was not replicated locally |
|
May be I am wrong
Best,
Andre Zapico
linkedin.com/in/andre-zapico
gitub.com/drezap
ME Information and Communication Engineering
University of Electronic Science and Technology of China
Consultant, Owner
likely llc
likelyllc.com
Stan Developer
mc-stan.org
BS Mathematical Sciences: Probabilistic Methods
BS Statistics
University of Michigan, Ann Arbor 2017
…On Fri, May 15, 2026 at 4:59 PM Steve Bronder ***@***.***> wrote:
*SteveBronder* left a comment (stan-dev/stan#3392)
<#3392 (comment)>
@WardBrian <https://github.com/WardBrian> reran this and the jenkins
probably just had a hiccup because this was not replicated locally
—
Reply to this email directly, view it on GitHub
<#3392 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACY543EF2JXTXX35DV4SVQT426AMNAVCNFSM6AAAAACY3GSXY2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHM2DINRTGU4DAMZZHA>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
|
But if we set the seed and we’ve only modified one thing shouldn’t the
result be the same? So there’s something we’re not controlling for, what am
I missing?
Best,
Andre Zapico
linkedin.com/in/andre-zapico
gitub.com/drezap
ME Information and Communication Engineering
University of Electronic Science and Technology of China
Consultant, Owner
likely llc
likelyllc.com
Stan Developer
mc-stan.org
BS Mathematical Sciences: Probabilistic Methods
BS Statistics
University of Michigan, Ann Arbor 2017
…On Fri, May 15, 2026 at 6:54 PM Andre Zapico ***@***.***> wrote:
May be I am wrong
Best,
Andre Zapico
linkedin.com/in/andre-zapico
gitub.com/drezap
ME Information and Communication Engineering
University of Electronic Science and Technology of China
Consultant, Owner
likely llc
likelyllc.com
Stan Developer
mc-stan.org
BS Mathematical Sciences: Probabilistic Methods
BS Statistics
University of Michigan, Ann Arbor 2017
On Fri, May 15, 2026 at 4:59 PM Steve Bronder ***@***.***>
wrote:
> *SteveBronder* left a comment (stan-dev/stan#3392)
> <#3392 (comment)>
>
> @WardBrian <https://github.com/WardBrian> reran this and the jenkins
> probably just had a hiccup because this was not replicated locally
>
> —
> Reply to this email directly, view it on GitHub
> <#3392 (comment)>, or
> unsubscribe
> <https://github.com/notifications/unsubscribe-auth/ACY543EF2JXTXX35DV4SVQT426AMNAVCNFSM6AAAAACY3GSXY2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHM2DINRTGU4DAMZZHA>
> .
> You are receiving this because you commented.Message ID:
> ***@***.***>
>
|
|
The server we run on can just randomly hiccup so it's good to run tests locally if something anomalous happens. The performance benchmarks that Jenkins posts are much more of a canary than a definitive yes/no |
Submission Checklist
./runTests.py src/test/unitmake cpplintSummary
Fixes #3384 and adds
names.size()to string reserveCopyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company): Simons Foundation
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses: