-
Notifications
You must be signed in to change notification settings - Fork 7
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
add 3D aerosol species mixing ratio #262
base: master
Are you sure you want to change the base?
Conversation
Posting discussion with @mingxuanwupnnl to here, two issues were raised during testing this PR.
The script used is attached:
|
I tried to look it up the error:"a process in the process pool was terminated abruptly". It seem to suggest there is problem spawning subprocesses with |
I don't remember if I ran into the Here are a few possible reasons why this error is raised:
I'll be on holiday and vacation the rest of the week and traveling all of next week. |
The error message "a process in the process pool was terminated abruptly" does not appear to be output by e2c directly, It would be good to have context (the surrounding e2c info/error messages) to better understand where the error occurs. Likewise, the first set of errors are output by the CMOR module. I think the expertise of Charlie or Xylar might be more familiar with these sort of error messages. I don't think I've run into them before. I'm guessing the the process-pool termination is a conseguence of the earlier CMOR errors, I note many of these module errors tend to return "0" (All is A-OK) to the calling process, which then tries valiantly to continue but fails. |
@mingxuanwupnnl I did some more testing, it does look like a memory issue. If running the e3sm_to_cmp command with an additional flag |
I would run a debugger to find the line that crashes with A random thought, but maybe this multiprocessing issue is similar to |
Hi @chengzhuzhang Thank you for bringing this up. I was distracted by other tasks at that time. I wonder how we can proceed. For this PR, how about we just do for 3D aerosol mass mixing ratio? I remember that If I just do two vars a time, it works. If there are three or more vars to be converted, then it will crash. I don't have ideas on debug this issue related to memory. Or we can leave it for now and do 2D variables first. |
@mingxuanwupnnl thank you for following up this PR. Sorry that we just get to revive this after so long. I think your PR is good as-is, the performance bottleneck seems to be around with the e3sm_to_cmip code base but was not reported. You can keep update this PR for additional aerosol variables, or if it is preferred, you could start a new PR for new additions. |
@mingxuanwupnnl Thank you. I will review and test when NERSC is back on line. |
Add aerosol species mixing ratio, inlcuding:
mmrbc, mmroa, mmrsoa, mmrdust, mmrss, mmrso4