Avoid false positive SLEPc detection

If we seem to have SLEPc, but there's some reason we can't scarf build
configuration from SLEPc, then we really don't have SLEPc.

I got caught by this when trying a new PETSC_ARCH for which I hadn't
yet built SLEPc.
Remove unused variable.
This function is supposed to return something, so we probably should.
rbOOmit change: made RBConstruction::assemble_affine_expansion public
Added convergence tests in the RBConstruction subclasses

Signed-off-by: David Knezevic <dknezevic@seas.harvard.edu>
rbOOmit change: cleaned up previous commit a bit.
rbOOmit change: fixed compiler warning due to initialization order
Update github traffic data:
* Add a few data points
* Add total pageviews
* Clean up
Refactoring ugly code in MeshTools::find_nodal_neighbors().

* Variable names were ugly.
* Variables unnecessarily declared outside of loops.
* "empty while loop with post-decrement" is ridiculous.
* Formatting inconsistent with rest of libmesh.
* Did not use Elem interfaces effectively.

This patch should also be cherry-picked onto master, it doesn't have
to wait until the subdivision surfaces stuff is merged.
Update github traffic.
fix for distcheck
fix for older PETScs
add convergence feedback for petsc<3.2.0
missed a __thread->LIBMESH_TLS change somewhere along the way
rbOOmit changes:
- Added ability to add a point loads on a nodesets in RBConstruction.
- Removed deprecated RBConstruction::assemble_scaled_matvec() method.
block to avoid incorrect completion if called in quick succession on two different MeshfreeInterpolation objects.
trailing whitespace-only patch
release libHilbert under LGPL - thanks, Chris.
rbOOmit change: Added new constructor for RBParameters.
Eigen 3.2.1 source