Skip to content

AnesIms A4#57

Open
AnesIms wants to merge 1 commit into
mainfrom
AnesIms-A4
Open

AnesIms A4#57
AnesIms wants to merge 1 commit into
mainfrom
AnesIms-A4

Conversation

@AnesIms

@AnesIms AnesIms commented May 5, 2026

Copy link
Copy Markdown
Collaborator

Please review :)

@najak04 najak04 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I could run the code just like it was explained in the read me and also it was clear to understand the implementation since I did it almost the same and there wasn't to much variety how to implement it. The only thing I saw in someone elses code which makes sense now is that all characters could be converted to small letters since it makes more sense to not have random capitals. But besides that I think there is not much to add and it works just like explained in lectures using a dictonary to represent the probabilities.:)

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