read_manifest() updates
No more `update` param, meaningless without return value
Remove return value/update from docstring
Add `job` param to docstring
@@ -344,7 +344,7 @@ class Chip:
# 4. read in all cfg files
if 'cfg' in cmdargs.keys():
for item in cmdargs['cfg']:
- self.read_manifest(item, update=True, clobber=True, clear=True)
+ self.read_manifest(item, clobber=True, clear=True)
# insert all parameters in dictionary
self.logger.info('Setting commandline arguments')
@@ -1742,7 +1742,7 @@ class Chip:
return True
###########################################################################
- def read_manifest(self, filename, job=None, update=True, clear=True, clobber=True):
+ def read_manifest(self, filename, job=None, clear=True, clobber=True):
"""
Reads a manifest from disk and merges it with the current compilation manifest.
@@ -1751,13 +1751,10 @@ class Chip:
Args:
filename (filepath): Path to a manifest file to be loaded.
- update (bool): If True, manifest is merged into chip object.
+ job (str): Specifies non-default job to merge into.
clear (bool): If True, disables append operations for list type.
clobber (bool): If True, overwrites existing parameter value.
- Returns:
- A manifest dictionary.
-
Examples:
>>> chip.read_manifest('mychip.json')
Loads the file mychip.json into the current Chip object.
@@ -1778,7 +1775,6 @@ class Chip:
f.close()
#Merging arguments with the Chip configuration
- if update:
self.merge_manifest(localcfg, job=job, clear=clear, clobber=clobber)
###########################################################################
