Reorganize code base - Githubissues

radical-cybertools / radical.analytics

Analytics for RADICAL-Cybertools

Other

1 stars 1 forks source link

Reorganize code base #113

Open mturilli opened 4 years ago

mturilli commented 4 years ago

Move plotting under example?

andre-merzky commented 4 years ago

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

mturilli commented 4 years ago

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 commented 4 years ago

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).