Fix #2466 - Studio Groom: Groomed meshes not always reloading after groom run

  The cache was being cleared after set_display_mode() was called in
  handle_groom_complete(). However, set_display_mode() emits an
  update_view_mode() signal which triggers update_display(), causing
  mesh lookups to hit the stale cache before it was cleared.

  This resulted in an alternating pattern where groomed meshes would
  only reload on every other groom run.

  Move handle_clear_cache() to before set_display_mode() so the cache
  is cleared before any display updates are triggered.

Author: akenmorris

Libs/Analyze/MeshGenerator.cpp

@@ -135,7 +135,6 @@ MeshHandle MeshGenerator::build_mesh_from_image(ImageType::Pointer image, float
 
 //---------------------------------------------------------------------------
 MeshHandle MeshGenerator::build_mesh_from_file(std::string filename, float iso_value) {
-  // std::cerr << "build_mesh from " << filename << "\n";
   MeshHandle mesh(new StudioMesh);
 
   if (!QFileInfo(QString::fromStdString(filename)).exists()) {

Studio/Interface/ShapeWorksStudioApp.cpp

@@ -1335,15 +1335,17 @@ void ShapeWorksStudioApp::handle_groom_start() {
 void ShapeWorksStudioApp::handle_groom_complete() {
   update_view_combo();
 
+  // Clear cache BEFORE setting display mode, because set_display_mode triggers
+  // update_view_mode() which calls update_display() and would use stale cached meshes
+  session_->handle_clear_cache();
+
   if (!session_->groomed_present()) {
     // grooming may have failed, if so, we don't want to switch to groomed that don't exist
     session_->set_display_mode(DisplayMode::Original);
   } else {
     session_->set_display_mode(DisplayMode::Groomed);
   }
 
-  session_->handle_clear_cache();
-
   update_display(true);
   visualizer_->reset_camera();
   enable_possible_actions();