Skip to content

refactor phase 5: API surface — one query(), eager validation, YAML schema, no how= #302

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
hussainsultan merged 6 commits into from
Aug 19, 2026
Merged

refactor phase 5: API surface — one query(), eager validation, YAML schema, no how= #302

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
13128a8
refactor!: one query() and one compare_periods, on the SemanticTable …
hussainsultan Aug 18, 2026
6ca1ead
feat: query() fails fast on unknown dimension/measure names
hussainsultan Aug 18, 2026
9487496
feat: YAML model configs are schema-validated at load
hussainsultan Aug 18, 2026
d62a7d9
refactor!: remove the vestigial how= parameter from semantic joins
hussainsultan Aug 18, 2026
f5cdede
refactor!: slim the top-level namespace; .name works on every expression
hussainsultan Aug 18, 2026
cdc8108
fix: un-commit WIP example scripts swept in by broad git-add
hussainsultan Aug 19, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions docs/md/doc/semantic-table.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -170,7 +170,7 @@ flights_with_deps.get_graph()['avg_distance_per_flight']['deps']
Use `graph_predecessors()` and `graph_successors()` to navigate dependencies:

```graph_traversal
from boring_semantic_layer import graph_predecessors, graph_successors
from boring_semantic_layer.graph_utils import graph_predecessors, graph_successors

graph = flights_with_deps.get_graph()

Expand Down Expand Up @@ -347,7 +347,8 @@ all_combinations = flights_st.join_cross(carriers)

### Requiring a Match

`join_one()` and `join_many()` only support `how="left"`. When a query should
`join_one()` and `join_many()` are always LEFT joins (there is no `how=`
parameter). When a query should
require a match, make the row removal explicit with a filter on a non-nullable
field from the right table:

Expand Down
16 changes: 0 additions & 16 deletions src/boring_semantic_layer/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,16 +26,8 @@
from .expr import (
SemanticModel,
SemanticTable,
to_tagged,
to_untagged,
)
from .graph_utils import (
graph_bfs,
graph_invert,
graph_predecessors,
graph_successors,
graph_to_dict,
)
from .ops import (
Dimension,
Measure,
Expand All @@ -44,7 +36,6 @@
ProfileError,
get_connection,
)
from .serialization import from_tagged
from .yaml import (
from_config,
from_yaml,
Expand All @@ -59,9 +50,7 @@
"SerializationError",
"UnknownFieldError",
"to_semantic_table",
"to_tagged",
"to_untagged",
"from_tagged",
"entity_dimension",
"time_dimension",
"SemanticModel",
Expand All @@ -73,11 +62,6 @@
"MCPSemanticModel",
"LangGraphBackend",
"options",
"graph_bfs",
"graph_invert",
"graph_predecessors",
"graph_successors",
"graph_to_dict",
"ProfileError",
"get_connection",
]
Expand Down
Loading
Loading