Skip to content

Change order of covariates,#11

Merged
nickreich merged 1 commit intomainfrom
ngr/fix-10
Oct 2, 2025
Merged

Change order of covariates,#11
nickreich merged 1 commit intomainfrom
ngr/fix-10

Conversation

@nickreich
Copy link
Copy Markdown
Member

Fixes #10

Copy link
Copy Markdown
Member

@matthewcornell matthewcornell left a comment

Choose a reason for hiding this comment

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

LGTM!

@nickreich
Copy link
Copy Markdown
Member Author

Thanks for the vote of confidence, @matthewcornell . Is the failing check a concern?

@nickreich nickreich changed the title Changer order of covariates, Change order of covariates, Oct 1, 2025
@matthewcornell
Copy link
Copy Markdown
Member

matthewcornell commented Oct 1, 2025

Is the failing check a concern?

Good catch. Looking back at https://github.com/reichlab/idmodels/actions , I see that the only passing test was the first commit by Evan on Nov 6, 2024 . All others have failed, presumably the same tests (Action history is not kept for a lot of the runs.) So I don't think your change was the culprit. I'll followup up on slack to see if anyone wants to pair on fixing this.

@nickreich
Copy link
Copy Markdown
Member Author

Merging based on comments from @elray1 @matthewcornell @trobacker that suggest this is a known failure and we're just going to 🤞 🙈

@nickreich nickreich merged commit 44692ec into main Oct 2, 2025
1 check failed
@nickreich nickreich deleted the ngr/fix-10 branch October 2, 2025 20:25
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.

fix bug in order of arguments

2 participants