Display multiple DataTables in one view

[mkommend]

It would be beneficial if we had the possibility to compare data tables (e.g., quality linechart) across several runs. Therefore a new view for run collections displaying data tables collected from the contained runs should be created.

[mkommend]

r7977: Added methods to handle collection of data rows in DataTableView.

[mkommend]

r7978: Removed unused references in HL.Analysis.Views.

[mkommend]

r7979: Corrected introduced bug in DataTableView concerning adding of DataRows.

[mkommend]

r7980: Added RunCollectionDataTableView.

[mkommend]

r7981: Forgot to add extension methods ToEnumerable.

[abeham]

I think you should yield return item rather than allocating an array for that. Anyway, what's the purpose of that method?

[mkommend]

Replying to abeham:

I think you should yield return item rather than allocating an array for that. Anyway, what's the purpose of that method?

I thought about yielding instead of returning an array, but I couldn't think of any advantage. yield allocates an iterator which is at least as expensive as the array allocation. yield is lazy which could lead to multiple evaluations (e.g, using Count and iterating). Therefore I implemented the method by allocating an array.

The purpose of this method is just to have a convenient way to call methods that require an IEnumerable with an object, which is quite handy sometimes and does no harm.

[mkommend]

r7986: Cleared combo box items if selected data table changes.

[gkronber]

Actually, I think we should not have the ToEnumerable extension at all. I think we don't need a ToEnumerable method for all classes.

[abeham]

It's a great feature, that I have long been looking forward to! Thanks for implementing this!

I reviewed the changes r7977:7981 and made some small adjustments. I also removed the extension method again. Currently I don't see that we need this, also I'm not sure if this shouldn't rather return an array of that type and be called ToArray() or AsArray() as it was actually a ToArray() disguising as a ToEnumerable().

r8108:

• Removed ToEnumerable extension method
• Reset rowNumber to 0 when rebuilding the combined data table
• Some minor changes (moved ctor below variable/property declaration, replaced some obvious variable types with var)

Please have a look at the changes I made and consult with me again or set to release if you're okay with it.

Anyway, very nice to have this in HL!

[abeham]

Let me just add another suggestion. When viewing "All DataRows" we could ignore the color of the runs and use a different color for each DataRow to better separate which row belongs to which group rather than to which run. But that's just a suggestion.

Another possible feature for the future would be to create a chart similar to the ones known from the financial world where a stock curve is shown as a series of box-whisker items. Or just a simple line with whiskers that would display some kind of confidence interval.

[gkronber]

I think you mean a candlestick chart. But it is not really applicable to what you are trying to represent here.

[mkommend]

Replying to abeham:

Let me just add another suggestion. When viewing "All DataRows" we could ignore the color of the runs and use a different color for each DataRow to better separate which row belongs to which group rather than to which run. But that's just a suggestion.

That's an interesting thought which I also had during interpreting. However, it also has it benefits to know which row corresponds to which run, especially if rows of the same runs are somehow correlated. It would suggest we leave it as it is now and track further improvements in separate tickets.

[mkommend]

r8128: Readded ToEnumerable extension method.