-
Notifications
You must be signed in to change notification settings - Fork 35
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
"KeyError: key ([#],[#]) not found" #260
Comments
I'm able to reproduce this. This seems to be an issue with parallel processing plus include_pairs. It did run properly in serial, but I needed to restart Julia to get it to work. For some reason, if you try to run in parallel and get this error, running in serial doesn't work until you restart Julia. Very strange (concerning) behavior. Summary of observations:
@slamander for now, because your resistance grid is so small, I would suggest running in serial (it is extremely fast) by setting @ranjanan is probably the person to look at this (I think he's dealt with some similar issues in the past). I'm just not familiar with the portions of the code that handle parallel processing or include_pairs. I know @ranjanan is very busy these days so he may not have time to look at this for while. |
Thank again, @vlandau. These are all very helpful notes, and--as you mentioned--since this is a very small job (although it is one in several hundred grids I'm applying this too), I'm more than satisfied with this solution. Stay well out there, -Alex. |
I'm leaving this open just so we can properly debug, but glad it's workable for you for now at least! |
@vlandau We may want to automatically pick cholmod for small problem sizes, and switch to cg+amg on larger ones. What do you think? |
@ViralBShah I like that idea. I'm planning to do something similar for Omniscape -- once CHOLMOD gets implemented for advanced mode :) (or once the PARDISO solver gets released I might use that if it's comparable in performance) |
Pardiso wasn't giving better enough performance, from what I could tell in the open PR. Unless you find it does on certain problems, I would suggest sticking with cholmod. |
@slamander the include pairs file is missing from your link (CMR_CAF.txt). Could you please add that to your Google Drive link? |
Hi, @ranjanan. Thanks for the help! Sorry, I made the mistake of altering the files in this folder while troubleshooting upstream aspects of my workflow... If my alterations produce the same error, I'll be sure to post again. If this same error doesn't throw in my updated workflow, I'll try to work backward to recreate it. In the future, I'll be sure to preserve the code and files for recreating issues when posted here. |
Alright, no problem. Would you like to reopen this issue if you encounter it again? |
Hey, @ranjanan & @ViralBShah. Sorry, I must not have hit 'comment' when I tried to respond earlier. That sounds like a good idea. I'll let you both know what happens after I complete my workflow. |
I believe I still have the files to reproduce. I can post them and reopen if it's okay with @slamander . |
@vlandau Fine by me! |
AZE_ARM_CS_inputs.zip |
@vlandau did you miss uploading the resistance surface? |
Ah shoot, sorry. I uploaded the .out file instead of the resistance surface.... this archive should contain everything. |
I don't see this error anymore here, and I can see an output. @vlandau if you get a minute, could you try CS on the latest version of these files to see if the output is correct? |
Not sure if this is relevant, but I was running Julia through R and needed to reconfigure my interfacing library to alleviate this issue. I can try to provide more details if necessary. |
@slamander nice to hear this issue is alleviated. Can you elaborate on the changes? Did you update your R and Julia and JuliaCall? |
Hello, CS community!
As a continuation from issue #258, I've encountered another CS error for which I do not know how to address. See error message pasted below:
The relevant files (nodes & resistance layers, init file, and included pair list) are here.
Thanks in advance for any help or explanation.
Best,
-Alex.
The text was updated successfully, but these errors were encountered: