diffix / syndiffix

Python implementation of the SynDiffix synthetic data generation mechanism.
Other
4 stars 1 forks source link

Refactor location of get_null_mapping #129

Closed yoid2000 closed 8 months ago

yoid2000 commented 8 months ago

In order to implement #128 , I want to make a new file called marginal.py. This file needs to import Forest, and microdata.py needs to import marginal.py.

From the current implementation, this creates an import loop, because forest.py imports microdata.py in order to get access to get_null_mappings().

I propose moving get_null_mappings() to interval.py. This seems a little cleaner in any event, since nothing in microdata.py uses get_null_mappings()

yoid2000 commented 8 months ago

Finished with https://github.com/diffix/syndiffix/pull/130