Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cleaned up basic-infection. Fixes #239. #274

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

RobertJacobsonCDC
Copy link
Collaborator

Moved disease_model from the docs to examples and made a symlink in the docs.

The disease_model example replaces basic-infection; basic-infection is still used as a benchmark.

It is probably wise to wait to merge this until #214 and #215 are merged, because this PR will have to modify changes to workflows modified by those PRs.

…he docs.

The disease_model example replaces basic-infection; basic-infection is still used as a benchmark.
…ommit adds enough `allow` annotations to get them to pass.
@RobertJacobsonCDC
Copy link
Collaborator Author

Some frustrating issues:

  • Because of limitations in Cargo, we cannot run disease_model with cargo run --example disease_model, because disease_model is itself structured as a package.
  • Clippy breaks if I include disease_model as a binary in Cargo.toml ([[bin]] name = disease_model). In particular, it doesn't "see" dev dependencies.
  • Clippy goes bananas on the benchmark's test modules. I added allow to everything in the test modules to get them to pass because of false positives.

I think structuring the project will fix these issues, but I'd have to try it to be sure.

@RobertJacobsonCDC RobertJacobsonCDC added the blocked Currently blocked by another issue label Mar 12, 2025
@RobertJacobsonCDC RobertJacobsonCDC marked this pull request as ready for review March 17, 2025 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Currently blocked by another issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant