call stacks should get captured if group_by_scope=true (#2803)

The `group_by_scope` setting wasn't working when passed in from Python
because it by itself wasn't enough to switch on stack trace capturing.

Author: minestarks

source/compiler/qsc/src/interpret/circuit_tests.rs

@@ -188,6 +188,43 @@ fn rotation_gate() {
     .assert_eq(&circ);
 }
 
+#[test]
+fn grouping_nested_callables() {
+    let circ = circuit_with_options(
+        r"
+            namespace Test {
+                @EntryPoint()
+                operation Main() : Unit {
+                    use q = Qubit();
+                    for i in 0..5 {
+                        Foo(q);
+                    }
+                    MResetZ(q);
+                }
+
+                operation Foo(q: Qubit) : Unit {
+                    H(q);
+                }
+            }
+        ",
+        Profile::Unrestricted,
+        CircuitEntryPoint::EntryPoint,
+        CircuitGenerationMethod::ClassicalEval,
+        TracerConfig {
+            max_operations: usize::MAX,
+            source_locations: false,
+            group_by_scope: true,
+        },
+    )
+    .expect("circuit generation should succeed");
+
+    expect![[r#"
+        q_0    ─ [ [Main] ─── [ [Foo] ─── H ──── H ──── H ──── H ──── H ──── H ──── ] ──── M ──── |0〉 ──── ] ──
+                     [                                                                     ╘══════════════ ] ══
+    "#]]
+    .assert_eq(&circ.display_with_groups().to_string());
+}
+
 #[test]
 fn classical_for_loop() {
     let circ = circuit(

source/compiler/qsc_circuit/src/builder.rs

@@ -134,7 +134,7 @@ impl Tracer for CircuitTracer {
     }
 
     fn is_stack_tracing_enabled(&self) -> bool {
-        self.config.source_locations
+        self.config.source_locations || self.config.group_by_scope
     }
 }