useBVH Storybook: Demo does not demonstrate how to set BVH split strategy correctly

[gkjohnson]

Problem description:

Hello! I was taking a look at the useBVH storybook and noticed that changing the "splitStrategy" drop down option did not actually change the BVH structure. It seems the line below returns a string rather than one of the three-mesh-bvh constants resulting in an invalid option.

In the newest version of three-mesh-bvh (v0.5.3) a warning will be logged and no splits will be made when an invalid split strategy is provided. Typescript declaration files have also been added which may make these things easier to spot.

Relevant code:

  const strat = select('split strategy', ['CENTER', 'AVERAGE', 'SAH'], 'CENTER')

Suggested solution:

Change the above line to return the appropriate constant rather than a string.