Skip to content

Research/bf16 phase0 review - #129

Closed
DeanTMaxim wants to merge 203 commits into
tensorcircuit:masterfrom
DeanTMaxim:research/bf16-phase0-review
Closed

Research/bf16 phase0 review#129
DeanTMaxim wants to merge 203 commits into
tensorcircuit:masterfrom
DeanTMaxim:research/bf16-phase0-review

docs(phase0): add research report and reviewer briefing

e455a91
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Jul 26, 2026 in 1m 35s

Quality Gate failed

Failed conditions
3.6% Duplication on New Code (required ≤ 3%)
C Reliability Rating on New Code (required ≥ A)
E Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Annotations

Check warning on line 777 in results/_phase0/region_proto.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Merge these implicitly concatenated strings; or did you forget a comma? 

See more on https://sonarcloud.io/project/issues?id=tensorcircuit_tensorcircuit-ng&issues=AZ-fw8bH1eeMvlV2rpZ7&open=AZ-fw8bH1eeMvlV2rpZ7&pullRequest=129

Check failure on line 16 in applications/benchmarks/aggregate_results.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 18 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=tensorcircuit_tensorcircuit-ng&issues=AZ-fw8dh1eeMvlV2rpbK&open=AZ-fw8dh1eeMvlV2rpbK&pullRequest=129

Check warning on line 46 in results/_phase0/c1_to_c2_map.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use concise character class syntax '\d' instead of '[0-9]'. 

See more on https://sonarcloud.io/project/issues?id=tensorcircuit_tensorcircuit-ng&issues=AZ-fw8aM1eeMvlV2rpYw&open=AZ-fw8aM1eeMvlV2rpYw&pullRequest=129

Check failure on line 195 in results/_phase0/c1.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

LLMs running this code with faulty CLI arguments can escape file system restrictions. Refactor this code to validate the constructed path before accessing the file system. 

See more on https://sonarcloud.io/project/issues?id=tensorcircuit_tensorcircuit-ng&issues=AZ-fw8Zk1eeMvlV2rpX0&open=AZ-fw8Zk1eeMvlV2rpX0&pullRequest=129

Check warning on line 394 in results/_phase0/cutlass_probe.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename this local variable "refIm" to match the regular expression ^[_a-z][a-z0-9_]*$. 

See more on https://sonarcloud.io/project/issues?id=tensorcircuit_tensorcircuit-ng&issues=AZ-fw8Y71eeMvlV2rpXN&open=AZ-fw8Y71eeMvlV2rpXN&pullRequest=129

Check failure on line 91 in results/_phase0/sanitize.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 16 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=tensorcircuit_tensorcircuit-ng&issues=AZ-fw8cq1eeMvlV2rpaw&open=AZ-fw8cq1eeMvlV2rpaw&pullRequest=129

Check failure on line 1315 in results/_phase0/gonogo.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 20 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=tensorcircuit_tensorcircuit-ng&issues=AZ-fw8a41eeMvlV2rpZK&open=AZ-fw8a41eeMvlV2rpZK&pullRequest=129

Check warning on line 384 in results/_phase0/region_proto.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename this local variable "BK_p" to match the regular expression ^[_a-z][a-z0-9_]*$. 

See more on https://sonarcloud.io/project/issues?id=tensorcircuit_tensorcircuit-ng&issues=AZ-fw8bH1eeMvlV2rpZo&open=AZ-fw8bH1eeMvlV2rpZo&pullRequest=129

Check failure on line 103 in results/_phase0/run_context.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 18 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=tensorcircuit_tensorcircuit-ng&issues=AZ-fw8aV1eeMvlV2rpY6&open=AZ-fw8aV1eeMvlV2rpY6&pullRequest=129

Check warning on line 88 in results/_phase0/region_proto.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename this parameter "B" to match the regular expression ^[_a-z][a-z0-9_]*$. 

See more on https://sonarcloud.io/project/issues?id=tensorcircuit_tensorcircuit-ng&issues=AZ-fw8bH1eeMvlV2rpZP&open=AZ-fw8bH1eeMvlV2rpZP&pullRequest=129

Check warning on line 429 in results/_phase0/region_proto.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename this local variable "E_tiled" to match the regular expression ^[_a-z][a-z0-9_]*$. 

See more on https://sonarcloud.io/project/issues?id=tensorcircuit_tensorcircuit-ng&issues=AZ-fw8bH1eeMvlV2rpZu&open=AZ-fw8bH1eeMvlV2rpZu&pullRequest=129

Check warning on line 1549 in results/_phase0/region_proto.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename this local variable "D_b" to match the regular expression ^[_a-z][a-z0-9_]*$. 

See more on https://sonarcloud.io/project/issues?id=tensorcircuit_tensorcircuit-ng&issues=AZ-fw8bH1eeMvlV2rpaG&open=AZ-fw8bH1eeMvlV2rpaG&pullRequest=129

Check failure on line 466 in results/_phase0/numerical.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 19 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=tensorcircuit_tensorcircuit-ng&issues=AZ-fw8aB1eeMvlV2rpYI&open=AZ-fw8aB1eeMvlV2rpYI&pullRequest=129

Check failure on line 356 in results/_phase0/region_proto.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 50 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=tensorcircuit_tensorcircuit-ng&issues=AZ-fw8bH1eeMvlV2rpZx&open=AZ-fw8bH1eeMvlV2rpZx&pullRequest=129

Check warning on line 505 in results/_phase0/manifest.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Make sure that this user-controlled command argument doesn't lead to unwanted behavior. 

See more on https://sonarcloud.io/project/issues?id=tensorcircuit_tensorcircuit-ng&issues=AZ-fw8bg1eeMvlV2rpad&open=AZ-fw8bg1eeMvlV2rpad&pullRequest=129

Check warning on line 393 in results/_phase0/cutlass_probe.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename this local variable "ImB_bf" to match the regular expression ^[_a-z][a-z0-9_]*$. 

See more on https://sonarcloud.io/project/issues?id=tensorcircuit_tensorcircuit-ng&issues=AZ-fw8Y71eeMvlV2rpXC&open=AZ-fw8Y71eeMvlV2rpXC&pullRequest=129

Check warning on line 503 in results/_phase0/cutlass_probe.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace set constructor call with a set comprehension. 

See more on https://sonarcloud.io/project/issues?id=tensorcircuit_tensorcircuit-ng&issues=AZ-fw8Y71eeMvlV2rpXP&open=AZ-fw8Y71eeMvlV2rpXP&pullRequest=129

Check warning on line 563 in results/_phase0/cutlass_probe.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename this local variable "ImA_list" to match the regular expression ^[_a-z][a-z0-9_]*$. 

See more on https://sonarcloud.io/project/issues?id=tensorcircuit_tensorcircuit-ng&issues=AZ-fw8Y71eeMvlV2rpXS&open=AZ-fw8Y71eeMvlV2rpXS&pullRequest=129

Check warning on line 471 in results/_phase0/c1.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use concise character class syntax '\d' instead of '[0-9]'. 

See more on https://sonarcloud.io/project/issues?id=tensorcircuit_tensorcircuit-ng&issues=AZ-fw8Zk1eeMvlV2rpXu&open=AZ-fw8Zk1eeMvlV2rpXu&pullRequest=129

Check failure on line 615 in results/_phase0/numerical.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 21 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=tensorcircuit_tensorcircuit-ng&issues=AZ-fw8aB1eeMvlV2rpYK&open=AZ-fw8aB1eeMvlV2rpYK&pullRequest=129

Check warning on line 392 in results/_phase0/cutlass_probe.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename this local variable "ReB_bf" to match the regular expression ^[_a-z][a-z0-9_]*$. 

See more on https://sonarcloud.io/project/issues?id=tensorcircuit_tensorcircuit-ng&issues=AZ-fw8Y71eeMvlV2rpXH&open=AZ-fw8Y71eeMvlV2rpXH&pullRequest=129

Check failure on line 446 in results/_phase0/gonogo.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 30 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=tensorcircuit_tensorcircuit-ng&issues=AZ-fw8a41eeMvlV2rpZC&open=AZ-fw8a41eeMvlV2rpZC&pullRequest=129

Check failure on line 872 in results/_phase0/c2.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 30 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=tensorcircuit_tensorcircuit-ng&issues=AZ-fw8c51eeMvlV2rpa3&open=AZ-fw8c51eeMvlV2rpa3&pullRequest=129

Check warning on line 59 in results/_phase0/region_proto.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the unused function parameter "depth". 

See more on https://sonarcloud.io/project/issues?id=tensorcircuit_tensorcircuit-ng&issues=AZ-fw8bH1eeMvlV2rpZN&open=AZ-fw8bH1eeMvlV2rpZN&pullRequest=129

Check failure on line 437 in results/_phase0/c2.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

LLMs running this code with faulty CLI arguments can escape file system restrictions. Refactor this code to validate the constructed path before accessing the file system. 

See more on https://sonarcloud.io/project/issues?id=tensorcircuit_tensorcircuit-ng&issues=AZ-fw8c51eeMvlV2rpa_&open=AZ-fw8c51eeMvlV2rpa_&pullRequest=129
View more details on SonarQubeCloud