ersilia-os/ersilia

๐Ÿˆ Task: Remove dead code from ersilia

DhanshreeA opened this issue ยท 0 comments

Summary

Ersilia currently houses a lot of dead code that was sketched out while planning for certain integrations, however we have moved away from those and therefore it makes sense to remove this code that is now being unused.

Objective(s)

  • Circle CI configuration since we neither rely on its outcome, nor actively maintain it. Since June 25th, this action has been consistently failing.
  • DVC integration in ersilia/utils/dvc.py, which further gets referenced in the ersilia/hub/fetch/actions/lake.py, ersilia/lake/manager.py, ersilia/publish/publish.py, and ersilia/publish/rebase.py.
  • Dummy model artifacts within the current test suite, namely, eos0t00, ..., eos0t04, since these models are not representative of ersilia's model implementation, and are not amenable to testing several key functionalities such as fetching a model from GitHub/S3/DockerHub, among other things.
  • handle_undecorated_exception.py within ersilia/utils/exception_utils subpackage since this is not being used anywhere currently and there appears to be no future requirement for this module.
  • issue_reporting.py within ersilia/utils/exception_utils which implements issue reporting via automated creation of issues through ersilia CLI. We do not intend to put this functionality to use, or maintain it, therefore it will be removed.
  • tables.py within ersilia/db/hubdata which contains skeleton classes for DynamoDB integration, however this functionality is neither being used now, nor will be implemented in the future. This will in likelihood be superseded by the work from GDI.

Documentation

No response