Allowing more flexible parametrization of the cost estimate functions

In the recent cost estimates functions, there are a number of hardcoded parameters in the system:

OpenFermion/src/openfermion/resource_estimates/surface_code_compilation/physical_costing.py

Lines 189 to 201 in cf53c06

    
           for logical_data_qubit_distance in range(7, 35, 2): 
        
               params = AlgorithmParameters( 
        
                   physical_error_rate=physical_error_rate, 
        
                   surface_code_cycle_time=datetime.timedelta(microseconds=1), 
        
                   logical_data_qubit_distance=logical_data_qubit_distance, 
        
                   magic_state_factory=factory, 
        
                   toffoli_count=num_toffoli, 
        
                   max_allocated_logical_qubits=num_logical_qubits, 
        
                   factory_count=4, 
        
                   routing_overhead_proportion=0.5, 
        
                   proportion_of_bounding_box=portion_of_bounding_box) 
        
               cost = params.estimate_cost() 
        
               if cost.algorithm_failure_probability > 0.1:

Should we add them as parameters to some class that the user can willingly change?

@mhucka

Hi @mhucka! I'm a student working on a course project. I'd love to take this issue and make these hardcoded parameters configurable. Could you assign this to me?

@dhanvanthbalaji

@dhanvanthbalaji Thank you for your interest! It's assigned to you now.

Created flexible paramtrization to cost estimate functions namely fac…

@mhucka

Hi @mhucka,I’ve opened Pull Request #1460 to make these cost estimator parameters configurable! All CI tests and formatting checks are now passing green. Whenever you have a chance, I’d appreciate a review. Thanks!

Navigation Menu

Repository navigation

Allowing more flexible parametrization of the cost estimate functions #768

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

	for logical_data_qubit_distance in range(7, 35, 2):
	params = AlgorithmParameters(
	physical_error_rate=physical_error_rate,
	surface_code_cycle_time=datetime.timedelta(microseconds=1),
	logical_data_qubit_distance=logical_data_qubit_distance,
	magic_state_factory=factory,
	toffoli_count=num_toffoli,
	max_allocated_logical_qubits=num_logical_qubits,
	factory_count=4,
	routing_overhead_proportion=0.5,
	proportion_of_bounding_box=portion_of_bounding_box)
	cost = params.estimate_cost()
	if cost.algorithm_failure_probability > 0.1:

Allowing more flexible parametrization of the cost estimate functions #768

Description

Activity

dhanvanthbalaji commented on Sep 29, 2026

mhucka commented on Oct 2, 2026

dhanvanthbalaji commented on Oct 4, 2026

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions