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

Expose MeasurementSet functionality #61

Closed
sjperkins opened this issue Mar 31, 2016 · 5 comments
Closed

Expose MeasurementSet functionality #61

sjperkins opened this issue Mar 31, 2016 · 5 comments
Milestone

Comments

@sjperkins
Copy link
Contributor

It would be helpful if requiredTableDesc was exposed in the python wrappers so that access to the basic table configuration was available for creating new tables.

Had a quick go at adding it in pytables.cc but requiredTableDesc doesn't live on TableProxy (looks like its a static method on MeasurementSet).

@gervandiepen
Copy link
Contributor

pytables and TableProxy form a generic casacore table interface, while
requiredTableDesc is a MeasurementSet-specific function. However, I can see
it is useful. I'll think about it. Maybe a python interface to MS
functionality is needed.

On Thu, Mar 31, 2016 at 12:41 PM, Simon Perkins notifications@github.com
wrote:

It would be helpful if requiredTableDesc was exposed in the python
wrappers so that access to the basic table configuration was available for
creating new tables.

Had a quick go at adding it in pytables.cc but requiredTableDesc doesn't
live on TableProxy (looks like its a static method on MeasurementSet).


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#61

@sjperkins
Copy link
Contributor Author

Hypothetically speaking, would such an interface also have access to methods such as createDefaultSubtables? Perhaps the issue needs to be reworded.

@gervandiepen
Copy link
Contributor

Yes, it would.

On Thu, Mar 31, 2016 at 1:04 PM, Simon Perkins notifications@github.com
wrote:

Hypothetically speaking, would such an interface also have access to
methods such as createDefaultSubtables?


You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#61 (comment)

@sjperkins sjperkins changed the title Expose requiredTableDesc Expose MeasurementSet functionality Mar 31, 2016
@brentjens
Copy link

Exposing the MS functionality would help greatly in e.g. writing an MS filler for LOFAR station visibility data too.

@gijzelaerr
Copy link
Member

this has been implemented in this PR #72

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants