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

CDAT Migration Phase 1: Replace cdp.cdp_run #426

Closed
forsyth2 opened this issue Mar 29, 2021 · 1 comment · Fixed by #641
Closed

CDAT Migration Phase 1: Replace cdp.cdp_run #426

forsyth2 opened this issue Mar 29, 2021 · 1 comment · Fixed by #641
Assignees
Labels
cdat-migration-fy24 CDAT Migration FY24 Task

Comments

@forsyth2
Copy link
Collaborator

As mentioned on #425, it may be a good idea to write our own multiprocessing code rather than using the cdp code. This would allow us to write finer-grained error messages (such as which sets in particular failed).

@tomvothecoder
Copy link
Collaborator

Related to #425 (comment)

@tomvothecoder tomvothecoder changed the title Improve multiprocessing code CDAT Migration Phase 1: Replace cdp.multiprocessing Dec 12, 2022
@tomvothecoder tomvothecoder changed the title CDAT Migration Phase 1: Replace cdp.multiprocessing CDAT Migration Phase 1: Replace cdp.cdp_run Dec 15, 2022
@tomvothecoder tomvothecoder added refactor cdat-migration-fy24 CDAT Migration FY24 Task and removed nice to have Lower priority task refactor labels Sep 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cdat-migration-fy24 CDAT Migration FY24 Task
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants