-
Notifications
You must be signed in to change notification settings - Fork 16
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
Empty arguments to cubical confuse the Stimela wrapper #672
Comments
@Arnabiitindore, could you please post the log ? |
This is the recipe I used: INPUT="input" ms12 = "Radio_halo_12SEP.ms" recipe = stimela.Recipe("Radio halo selfcal with cubical", recipe.add("cab/cubical", "calibrate 1st ms with cubical", { ##data## 'data-ms':ms12, ##sel## 'sel-field':0, model'model-list':spf('MODEL_DATA'), out#'out-':'selfcal', weight'weight-column':'WEIGHT_SPECTRUM', degridding'degridding-OverS':11, sol'sol-jones':'G', log#'log-memory': True, debug#'debug-pdb': False, g'g-solvable': True, }, input=INPUT, output=OUTPUT, label="cubical on 1st ms") STEPS = ["cubical on 1st ms"] |
|
Yeah the workaround is to set both to The problem is here: https://github.com/ratt-ru/Stimela/blob/master/stimela/cargo/cab/cubical/src/run.py#L57. @SpheMakh only half-scabhafied the runscript. The join/shlex.split pattern is Ancient Evil that needs to be eradicated with fire (this problem here is an example of why). Just pass the argument list to |
There seems lots of other issues:
Has anyone tested cubical with stimela before? Or am I missing something? The log.txt are attached. The recipe: INPUT="input" ms12 = "Radio_halo_12SEP.ms" recipe = stimela.Recipe("Radio halo selfcal with cubical", recipe.add("cab/cubical", "calibrate 1st ms with cubical", { ##data## 'data-ms':ms12, ##sel## 'sel-field':0, model'model-list':spf('MODEL_DATA'), out#'out-':'selfcal', weight'weight-column':'WEIGHT_SPECTRUM', madmax'madmax-enable':False, bbc'bbc-load-from':None, degridding'degridding-OverS':11, sol'sol-jones':'G', log'log-memory': True, debug'debug-pdb': False, g'g-solvable': True, }, input=INPUT, output=OUTPUT, label="cubical on 1st ms") STEPS = ["cubical on 1st ms"] |
Hmmm I was looking at master, but of course you're probably using an older release. What's your version? It is true that not all CubiCal options are propagated into the Stimela wrapper, probably out of laziness (or a sense that they weren't really needed). If you spot one you need, please raise it here or on an issue. And yes, some of these options will have no default (i.e. an empty string by default), but if they're not exposed by Stimela, then no harm, no foul. Yes, CubiCal is used with Stimela, and routinely at that. So we just have to figure out your version/config problems. |
stimela 1.6.5 Thanks for the help. |
You can |
Yes, I have done it from the master. |
Ah good! Feel free to push to a branch with the fix then. |
It started and sunning well. But then I am getting a error: The log file is attached. Not sure what is the problem here. The option I used : } |
It looks like the I/O worker just fell over unceremoniously. My first guess would be out of memory -- how much RAM have you got on the box? Try decreasing --data-time-chunk to load less data at once (generally, time chunk should be a multiple of the solution interval i.e. --g-time-int, your interval is 16 so you could even set the time chunk to be as low as 16). |
Hi, |
Very glad to hear that! |
Actually let's leave this open, as the caveat of the empty arguments causing a mysterious error remains... |
Hi,
I am trying to run selfcal using cubical. But getting some unexpected error. Not sure what is the problem.
check the screenshot.
The text was updated successfully, but these errors were encountered: