Skip to content

don't compute predictions for all replicas if we only want replica0 - #2004

Merged
RoyStegeman merged 4 commits into
masterfrom
rs-quickfix
Mar 14, 2024
Merged

don't compute predictions for all replicas if we only want replica0#2004
RoyStegeman merged 4 commits into
masterfrom
rs-quickfix

Conversation

@RoyStegeman

Copy link
Copy Markdown
Member

No description provided.

@RoyStegeman
RoyStegeman requested a review from scarlehoff March 14, 2024 10:39
@RoyStegeman
RoyStegeman force-pushed the rs-quickfix branch 2 times, most recently from 5c76ad1 to 99dc6a5 Compare March 14, 2024 10:46
@RoyStegeman
RoyStegeman requested a review from andreab1997 March 14, 2024 11:29
Comment thread validphys2/src/validphys/results.py Outdated

@andreab1997 andreab1997 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems fine but, remind me, what is it used in the thcovmat creation? I remember we faced this issue some months ago as well

@RoyStegeman

RoyStegeman commented Mar 14, 2024

Copy link
Copy Markdown
Member Author

Indeed in the function that enables me to do this is the on created to make the thcovmat computation cheaper: #1863.

@andreab1997 this function derives from the same function, but specifically this is the one in the alphas TCM notebook. So it should make computing the posterior quite a bit lighter and faster.

@RoyStegeman
RoyStegeman merged commit 3b228eb into master Mar 14, 2024
@RoyStegeman
RoyStegeman deleted the rs-quickfix branch March 14, 2024 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants