Skip to content

Lab 1 - submission in progress#19

Open
m-drop682 wants to merge 1 commit intoGU-CLASP:mainfrom
m-drop682:main
Open

Lab 1 - submission in progress#19
m-drop682 wants to merge 1 commit intoGU-CLASP:mainfrom
m-drop682:main

Conversation

@m-drop682
Copy link

Solved tasks 1, 2A and 2B
In progress 2C and 3

Tests 5, 6 and 7 are test cases for 2C and 3. I had previously written solutions for them in the code, but I removed them after I kept getting errors that were unexplainable to me. I tried implementing 2C by adding a counter for silences from the user in the IS, and updating that throughout the dialog. This updating and incrementing was quite confusing, and I though it be best to hand in the assignment and discuss the VG parts later.

Solved tasks 1, 2A and 2B
In progress 2C and 3

Tests 5, 6 and 7 are test cases for 2C and 3. I had previously written solutions for them in the code, but I removed them after I kept getting errors that were unexplainable to me. I tried implementing 2C by adding a counter for silences from the user in the IS, and updating that throughout the dialog. This updating and incrementing was quite confusing, and I though it be best to hand in the assignment and discuss the VG parts later.
Copy link
Member

@vladmaraev vladmaraev 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 with the G part of the lab! Nice to see the test for VG part being added, let me know if you have any questions regarding the VG part.

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