Skip to content

Assignment 1#1

Open
ManpreetkBhamrah wants to merge 9 commits intomainfrom
Assignment-1
Open

Assignment 1#1
ManpreetkBhamrah wants to merge 9 commits intomainfrom
Assignment-1

Conversation

@ManpreetkBhamrah
Copy link
Owner

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)

Completed Assignment 1.

What did you learn from the changes you have made?

I have learned about using the LLM to summarize an article, learned how to write formatted and detailed prompts. Also learned how can I evaluate the responses of LLM according to my requirement like adjusting the tone of response. Using the DeepEvaL parameters. Understanding the importance of temperature.

Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?

It was a new learning for me I understand and implemented the approach provided.

Were there any challenges? If so, what issue(s) did you face? How did you overcome it?

Most of the Assignment was based on the labs that we have. Links to the documentation really helped. I asked questions in the Office Hours Sessions. I used the examples from the documentation and review the labs.
For assessment questions of SummarizationMetric, I was getting a score of 0.000 for all the times.
To identify the issue I took help from copilot. It responded that the reason might be assessment questions are stricter and are resulting in a 'No'. I modified my questions to get the response of SummarizationMetric.

How were these changes tested?

I have run the code and there are two versions of the prompt. The first one gave me the response
image
After I modified the system prompt, I added more detail to the role. It improved the overall Summarization Score and Tonality Score.

image

A reference to a related issue in your repository (if applicable)

Checklist

  • I can confirm that my changes are working as intended

Copy link

@Dmytro-Bonislavskyi Dmytro-Bonislavskyi left a comment

Choose a reason for hiding this comment

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

Well done!

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