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

fixed repeat function #33

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fixed repeat function #33

wants to merge 1 commit into from

Conversation

pazs10ve
Copy link

This PR improves the Data Mapping Analysis script by:

  1. Adding a main() function for better organization
  2. Implementing command-line arguments for CSV filename and columns
  3. Removing hardcoded values for improved modularity
  4. Enhancing error handling and logging

Usage:
python script_name.py mamo_patients_mapping_data.csv anonymized_patient_id real_patient_id

The script is now more adaptable to different datasets without code changes. Please review and provide feedback.

@FMCalisto FMCalisto self-assigned this Oct 20, 2024
@FMCalisto FMCalisto added enhancement Type: enhancement when we have a new request to improve something. info Status: signifies issues or PRs awaiting further details from the submitter. low Priority: low priority issue. labels Oct 20, 2024
@FMCalisto FMCalisto added this to the 1.1 milestone Oct 20, 2024
@FMCalisto FMCalisto linked an issue Oct 20, 2024 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Type: enhancement when we have a new request to improve something. info Status: signifies issues or PRs awaiting further details from the submitter. low Priority: low priority issue.
Projects
Status: In progress
Development

Successfully merging this pull request may close these issues.

Streamline Data Pipeline Deployment Process
2 participants