Skip to content

Creating augmented suggester#54

Merged
amit-sharma merged 30 commits into
mainfrom
creating_augmented_suggester
Jun 28, 2025
Merged

Creating augmented suggester#54
amit-sharma merged 30 commits into
mainfrom
creating_augmented_suggester

Conversation

@grace-sng7

Copy link
Copy Markdown
Collaborator

No description provided.

grace-sng7 and others added 16 commits May 18, 2025 20:30
Signed-off-by: Grace Sng <grace.sng75@gmail.com>
…sponding utils.

Signed-off-by: Grace Sng <grace.sng75@gmail.com>
Signed-off-by: Grace Sng <grace.sng75@gmail.com>
Signed-off-by: Grace Sng <grace.sng75@gmail.com>
Signed-off-by: Grace Sng <grace.sng75@gmail.com>
Signed-off-by: Grace Sng <grace.sng75@gmail.com>
Signed-off-by: Grace Sng <108954669+grace-sng7@users.noreply.github.com>
Signed-off-by: Grace Sng <108954669+grace-sng7@users.noreply.github.com>
Signed-off-by: Grace Sng <grace.sng75@gmail.com>
Signed-off-by: Grace Sng <grace.sng75@gmail.com>
Signed-off-by: Grace Sng <grace.sng75@gmail.com>

@amit-sharma amit-sharma 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 for the PR @grace-sng7 I've shared a few inline comments.

I also notice that the build is failing. I'm working on a fix and will update this PR.

Comment thread docs/notebooks/augmented_model_suggester_examples.ipynb
Comment thread pywhyllm/suggesters/augmented_model_suggester.py
Comment thread pywhyllm/utils/data_loader.py Outdated
Comment thread pywhyllm/suggesters/augmented_model_suggester.py Outdated
@grace-sng7

Copy link
Copy Markdown
Collaborator Author

@amit-sharma

Hi Dr. Sharma, thanks for the comments. I've added a few updates to the PR.

@amit-sharma amit-sharma 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 @grace-sng7 I just have one comment for the notebook.

Comment thread docs/notebooks/augmented_model_suggester_examples.ipynb Outdated
grace-sng7 and others added 9 commits June 11, 2025 00:07
Signed-off-by: Grace Sng <grace.sng75@gmail.com>
Signed-off-by: Grace Sng <grace.sng75@gmail.com>
Signed-off-by: Grace Sng <grace.sng75@gmail.com>
Signed-off-by: Grace Sng <grace.sng75@gmail.com>
Signed-off-by: Grace Sng <grace.sng75@gmail.com>
grace-sng7 and others added 3 commits June 28, 2025 00:00
Signed-off-by: Grace Sng <grace.sng75@gmail.com>
@grace-sng7 grace-sng7 requested a review from amit-sharma June 28, 2025 05:09

@amit-sharma amit-sharma 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 @grace-sng7 The new edits remove the installation errors. Merging. ..

@amit-sharma amit-sharma merged commit 6476b90 into main Jun 28, 2025
4 checks passed
@amit-sharma amit-sharma deleted the creating_augmented_suggester branch June 28, 2025 11:03
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.

2 participants