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

upstream the results code #28

Open
jordens opened this issue Dec 10, 2019 · 4 comments
Open

upstream the results code #28

jordens opened this issue Dec 10, 2019 · 4 comments

Comments

@jordens
Copy link

jordens commented Dec 10, 2019

Looks clean, generic, and mature. Is ARTIQ-specific but experiment-agnostic.
Any chance to get a PR for it?

(maybe without artiq_results_path())

@dnadlinger
Copy link
Member

From my perspective, the main utility of this code is actually the artiq_results_path machinery, i.e. that you can just load results from a Jupyter notebook/script/… on any of our group's machines without having to worry about which paths to use.

But yes, just the directory listing/filtering aspect might also be useful. I'm not too sure about cleanliness (there are inconsistencies such as cls vs. class_name), but YMMV.

Where would you want to put this?

@jordens
Copy link
Author

jordens commented Dec 12, 2019

artiq_results_path seemed very specific with the environment vars and all.

Maybe protocols or a new user facing scripting module?

@dnadlinger
Copy link
Member

dnadlinger commented Dec 17, 2019

artiq_results_path seemed very specific with the environment vars and all.

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 2019-12-12/21/000….h5.

If you are interested in this, we should probably move the result tree walking code to whatever new data analysis scripting module, and reimplement oitg.results on top of that.

@jordens
Copy link
Author

jordens commented Dec 17, 2019

Agreed

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

2 participants