Closed rfrenchseti closed 11 months ago
@jnspitale A few things:
/home/spitale/pds-/pds-tools/vicar.py
, which means you still have pds-tools
in your PYTHONPATH
. As discussed you should really remove that repo entirely from your disk to prevent things like this from happening (or at least rename it), and make sure it is removed from your environment setup as well. This explains the error TypeError: from_file() got an unexpected keyword argument 'tkdict'
, since the version of textkernel
in pds-tools
and the version that's pip-installable have radically diverged. This is an error you reported on a previous pull as well.rms-oops
in your PYTHONPATH if you want it to find the top-level module.
Major restructuring:
hosts/solar
to new pip-installablerms-solar
repo.polymath
to new pip-installablerms-polymath
repo. Fixes #100.hosts
directory underoops
and change all references as appropriate. Fixes #78.Changes to testing:
backplane/unittester.py
since it was really just a Cassini ISS hosts test, and put it underhosts/cassini
for now. At some point we will want to restructure the Cassini hosts module to follow the same format as Galileo SSI.backplane/__init__.py
for the same reason.oops.hosts
instead of justhosts
I had to create new gold master directories in Dropbox (just copies of the existing ones). Once this pull is merged, we can delete the old directories.--arrays
and--no-arrays
that control writing of the backplane arrays.--summary
and--no-summary
that control writing of thesummary.py
file.--debug
which turns on--arrays --browse --log --summary
. You may hate this, in which case we can revert to the original behavior, but I thought that by default running a test suite should not create a bunch of files.Minor changes:
requirements.txt
to use new versions of modules and new modules above. Tested with Python 3.8 and 3.12..gitignore
to be the GitHub-standard template with our local additions..coveragerc
to the top-level directory and clean up as needed.oops/.vscode
which shouldn't be under source code control in the first place.Known problems:
oops
unit tests include some combination of:you can't currently mix hosts and non-hosts tests in the same unittest run. Since the order of tests can't be specified, the registries are emptied and then a host needs those entries that were previously loaded. So for now you have to run, separately:
Doing this, all tests pass.