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

Rename and added a robust error handling functionality to the R/assign_job_queue.R file functions #73

Closed
wants to merge 3 commits into from

Commits on Oct 5, 2024

  1. usethis::pr_init("Implement error handling in acc2lin.R functions

    - Added validation checks for input parameters (accessions, ipg_file, assembly_path, lineagelookup_path).
    - Included error messages for missing or invalid inputs and file existence checks.
    - Wrapped main logic in tryCatch for graceful error handling during execution.
    ")
    Seyi007 committed Oct 5, 2024
    Configuration menu
    Copy the full SHA
    30d4bf3 View commit details
    Browse the repository at this point in the history

Commits on Oct 7, 2024

  1. Add error handling to multiple functions

    - Implement error handling for mapOption2Process, get_proc_medians, write_proc_medians_table, get_proc_weights, advanced_opts2est_walltime, assign_job_queue, and plot_estimated_walltimes .
    - Validate input arguments for each function to ensure they meet expected criteria.
    - Use tryCatch blocks to gracefully handle errors and warnings.
    - Provide informative error messages and detailed logging where appropriate.
    - Ensure functions fail gracefully and provide useful feedback.
    
    Also renamed the functions to the following;
    assign_job_queue -> assignJobQueue
    make_opts2procs	-> mapOption2Process
    map_advanced_opts2procs	-> mapAdvOption2Process
    get_proc_medians - calculateProcessRuntime
    write_proc_medians_table -> writeProcessRuntime2TSV
    write_proc_medians_yml -> writeProcessRuntime2YML
    get_proc_weights -> getProcessRuntimeWeights
    advanced_opts2est_walltime -> calculateEstimatedWallTimeFromOpts
    plot_estimated_walltimes -> plotEstimatedWallTimes
    Seyi007 committed Oct 7, 2024
    Configuration menu
    Copy the full SHA
    4aeaa11 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    8798c2f View commit details
    Browse the repository at this point in the history