BasisResearch / cities

Home of Basis development for the 2023 TOP Sprint
MIT License
6 stars 0 forks source link

refactor CausalInsight as subclass for CausalInsightSlim #101

Open rfl-urbaniak opened 9 months ago

rfl-urbaniak commented 9 months ago

The latter has been introduced to avoid reliance of Torch and on anything that uses Torch in the front-end, on a rather short notice, which leads to some code duplication between some methods of the latter and the former. To avoid code duplication, make sure CausalInsight inherits from CausalinsightSlim instead. Note: the latter depends on cached available years data, which the former computes in a way that now uses tensors, you'll need to pay attention when unifying.