qiskit-metal icon indicating copy to clipboard operation
qiskit-metal copied to clipboard

Check what we are doing with default units and ensure consistency

Open JeremyDrysdale-ibm opened this issue 4 years ago • 1 comments

From @zlatko-minev

What is the feature being requested?

Pull in default units once, at module load time, save for module, so no need to access again and take time

What are use cases for this feature?

No pointless reloading of default units

JeremyDrysdale-ibm avatar Nov 16 '20 22:11 JeremyDrysdale-ibm

@zlatko-minev, I am trying to determine the scope of this issue.

Looked at the code in class ParsedDynamicAttributes_Component() and methods in parsing.py.

My evaluation of request: When a QComponent is loaded into QDesign the first time, the parsed default options could be stored in QDesign. So, when the same QComponent is loaded into QDesign again, the parsed options can be read from QDesign. So parsing doesn't have to happen again.

This is not correct!!!!!!!

update: Make sure that changing default units works when they are changed. Would need to look at he parsing module. At a global level, switch units.

priti-ashvin-shah-ibm avatar Feb 16 '21 14:02 priti-ashvin-shah-ibm