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

Adding get_size_on_remote to the Transport interface #6665

Open
GeigerJ2 opened this issue Dec 12, 2024 · 1 comment
Open

Adding get_size_on_remote to the Transport interface #6665

GeigerJ2 opened this issue Dec 12, 2024 · 1 comment
Labels
type/feature request status undecided

Comments

@GeigerJ2
Copy link
Contributor

As noted during discussions with the core office, @agoscinski, @khsrali, @unkcpz, on #6584:
It would be nice if the Transport could implement a method such as get_size or get_size_on_remote to which a path can be provided (as well as additional endpoints, if we think of more in the future).

In #6584, some custom logic to be added because FirecREST doesn't support arbitrary command execution and the aiida-firecrest transport/scheduler plugin therefore doesn't have an exec_command_wait method. However, this seems to be more a concern of the transport. As this change would require adding to the abstract interface of Transport, as well as implementing the methods it in its subclasses, just putting out the issue here as a reminder for the future for now.

@GeigerJ2 GeigerJ2 added the type/feature request status undecided label Dec 12, 2024
@khsrali
Copy link
Contributor

khsrali commented Dec 13, 2024

transport.du() 🥇

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/feature request status undecided
Projects
None yet
Development

No branches or pull requests

2 participants