quantum icon indicating copy to clipboard operation
quantum copied to clipboard

Duplicate module paths

Open MichaelBroughton opened this issue 4 years ago • 3 comments

While not a breaking issue, if a user looks at tfq.layers and then lists all available modules there are two paths to any given layer at the moment:

tfq.layers.AddCircuit(           tfq.layers.PQC(
tfq.layers.circuit_construction  tfq.layers.Sample(
tfq.layers.circuit_executors     tfq.layers.SampledExpectation(
tfq.layers.ControlledPQC(        tfq.layers.State(
tfq.layers.Expectation(          tfq.layers.Unitary(
tfq.layers.high_level  

One could go PQC and high_level.PQC. This should be a quick fix.

MichaelBroughton avatar May 05 '20 22:05 MichaelBroughton

This issue has not had any activity in a month. Is it stale ?

github-actions[bot] avatar Aug 01 '20 00:08 github-actions[bot]

For reference the issues we currently have are:

differentiators show modules layers show module util shows module

MichaelBroughton avatar Sep 01 '20 22:09 MichaelBroughton

What command to run to see the duplicate paths? When I run dir(tfq.layers) I get

['AddCircuit',
 'ControlledPQC',
 'Expectation',
 'NoisyControlledPQC',
 'NoisyPQC',
 'PQC',
 'Sample',
 'SampledExpectation',
 'State',
 'Unitary',
 '__builtins__',
 '__cached__',
 '__doc__',
 '__file__',
 '__loader__',
 '__name__',
 '__package__',
 '__path__',
 '__spec__',
 'circuit_construction',
 'circuit_executors',
 'high_level']

Is this issue referencing the fact that hight_level is present in this list?

zaqqwerty avatar Apr 07 '22 17:04 zaqqwerty