Initial commit for module fix

aalto-ele / bag_ecd

Python class to take care of path and variable setups between userspace and python

Other

2 stars 0 forks source link

Initial commit for module fix #4

Closed vlahtin closed 3 years ago

vlahtin commented 3 years ago

Adds the ability to merge BAG and TheSyDeKick entities.

Fixes the BAG schematic path to yaml file
Checks where the BAG generator is run from, to avoid adding same name package twice. If the generator is run from BAG directory, it is added normally. If the generator is run from SyDeKick, the package is not added again.

mkosunen commented 3 years ago

@vlahtin Does this assume that the modules are identical?

mkosunen commented 3 years ago

I think I got it: If we are in the sydekick, thesdk module adds also the packages from BAG_WORK_DIR to the path as it is in the PYTHONPATH, and they should not be added twice. Is that the logic?

I thik it is safe to merge. If somethig breaks, it just need to be fixed. :)

vlahtin commented 3 years ago

Yes, when running from sydekick, first sydekick packages are added, then BAG packages are added.

The addition in init checks if sydekick already added a package with the same name, and if the package already exists it is not added again.