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

add option to tile XXX_DATA to argo.addcol() #23

Open
SpheMakh opened this issue Dec 24, 2014 · 2 comments
Open

add option to tile XXX_DATA to argo.addcol() #23

SpheMakh opened this issue Dec 24, 2014 · 2 comments
Assignees
Milestone

Comments

@SpheMakh
Copy link
Contributor

As requested by @o-smirnov:
The xxx_DATA columns can be "tiled" for performance reasons. I don't know if there's a way to create a tiled column from pyrap, at least there wasn't one at the time, so I wrote the addtiedmscol utility (in C++) to do this. You might want to integrate that into addcol() somehow.

@SpheMakh SpheMakh self-assigned this Dec 24, 2014
@o-smirnov
Copy link
Contributor

Come to think of it, doesn't the add column functionality belong in ms.py?
On 24 Dec 2014 12:29, "Sphesihle Makhathini" [email protected]
wrote:

As requested by @o-smirnov https://github.com/o-smirnov:
The xxx_DATA columns can be "tiled" for performance reasons. I don't know
if there's a way to create a tiled column from pyrap, at least there wasn't
one at the time, so I wrote the addtiedmscol utility (in C++) to do this.
You might want to integrate that into addcol() somehow.


Reply to this email directly or view it on GitHub
#23.

@SpheMakh
Copy link
Contributor Author

Yep, it does. I will move it there when I fix this issue. But maybe I should keep a mapping to the function in argo.py, else stuff that uses it will break.

@gijzelaerr gijzelaerr modified the milestone: v1.5.0 Feb 10, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants