fix: python decompose model change and pipeline fix

[AngeloDanducci]

Misc PR

Type of PR

• Bug Fix
• New Feature
• Documentation
• Other

Description

• Link to Issue: fix: update python decompose example #566

Testing

• Tests added to the respective file if code was changed
• New code has 100% coverage if code as added
• Ensure existing tests and github automation passes (a maintainer will kick off the github automation when the rest of the PR is populated)

[github-actions]

The PR description has been updated. Please fill out the template for your PR to be reviewed.

[mergify]

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

• title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert|release)(?:\(.+\))?:

[AngeloDanducci]

@psschwei that issue you ran into should be solved with this update

@csbobby could use your feedback as well, the change in cli/decompose/pipeline.py assigns a validation strategy of llm if none exists - though I think the more correct long term change is probably to pass the validation strategy down when breaking up constraints. Prior to #445 a single constraint would not be broken down so this was a non issue.

It looks like this area is getting reworked in #556 so maybe this is fine in the interim.

[jakelorocco]

@csbobby, can you please confirm that you are okay with changing this and always having an LLM default?