todo tracker

[Dustin-Ray]

There are a few todos spread throughout the repo, most of them are notating planned features that are not yet supported. See #149 for a few TimeStamp related todo!()s that actually would cause a panic under certain conditions.

Here are a few non-panicking todos labelled in doc comments throughout the repo that may or may not need some attention:

sxt-proof-of-sql/crates/proof-of-sql/README.md

Line 104 in 6740144

cargo install --example posql_db --path crates/proof-of-sql #TODO: update once this is published to crates.io

sxt-proof-of-sql/crates/proof-of-sql/src/base/commitment/column_commitments.rs

Line 355 in 6740144

// "int128_column" => [100i128, 200, 300, 400], TODO: enable this column once blitzar

sxt-proof-of-sql/crates/proof-of-sql/src/base/commitment/column_commitments.rs

Line 429 in 6740144

// "int128_column" => [100i128, 200, 300, 400], TODO: enable this column once blitzar

sxt-proof-of-sql/crates/proof-of-sql/src/base/commitment/table_commitment.rs

Line 517 in 6740144

// "int128_column" => [100i128, 200, 300, 400], TODO: enable this column once blitzar

sxt-proof-of-sql/crates/proof-of-sql/src/proof_primitive/mod.rs

Line 1 in 6740144

//! TODO: add docs

sxt-proof-of-sql/crates/proof-of-sql/src/sql/postprocessing/group_by_postprocessing.rs

Line 208 in 6740144

// TODO: Allow a filter

sxt-proof-of-sql/crates/proof-of-sql/src/sql/proof/mod.rs

Line 1 in 6740144

//! TODO: add docs

Work is currently underway on range check which should resolve this, see #141

sxt-proof-of-sql/crates/proof-of-sql/src/sql/proof_exprs/sign_expr.rs

Line 41 in 6740144

/// todo! make this more efficient and targeted at just the sign bit rather than all bits to create a proof

sxt-proof-of-sql/crates/proof-of-sql/tests/integration_tests.rs

Line 494 in 6740144

//TODO: This test uses postprocessing now. Check proof results once PROOF-765 is done.

sxt-proof-of-sql/crates/proof-of-sql-parser/src/intermediate_ast_tests.rs

Line 253 in 6740144

//TODO: Add unary negative operator so that it will no longer be necessary to use the minus sign with brackets

sxt-proof-of-sql/crates/proof-of-sql-parser/src/intermediate_ast_tests.rs

Lines 773 to 775 in 6740144

	// TODO: we should be able to pass this test.
	// But due to some lalrpop restriction, we aren't.
	// This problem will be addressed in a future PR.

[JayWhite2357]

Closing in favor of individual issues for each todo.