Skip to content

add an option allowCommonSenseSolution in MatchCdOdConfig to support …#635

Closed
guard-who wants to merge 1 commit intodevfrom
issue-632
Closed

add an option allowCommonSenseSolution in MatchCdOdConfig to support …#635
guard-who wants to merge 1 commit intodevfrom
issue-632

Conversation

@guard-who
Copy link
Copy Markdown
Member

close #632

@jvoigtlaender
Copy link
Copy Markdown
Member

This would be the first time where a config option is introduced in modelling-tasks that has no meaning at all for users of "just" modelling-tasks without concept injection?

So maybe we should reconsider, and this additional configuration possibility should be introduced into the workflow of llm-task-wrappers. That is, a user would decide about this option when they start a concept injection run, not when they generate the original instances to be used as input for concept injection.

@guard-who
Copy link
Copy Markdown
Member Author

guard-who commented Mar 31, 2026

This would be the first time where a config option is introduced in modelling-tasks that has no meaning at all for users of "just" modelling-tasks without concept injection?

This is also my concern, because this configuration option will not used here, I considered using the Maybe type (Nothing is used to represent no usage).

If this option is configured in llm-task-wrappers, I'm unsure if the Maybe type is still necessary. Because when it equals Nothing, it means we want to display CD3, but not force concepts to match CD3. This is clear for configuring formal constraints in Alloy, but for LLM, the impact of displaying CD3 without any constraints or instructions is unpredictable (it might affect finding good concepts for CD1 and CD2). Actually, we don't need Nothing, right?

@jvoigtlaender
Copy link
Copy Markdown
Member

Let‘s have that discussion (about Nothing being needed or not) in the other repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Provide an option to control whether the LLM should take the hiddenReferenceCd into account when generating concepts

2 participants