add a few missing tests, fix formatting

include/polyscope/parameterization_quantity.ipp

@@ -242,7 +242,7 @@ QuantityT* ParameterizationQuantity<QuantityT>::setStyle(ParamVizStyle newStyle)
     exception("Cannot set parameterization visualization style to 'CHECKER_ISLANDS', no islands have been set");
     newStyle = ParamVizStyle::CHECKER;
   }
-  
+
   if (newStyle == ParamVizStyle::CHECKER_ISLANDS) {
     cMap.setPassive("turbo"); // use turbo as default cmap for CHECKER_ISLANDS
   }

include/polyscope/volume_grid.h

@@ -119,7 +119,7 @@ class VolumeGrid : public QuantityStructure<VolumeGrid> {
 
   // === Getters and setters for visualization settings
 
-  // Color of the mesh
+  // Color of the grid volume
   VolumeGrid* setColor(glm::vec3 val);
   glm::vec3 getColor();
 
@@ -136,6 +136,9 @@ class VolumeGrid : public QuantityStructure<VolumeGrid> {
   VolumeGrid* setEdgeWidth(double newVal);
   double getEdgeWidth();
 
+  // Scaling factor for the size of the little cubes
+  VolumeGrid* setCubeSizeFactor(double newVal);
+  double getCubeSizeFactor();
 
 private:
 

src/polyscope.cpp

@@ -814,7 +814,8 @@ void draw(bool withUI, bool withContextCallback) {
   }
 
   // Execute the context callback, if there is one.
-  // This callback is a Polyscope implementation detail, which is distinct from the userCallback (which gets called above)
+  // This callback is a Polyscope implementation detail, which is distinct from the userCallback (which gets called
+  // above)
   if (withContextCallback && contextStack.back().callback) {
     (contextStack.back().callback)();
   }

src/volume_grid.cpp

@@ -389,6 +389,12 @@ VolumeGrid* VolumeGrid::setEdgeWidth(double newVal) {
 }
 double VolumeGrid::getEdgeWidth() { return edgeWidth.get(); }
 
+VolumeGrid* VolumeGrid::setCubeSizeFactor(double newVal) {
+  cubeSizeFactor = newVal;
+  requestRedraw();
+  return this;
+}
+double VolumeGrid::getCubeSizeFactor() { return cubeSizeFactor.get(); }
 
 // === Register functions
 

test/src/basics_test.cpp

@@ -67,6 +67,23 @@ TEST_F(PolyscopeTest, NestedShowWithFrameTick) {
   polyscope::state::userCallback = nullptr;
 }
 
+TEST_F(PolyscopeTest, Unshow) {
+
+  int32_t count = 0;
+  auto showCallback = [&]() {
+    if (count > 1) {
+      polyscope::unshow();
+    }
+    count++;
+  };
+  polyscope::state::userCallback = showCallback;
+  polyscope::show(10);
+
+  EXPECT_LT(count, 4);
+
+  polyscope::state::userCallback = nullptr;
+}
+
 // Make sure that creating an empty buffer does not throw errors
 TEST_F(PolyscopeTest, EmptyBuffer) {
 

test/src/surface_mesh_test.cpp

@@ -121,6 +121,33 @@ TEST_F(PolyscopeTest, SurfaceMeshPick) {
   polyscope::removeAllStructures();
 }
 
+TEST_F(PolyscopeTest, SurfaceMeshMark) {
+  auto psMesh = registerTriangleMesh();
+
+  // edges
+  size_t nEdges = 6;
+  std::vector<size_t> ePerm = {5, 3, 1, 2, 4, 0};
+  psMesh->setEdgePermutation(ePerm);
+  psMesh->markEdgesAsUsed();
+  polyscope::show(3);
+
+  // halfedges
+  std::vector<size_t> hePerm;
+  for (size_t i = 0; i < psMesh->nCorners(); i++) {
+    hePerm.push_back(5 + i);
+  }
+  psMesh->setHalfedgePermutation(hePerm);
+  psMesh->markHalfedgesAsUsed();
+  polyscope::show(3);
+
+  // corners
+  // (permutation is not required for this one)
+  psMesh->markCornersAsUsed();
+  polyscope::show(3);
+
+  polyscope::removeAllStructures();
+}
+
 TEST_F(PolyscopeTest, SurfaceMeshBackface) {
   auto psMesh = registerTriangleMesh();
 
@@ -321,7 +348,7 @@ TEST_F(PolyscopeTest, SurfaceMeshCornerParam) {
   q1->setIslandLabels(islandLabels);
   q1->setStyle(polyscope::ParamVizStyle::CHECKER_ISLANDS);
   polyscope::show(3);
-  
+
 
   // create the curve network
   q1->createCurveNetworkFromSeams();

test/src/volume_grid_test.cpp

@@ -47,6 +47,10 @@ TEST_F(PolyscopeTest, VolumeGridBasicOptions) {
   // Edge width
   psGrid->setEdgeWidth(0.5);
   polyscope::show(3);
+
+  // Grid size factor
+  psGrid->setCubeSizeFactor(0.5);
+  polyscope::show(3);
 
   polyscope::removeAllStructures();
 }