Skip to content

Move contact matrix transpose inside model functions - #271

Merged
joshwlambert merged 23 commits into
mainfrom
bahadzie/issue259
Jul 19, 2026
Merged

Move contact matrix transpose inside model functions#271
joshwlambert merged 23 commits into
mainfrom
bahadzie/issue259

Conversation

@bahadzie

@bahadzie bahadzie commented Jul 30, 2025

Copy link
Copy Markdown
Member

Fixes #259

@bahadzie
bahadzie requested a review from adamkucharski July 30, 2025 18:12
@bahadzie bahadzie changed the title Bahadzie/issue259 Move contact matrix transpose inside model function? Jul 30, 2025
@bahadzie bahadzie self-assigned this Aug 6, 2025

@adamkucharski adamkucharski left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks – this looks fine to me, but tagging @avallecam before merged in case influences any ongoing training.

@avallecam

Copy link
Copy Markdown
Member

Thanks – this looks fine to me, but tagging @avallecam before merged in case influences any ongoing training.

Thank @adamkucharski , @bahadzie could we wait for one more week? this week we run {epidemics}

@avallecam

avallecam commented Aug 25, 2025

Copy link
Copy Markdown
Member

Hi @bahadzie, now we should be in a better position to incorporate this feature and update all the dependent materials in tutorials and howto's.

Also, given the scale of current dissemination, we should plan a blog, showcase or other event to update users, trained and engaged audience on this. @adamkucharski

@joshwlambert joshwlambert changed the title Move contact matrix transpose inside model function? Move contact matrix transpose inside model functions Jul 2, 2026
@joshwlambert

Copy link
Copy Markdown
Member

Thanks @bahadzie for making these changes and @adamkucharski for reviewing. I've rebased this feature branch onto main as there were some other feature branches merged into main in the mean time. There were a couple of other minor changes I've made. Overall these changes LGTM.

I'll now merge this PR.

@joshwlambert

Copy link
Copy Markdown
Member

The dependency-change.yaml workflow is failing on GitHub actions, but the error seems to be unrelated to the changes in this PR so I'll move forward with merging.

@joshwlambert
joshwlambert merged commit 36b8950 into main Jul 19, 2026
10 of 11 checks passed
@joshwlambert
joshwlambert deleted the bahadzie/issue259 branch July 19, 2026 21:08
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.

Move contact matrix transpose inside model function?

5 participants