-
Notifications
You must be signed in to change notification settings - Fork 3
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
MS is not conformant #9
Comments
Hi sorry for not replying your email before - I'll send you an invite to the kMS/DDF development repo to try it with a newer version and discuss the issue... |
Hey Sarah - check this out: https://github.com/cyriltasse/killMS/issues/40 |
Hi, |
It is raised when not all baselines are present for all times (ie not all the data is on a regular grid). It could be caused by CASA split set to remove fully flagged rows for instance. You will need to repad your data to contain these missing rows (or resplit with the flag set to false) |
For now, I just commented out the shape checking portion of the kMS code and it seems to run happily after that. Not sure why it would stop everything when the rest of the calibration works just fine. |
@cyriltasse should comment but I think this breaks assumptions further down in the code. It is probably not safe to comment the check |
Yeah check the results very carefully. There are many ways to be non-conformant, and not all of them are necessarily fatal, so the check you commented out may well be a little over-zealous at times... but you're on shaky ground. |
Noted. I'll compare source measurements after I finish processing.
…________________________________
From: Oleg Smirnov ***@***.***>
Sent: Tuesday, June 6, 2023 11:10 AM
To: saopicc/killMS ***@***.***>
Cc: Sarah Chastain ***@***.***>; Author ***@***.***>
Subject: Re: [saopicc/killMS] MS is not conformant (#9)
[EXTERNAL]
Yeah check the results very carefully. There are many ways to be non-conformant, and not all of them are necessarily fatal, so the check you commented out may well be a little over-zealous at times... but you're on shaky ground.
—
Reply to this email directly, view it on GitHub<#9 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AF35E67EQ3GS5BKJZTG57ETXJ5QAJANCNFSM4S2DKJGQ>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
Yeah - i can't remember why but there was some reason to request for this... I could dig again if it doesn't work for you @dentalfloss1 |
Hi,
I am attempting to run killMS and I see the following pasted below. I am unsure how to fix this. Attached are the log files.
kilMS-373073-err.txt
kilMS-373073-out.txt
The text was updated successfully, but these errors were encountered: