-
Notifications
You must be signed in to change notification settings - Fork 10
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
upstream the results code #28
Comments
From my perspective, the main utility of this code is actually the But yes, just the directory listing/filtering aspect might also be useful. I'm not too sure about cleanliness (there are inconsistencies such as Where would you want to put this? |
Maybe protocols or a new user facing scripting module? |
It is; I just wanted to point out that much utility of this in practice stems from not having to hardcode paths everywhere, rather than not typing out If you are interested in this, we should probably move the result tree walking code to whatever new data analysis scripting module, and reimplement |
Agreed |
Looks clean, generic, and mature. Is ARTIQ-specific but experiment-agnostic.
Any chance to get a PR for it?
(maybe without
artiq_results_path()
)The text was updated successfully, but these errors were encountered: