variable naming / destructuring

[penelopeysm]

I thought I'd put some code suggestions in a PR rather than review comments, because to suggest all these changes would have been fiddly and error-prone.

Basically, I'm proposing these changes:

• the _local variable names are a bit repetitive and have lost their meaning since the loop was extracted into a function, so I got rid of most of them.
• I prefer to call the top thing varname_tuples, as that better reflects the fact that there are two nested levels of containers (as opposed to varnames which suggests only one level)
• using h, t... = container as opposed to explicit indexing [2:end], as I think it's cleaner and better reflects what we're trying to do (pluck off the first element and keep the rest for later)

[penelopeysm]

Of course, feel free to push back on any of it like normal review comments:)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.01%. Comparing base (c44d81a) to head (2345126).
Report is 2 commits behind head on mhauru/recursive-gibbs.

Additional details and impacted files

@@                   Coverage Diff                   @@
##           mhauru/recursive-gibbs    #2465   +/-   ##
=======================================================
  Coverage                   85.01%   85.01%           
=======================================================
  Files                          21       21           
  Lines                        1582     1582           
=======================================================
  Hits                         1345     1345           
  Misses                        237      237           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[mhauru]

Happy with the other changes, but see comment above for varname_tuples.

[mhauru]

Good changes, thanks!