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

User defined date specificity #46

Open
mikeyweigand opened this issue Mar 19, 2024 · 1 comment
Open

User defined date specificity #46

mikeyweigand opened this issue Mar 19, 2024 · 1 comment

Comments

@mikeyweigand
Copy link

Is your feature request related to a problem? Please describe.
Hard-coded date formatting at YYYY-MM-DD creates challenges for generalizing to other microbial pathogens, the majority of which must be submitted to BioSample with only YYYY or YYYY-MM to ensure privacy.

Describe the solution you'd like
Consider letting users define their own date specificity, perhaps in the *_config.yaml. That would preserve the current default requirements for SC2 and Flu. A more advanced option would be to allow setting a minimum (or maximum) specificity rather than a fixed requirement for flexibility during submission (e.g. [1] YYYY or YYYY-MM, but not YYYY-MM-DD vs [2] YYYY-MM or YYYY-MM-DD, but not YYYY).

Describe alternatives you've considered
Maybe this also gets covered in your solution to #43 but BioSample itself does not impose strict requirements for date specificity and it's generally up to submitters to determine what is appropriate.

Additional context
Add any other context or screenshots about the feature request here.

@dthoward96
Copy link
Collaborator

Hey,

Yes, I've noticed this and am working on resolving it. You are correct in the issue you tagged will resolve this problem as the pandera schema's will be used to validate the other database fields as well. I've moved that feature to be added in the next version update to resolve other issues but it will also resolve this one as well. I'm expecting to have it live on the version update branch later this week. After the changes are live and testing has been done that version should be going live to master shortly then after.

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

No branches or pull requests

2 participants