Google Sheets OPR Calculator

[Ether]

Originally Posted by Rachel Lim

I finally had some time to go through them, and combined both suggestions (or at least think I did) in the code below.

Hi Rachel.

I think you missed the boat here.

Your code has three passes through the matches, and the final pass has an inner loop which loops through the teams.

The pseudo-code I posted does a single pass through the 8col match data, with no inner loops in that single pass. That single pass accomplishes everything you need.

Give it a try!

[Rachel Lim]

Originally Posted by Ether

Hi Rachel.

I think you missed the boat here.

Your code has three passes through the matches, and the final pass has an inner loop which loops through the teams.

The pseudo-code I posted does a single pass through the 8col match data, with no inner loops in that single pass. That single pass accomplishes everything you need.

Give it a try!

Yep...I realized what you meant after I posted it.

I also figured out what's taking so much time: it's the repeated .getRange().setValue(). There's another function, setValues() that allows me to print the entire array at once that really speeds it up.

However, the array must be a consistent size, so I had to keep the loop to create T so I could initialize A. It also seems like fill() doesn't work in scripts, so I had to loop through T to set A to zero, otherwise when I printed out A it'd be filled with "NOT_FOUND". I did cut out the loop for b (not sure why I had it there in the first place).

Although I did manage to implement your pseudo-code, the only method I found to make that work would be to print A directly (instead of creating the array first), which took longer than looping. It also results in the OPRs not being sorted, which is a convenience I'd sacrifice a bit of time for.

Hopefully that now addresses your point? I'll post my updated code once I clean it up a bit.

[Ether]

Originally Posted by Rachel Lim

the array must be a consistent size, so I had to keep the loop to create T so I could initialize A.

Can't you simply initialize A with a single command (no looping) to a size greater than what you expect is needed? And when you so initialize it, doesn't it set all the array elements to zero (some languages do)?

It also seems like fill() doesn't work in scripts,

I find it hard to imagine the script language provides no way to zero an array without looping through all its elements.

so I had to loop through T to set A to zero

why do you need to "loop through T" to set A? Just initialize the entire array A to zero before beginning the processing of the 8col data

otherwise when I printed out A...

I meant to ask this earlier: What does "print out" mean in this context, and why do you need to do that?

It also results in the OPRs not being sorted, which is a convenience I'd sacrifice a bit of time for.

Please read my earlier post#15 in this thread.

Hopefully that now addresses your point?

Not really. But if you're satisfied with what you have I won't bug you

[Rachel Lim]

Originally Posted by Ether

Can't you simply initialize A with a single command (no looping) to a size greater than what you expect is needed? And when you so initialize it, doesn't it set all the array elements to zero (some languages do)?

I could do something like set A to the number of matches. I'm not sure if it's really worth it, or how much longer it'd take to set that many cells, but I guess that's the alternative.

Array values are set to null (I think--if I set a cell value to that, it comes out as either "undefined" or "NOT_FOUND" depending on whether I set an individual cell or range)

I find it hard to imagine the script language provides no way to zero an array without looping through all its elements.

I think javascript has 2 ways to do that: fill() and apply(), neither of which seems to work in google scripts.

why do you need to "loop through T" to set A? Just initialize the entire array A to zero before beginning the processing of the 8col data

Each row of A is T elements long, and I need to set each value to 0. I can't find another way to do it (see above).

I meant to ask this earlier: What does "print out" mean in this context, and why do you need to do that?

Setting the cells in a certain range to those values (using .getRange().setValues()). Basically google sheets has matrix formulas but google scripts doesn't seem to have an equivalent, so it's easier for me to put the matrices into a sheet, set the formula, and do the final calculations directly in the spreadsheet.

Please read my earlier post#15 in this thread.

I know I can sort it after, or set an equation to create another column sorted, but it makes it more complicated.

Not really. But if you're satisfied with what you have I won't bug you

I understand the reason to optimize it further, but in this case I think I'll spend the time adding in some additional functionality (like importing data automatically, multiple components, etc.). I never intended this to replace other, faster methods of calculating OPR, since other languages (especially matlab/octave) are just inherently faster at these calculations. I was mainly looking for an easy method where I could ask someone at a competition to run it without them needing to do more than type in values / press a button, and get a sorted list out of it.

This has been an interesting conversation though, so thank you for that.



I will be updating the posted spreadsheet with the new script soon (currently cleaning it up a bit, but it should be up by tonight) for anyone who wants it.