Skip to content

Updated crop yield concept captions. Examples only for now.#103

Merged
vdplasthijs merged 1 commit into
developfrom
feature/sample-concept-captions
May 11, 2026
Merged

Updated crop yield concept captions. Examples only for now.#103
vdplasthijs merged 1 commit into
developfrom
feature/sample-concept-captions

Conversation

@robknapen
Copy link
Copy Markdown
Collaborator

@robknapen robknapen commented May 7, 2026

What does this PR do?

Adds examples of concept captions for the crop yield use case, more aligned to the (proxy) information that (might) be encoded in EO embeddings.

Before submitting

  • Did you make sure title is self-explanatory and the description concisely explains the PR?
  • Did you make sure your PR does only one thing, instead of bundling different changes together?
  • Did you list all the breaking changes introduced by this pull request?
  • Did you test your PR locally with pytest command?

Copy link
Copy Markdown
Contributor

@gabrieletijunaityte gabrieletijunaityte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder, did you try training the alignment with these captions? I think having threshold values in the concept caption is not so useful for llms.

@vdplasthijs
Copy link
Copy Markdown
Collaborator

@robknapen do these threshold values come from the elbow method or hand-picked?

@robknapen
Copy link
Copy Markdown
Collaborator Author

@gabrieletijunaityte @vdplasthijs Threshold values are hand-picked, and I haven't tried training alignment so far. For now I am mostly trying to figure out which concepts to use in the captions that make sense for the crop yield use case. It is also fine to keep them out of the codebase until I have more definitive templates of course.

@vdplasthijs
Copy link
Copy Markdown
Collaborator

hi @robknapen , no that's totally fine, we were both just wondering I think! Keep them in for now, we can always used elbow values to overwrite/test both. Thanks

@vdplasthijs vdplasthijs merged commit 8342540 into develop May 11, 2026
4 checks passed
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.

3 participants