SDV icon indicating copy to clipboard operation
SDV copied to clipboard

Allow for disconnected tables

Open lajohn4747 opened this issue 1 year ago • 1 comments

CU-86b092r53 Issue

Allow for disconnected schemas by removing the check.

Metadata visualization still works All other synthesizers work fine with no relationships defined in metadata (HMA and SDV Enterprise multi-table synthesizers) Evaluations and quality report still work (Eye test scores look fine, cardinality and intertable trends drop in score)

Does not require any other fixes for running multi-table synthesizer

lajohn4747 avatar May 02 '24 15:05 lajohn4747

Now that datacebo/sdv-enterprise#585, datacebo/sdv-enterprise#578, and datacebo/sdv-enterprise#579 are in can we re-run the experiments to make sure everything works as expected? @lajohn4747

Assuming everything works and no other issues arise, are we good to mark this as ready @npatki?

frances-h avatar Jun 04 '24 19:06 frances-h

Now that https://github.com/datacebo/SDV-Enterprise/issues/585, https://github.com/datacebo/SDV-Enterprise/issues/578, and https://github.com/datacebo/SDV-Enterprise/issues/579 are in can we re-run the experiments to make sure everything works as expected? @lajohn4747

Validated it works for demo datasets for all our multi-synthesizers.

lajohn4747 avatar Jun 05 '24 19:06 lajohn4747

So if you run the tests on enterprise (including the benchmarking) with these updates everything passes?

Yep all the tests and benchmark tests pass

lajohn4747 avatar Jun 07 '24 15:06 lajohn4747

Can we add an integration test?

Added one for HMA. I also plan to add more tests in our other synthesizers as well as well: https://github.com/datacebo/SDV-Enterprise/pull/621

lajohn4747 avatar Jun 07 '24 16:06 lajohn4747

So if you run the tests on enterprise (including the benchmarking) with these updates everything passes?

Yep all the tests and benchmark tests pass

Just wanted to note that HMA benchmark leaves out a lot of datasets because they take a very long time for the larger datasets and diagnostic reports do not go over HMA at all.

@amontanez24 @npatki

lajohn4747 avatar Jun 07 '24 17:06 lajohn4747