Support tables without bounding box columns

[imagejan]

When adding a source segmentation including table data using (add_source_to_dataset), the table is required to have bb_min_* and bb_max_* columns, although their presence shouldn't be strictly required.

The column checking happens here:

mobie-utils-python/mobie/tables/default_table.py

Lines 17 to 22 in d02ca53

	expected_column_names = {
	"label_id",
	"anchor_x", "anchor_y",
	"bb_min_x", "bb_min_y",
	"bb_max_x", "bb_max_y",
	}

and in the validation code here:

mobie-utils-python/mobie/validation/tables.py

Lines 74 to 77 in 0e2a5fd

	default_columns = [
	"label_id", "anchor_x", "anchor_y",
	"bb_min_x", "bb_min_y", "bb_max_x", "bb_max_y"
	]

[constantinpape]

Good point! I am implementing less strict checks here: #99

[constantinpape]

This is available on the current master branch now. @imagejan: Do you have an environment where you can test this from master?

[imagejan]

@constantinpape thanks for the quick fix. This works fine for me now.

A flag to suppress the warnings (about missing "recommended" columns) would be great though, otherwise my output gets spammed when adding many tables. 🙂

[constantinpape]

A flag to suppress the warnings (about missing "recommended" columns) would be great though, otherwise my output gets spammed when adding many tables. slightly_smiling_face

Ok, I'll look into it ;).

[constantinpape]

This is now implemented in #102. You can pass suppress_warnings=True to add_source_to_dataset, which should prevent the warnings from missing recommended columns. I haven't tested this yet though, so let me know if anything unexpected happens when you use this.

[constantinpape]

This will be included in the new release and I will close the issue. Feel free to reopen if anything is still missing or does not work as expected @imagejan.