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

Planting the seeds for a vospace module #528

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

andamian
Copy link
Contributor

@andamian andamian commented Feb 23, 2024

I started with something very simple - read a node and read a container node and its entire content. There's a lot of functionality missing but initially I'm seeking comments general comments regarding the data representation as well as style and overall functionality. There's already a stand-alone VOSpace Python client (https://github.com/opencadc/vostools/tree/master/vos). This implementation aims to re-use as much as possible the existing PyVO infrastructure.
Even if it's work in progress, I still welcome early feedback.

@msdemlei
Copy link
Contributor

msdemlei commented Feb 23, 2024 via email

@andamian
Copy link
Contributor Author

Yes, it should be able to resolve URIs of the form vos://example.com!vospace/container-6/siap-out-1.vot?foo=bar via dismantle, reg lookup and reassemble which should be quite cool if it works.

Our existing tool installs a number of CLI commands such as vls, vcm etc (similar to their linux counterparts or to the scp remote one). These tools are very popular with our users. Do you see that as being outside the scope of PyVO which is just a library?

@msdemlei
Copy link
Contributor

msdemlei commented Mar 4, 2024 via email

@bsipocz
Copy link
Member

bsipocz commented Apr 18, 2024

I seem to remember astropy came with a few command line programs,
even if I now can't find them any more.

They are here:
https://github.com/astropy/astropy/blob/main/pyproject.toml#L114

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

Successfully merging this pull request may close these issues.

None yet

3 participants