Skip to content

Improve context collection#15

Merged
brichet merged 11 commits into
QuantStack:mainfrom
Yahiewi:context
Jul 10, 2026
Merged

Improve context collection#15
brichet merged 11 commits into
QuantStack:mainfrom
Yahiewi:context

Conversation

@Yahiewi

@Yahiewi Yahiewi commented Jul 6, 2026

Copy link
Copy Markdown
Member

This PR improves the context collection by traversing preceding cells and formatting student inputs into structured XML tags for the LLM.

@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

Binder 👈 Launch a Binder on branch Yahiewi/jupyter-ai-tutor/context

@Yahiewi Yahiewi marked this pull request as draft July 6, 2026 10:37
@Yahiewi Yahiewi marked this pull request as ready for review July 6, 2026 15:35
@brichet

brichet commented Jul 8, 2026

Copy link
Copy Markdown
Member

This PR will conflict with #16.
Let merge #16 first and finalize this one after.

@brichet brichet added the enhancement New feature or request label Jul 9, 2026

@brichet brichet left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @Yahiewi for working on this, it looks good overall.

We should probably add a short description about the tags in the default TUTOR.md (<student_context>, <student_answer>, <reference_solution> and <evaluation_criteria>)

Comment thread src/index.ts Outdated
Comment thread src/index.ts Outdated
Comment thread src/index.ts
@Yahiewi

Yahiewi commented Jul 10, 2026

Copy link
Copy Markdown
Member Author

Thanks @brichet for the reviews. I just updated the PR to address all of them (aside from adding descriptions to the TUTOR.md as it's still not entirely clear how we should do that I think)

@brichet

brichet commented Jul 10, 2026

Copy link
Copy Markdown
Member

I added a short description of the content tags in 1cba91d

@brichet

brichet commented Jul 10, 2026

Copy link
Copy Markdown
Member

@Yahiewi if you want to take a look.

@Yahiewi

Yahiewi commented Jul 10, 2026

Copy link
Copy Markdown
Member Author

This looks great @brichet thanks! I think I'll open a PR to add the initial_source tag like we discussed, then another PR to implement a first version of the "Review" command.

@brichet brichet merged commit 6a85d88 into QuantStack:main Jul 10, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants