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

Reorganize code base #113

Open
mturilli opened this issue Feb 11, 2020 · 3 comments
Open

Reorganize code base #113

mturilli opened this issue Feb 11, 2020 · 3 comments

Comments

@mturilli
Copy link
Contributor

  • Move plotting under example?
@andre-merzky
Copy link
Member

I actually wonder if we should move the plotter into RP... - what do you think?

@mturilli
Copy link
Contributor Author

I reviewed radical-analytics-plot.py and I am not sure I see too much use for it. Analytics is a low-level library that returns timestamps and duration for a given state/event model. As such, it is agnostic towards how those data are plotted. From the code it is not clear what the plot is plotting. Maybe it might have a place in the upcoming tutorial? Otherwise, I would be inclined to clean it up. Suggestions?

@andre-merzky
Copy link
Member

I use it to quickly plot time series or histograms for data sets extracted via RA. I am not sure either if and what place it should have in the code base, and can maintain it in my own space if you want to clean it out, by all means. If you think it is useful, I can also create some examples on how it can be efficiently used and add some documentation. Either way works - but I agree that the current state is not veryr useful (beyond for myself).

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

No branches or pull requests

3 participants