unihd-cag / skillbridge Goto Github PK

Remote objects should provide a way to print their Skill ID:
db:0x12345678 -> 0x12345678

When virtuoso starts the python server we know the path to python_server.py and we also know that this file is inside the site-packages directory of a python installation. Maybe we can find out the correct python executable that belongs to this installation.

This removes the need to use a virtual environment.

Some things should be removed:

• unused features (for loop on Skill side)
• function calls still create variables (unnecessary)
• horrible type hinting for Skill types

There are skill function alias which are not dumped at the moment:

Example:

getCurrentWindow <-> hiGetCurrentWindow

Using an underscore in the name string handed to DummyWorkspace.prepare_remote results in a ValueError: too many values to unpack (expected 2)

See: skillbridge/client/objects.py:39: ValueError

cellView->type returns a Symbol.

This should be wrapped into a Python Symbol object:

• the symbol stores the name of the Skill symbol
• printing the symbol prints the name of the symbol
• converting the symbol to Skill appends a single quote

I was seeing a timeout error when I stay connected to the server and run some operations for a few hours. So I decided to set a longer timeout period by pyStartServer ?timeout nil. But virtuoso returns this error

*Error* ilGetString: arg must be symbol or string - nil

I tried the command mentioned in the documentation pyStartServer ?timeout 10.5 which also returns the same error.

*Error* ilGetString: arg must be symbol or string - 10.5

sending 10.5 as string "10.5" returns

pyStartServer ?timeout "10.5" t error "usage: /......./lib/python3.6/site-packages/skillbridge/server/python_server.py\n [-h] [--notify] [--single]\n socket {DEBUG,WARNING,INFO,ERROR,CRITICAL,FATAL}\n/tmp/mhasan13-lws54/anaconda3/envs/brian/lib/python3.6/site-packages/skillbridge/server/python_server.py: error: argument log_level: invalid choice: '10.5' (choose from 'DEBUG', 'WARNING', 'INFO', 'ERROR', 'CRITICAL', 'FATAL')\n" server was stopped with code 2

What is the correct way to set timeout to some value or nil?

Currently the python_server.il dumps every variable into the global namespace. Use let to prevent that.

Some Skill functions show a warning instead of throwing an error.

The Skillbridge should capture these warnings and replay them on the Python side.

Why

• Better integration into GitHub
• No need for passing passwords around
• Plugins for all tasks
  • Sphinx deploy
  • PyPi deploy
• Seems to be more responsive, i.e. Workflows trigger faster

It could be useful to run the skillbridge in a direct mode such that it does not use the TCP server, but communicates with Virtuoso, etc. directly.

The C Parser should compile on all normal Linux systems. It doesn't make sense to keep the Python implamentation any longer.

pyStartServer with any logLevel even "WARNING" results in the server not responding
pyStartServer without the logLevel argument works.

The way jupyter notebook works multiple connections exist. Implement this as the default but add a parameter so disallow multiple connections

Maybe we need a CLI script as a replacement to the __main__.py script for users that are not familiar with Python.

E.g.

skillbridge
# should print the path to the server
# should print if the functions have been dumped
# should print if the static completion stubs have been generated

skillbridge generate
# should generate the static completion stubs

Other commands could include

• finding all running python server (and killing them)

the tab completion here also yields all methods this does not seem right.

Example:

In Skill:

rodCreatePath(..., ?cvId cellViewId)

is currently in python:

ws.rod.create_path(..., cvId=cell_view_id)

but it should be

ws.rod.create_path(..., cv_id=cell_view_id)

Allow overwrite of already existing definitions in skillbridge import via a yes flag.
This would be useful in automated environments.

cds ipc hdb pcdb tx ciw dds dmb dm de deo lbui env nr dl dag gdm nmp ccp lmgr ps tech tc cds dr ann he hsm opc lm cpf ci cci cpf cst imp vhdl conn pte tpa bnd lce ncl soi vfo abs icc cpfe rte rde adp adpnl vfp vpa ap gpe lob mg idf lo odc cph ct we drd mpt abe pc xpc vsa elec ead vdr asi nl ans hnl art apa drpl amse ann ahdl alm anc dr fam msp sev cdf aed relx axl verif awv arm cal adt rdb vv viva ael auLvs par ams ael fnl ocnxl sim ise hnl vos vhms po pi xst pipo xoasis xoas ldtr edi edifin edif edifout

skillbridge import FILE and skillbridge export FILE require absolute paths. Relative paths should also work.

I used pip install skillbirdge. Then in Virtuoso CIW ran load("path\to\python_server.il") and pyDumpFunctionDefinitions "<install>".
After that on the same CIW, when I run pyStartServer I get
error "python: can't open file '/path/to/lib/python3.6/site-packages/skillbridge/server/python_server.ipy': [Errno 2] No such file or directory\n" server was stopped with code 2

I thought I needed to close Virtuoso and run the two commands again. But when I did that I got this.
error " File \"/path/to/lib/python3.6/site-packages/skillbridge/server/python_server.py\", line 21\n def send_to_skill(data: str) -> None:\n ^\nSyntaxError: invalid syntax\n" server was stopped with code 1

Am I missing anything? Isn't "Open the Skill console in Virtuoso and type" the same thing as "typing in Virtuoso CIW" ?
python version 3.6
virtuoso 6.1.7

Currently all the path handling is done with strcat and substr. We must find a more robust way to manipulate the paths we use inside the skill server.

E.g. load("path-to-server.il ") should work even if there are too many spaces.

• pyDumpFunctionDefinitions "" in Sphinx (is already present)
• pip install skillbridge in Readme
• Getting Started separate Examples
• Add how-to update package in Installation

Some skill functions are not valid python identifiers: PREFIXreturn, PREFIXas when you remove the prefix.

There is no static completion for skill functions in IDEs:

Typing ws.db.<TAB> should print all db functions. That list is known, it is inside the definitions.txt.

This tools works with IC6.1.7 and ICADV and upwards

UserWarning: Function definitions are not generated Run 'dumpFunctionDefinitions "<install>"' first, before using this module.

but should be pyDumpFunctionDefinitions <install>

Usecase is different people using the tool on the same machine.

DB objects have an attribute objType and DD objects have an attribute type. We should use them to improve the string representation of the remote objects.

db.check() is missing dbCheck skill function is not extracted.

We should revert to the old dump method

There is no way to test the skillbridge in an external package without a running Virtuoso instance.

In line 100 python must be exchanged with __py_python

Verify correct documentation generation

Some ROD functions return these IDs: rodObj:... but the parser assumed, that IDs are lower case.

test_translator.py must import call from the translator.py module.

I was trying to get OCEAN functions like simulator, modelFile, resultsDir etc available to python through skillbridge. I modified the listFunctions function argument as "[a-z]" in the python_server.il file so that it includes every kind of SKILL functions when I run pyDumpFunctionDefinitions.

However I don't know how to use those the newly added OCEAN functions (like simulator).
I could not get those functions through ws = Workspace.open() then ws.simulator.

Does skillbridge has the functionality to use OCEAN functions?

Some Cadence tools on Windows also use a built in Skill interpreter (Allegro).

For the Windows support, we need a new Channel implementation, because currently we use TCP Unix Domain Sockets.

Windows has the SIO_LOOPBACK_FAST_PATH option since Windows 7. This looks like a viable alternative.

For a minimal example:

• open a schematic add a transistor
• create a design intend
• constraint = ws.ci.get_cell_view()
• constraint.constraints[0].members yields the error