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

Handling of case column in linelist data class #413

Open
athowes opened this issue Nov 1, 2024 · 0 comments
Open

Handling of case column in linelist data class #413

athowes opened this issue Nov 1, 2024 · 0 comments
Labels
medium Nice to have for next release

Comments

@athowes
Copy link
Collaborator

athowes commented Nov 1, 2024

Following PR #390, at the moment as_epidist_linelist does not require there to be a case ID column.

At the moment, the models in the package do not use a case column.

That said, a case column would be useful for plotting functionality.

Our options:

  1. Enforce users providing a case column
  2. Don't expect a case column and don't create one
  3. Create a case column if not provided one (e.g. in the order of the data)
@athowes athowes added the medium Nice to have for next release label Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
medium Nice to have for next release
Projects
None yet
Development

No branches or pull requests

1 participant