Skip to content

Feedback#1

Open
github-classroom[bot] wants to merge 15 commits into
feedbackfrom
main
Open

Feedback#1
github-classroom[bot] wants to merge 15 commits into
feedbackfrom
main

Conversation

@github-classroom
Copy link
Copy Markdown
Contributor

@github-classroom github-classroom Bot commented Apr 6, 2026

👋! GitHub Classroom created this pull request as a place for your teacher to leave feedback on your work. It will update automatically. Don’t close or merge this pull request, unless you’re instructed to do so by your teacher.
In this pull request, your teacher can leave comments and feedback on your code. Click the Subscribe button to be notified if that happens.
Click the Files changed or Commits tab to see all of the changes pushed to the default branch since the assignment started. Your teacher can see this too.

Notes for teachers

Use this PR to leave feedback. Here are some tips:

  • Click the Files changed tab to see all of the changes pushed to the default branch since the assignment started. To leave comments on specific lines of code, put your cursor over a line of code and click the blue + (plus sign). To learn more about comments, read “Commenting on a pull request”.
  • Click the Commits tab to see the commits pushed to the default branch. Click a commit to see specific changes.
  • If you turned on autograding, then click the Checks tab to see the results.
  • This page is an overview. It shows commits, line comments, and general comments. You can leave a general comment below.
    For more information about this pull request, read “Leaving assignment feedback in GitHub”.

Subscribed: @StGCo

Copy link
Copy Markdown

@ProfMStoica ProfMStoica left a comment

Choose a reason for hiding this comment

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

This is a full solution with a detailed report. No video demo was submitted. I apreciate you openly sharing your AI chat history so I can give you feedback on it. While you started well asking questions about the meaning of DateTimes and TimeSpan you went beyond the allowed guidelines by posting the entire assignment, your entire code base and getting back answers with full solutions. In the process, you lost control of the development process. Having said that, I can see you worked hard to understand and developed the solution highly iteratively so please take this feedback constructively. It is not meant to be punitive. In the future please do not upload the PDF instructions of the assignment to AI and ask questions that are much more specific. Do not upload wholesale solutions and ask generic questions like "What is wrong with my code?"

@@ -0,0 +1,46 @@
[ChatGPT conversation](https://chatgpt.com/share/69e50eda-9268-83ea-90f1-a0844c2c96dd)
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 am glad I finally have access to your chat history and appreciate you including it. The others had expired because of how late I was in looking at them.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This goes beyond AI Fair Use Guidelines. It is also where you've let AI drive your solution instead of you driving it and using AI to help you learn as you did before this point.

image

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Troubleshooting with debugging and using error messages is a significant part and tool for learning and you've offloaded that learning to AI here.

image

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

We find out what is needed through debugging and testing, not by asking AI and implementing their suggestions.

image

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

When AI answers with a full solution, you know you went too far with fair use of AI

image

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

By publishing the assignment to AI you went above what was permitted. I can see you've used it to learn and iteratively develop your application, but it is still over the fair use line, which is to the detriment of your learning.

image image

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

You can see that you went above even what ChatGPT suggested to do according to your initial prompts. An ethical AI would have stopped answering questions like this and helped you figure things out by yourself but ChatGPT is not an ethical AI and is not interested in your learning although it sounds like it is. It is just a pattern matching algorithm.

image

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Asking for a peer review of working code to improve it is useful, but it is also an advanced use of AI. Letting AI drive the development of the code, the troubleshooting and identification of errors, the fixing and everything in between is not useful to your learning.

image


Throughout this assignment, I leisurely took the time to implement the code, following the UML.
I at some points felt lost, and paused, came back after a few days used ChatGPT as a way to unblock the path forward.
I know very well that I need to parse the final result to get all the pieces together. I find myself often seeing the end, and not how to get there.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The answer to this is debug your code in small increments and see how it works. You don't need to ask ChatGPT how it works, you can see it yourself.

Throughout this assignment, I leisurely took the time to implement the code, following the UML.
I at some points felt lost, and paused, came back after a few days used ChatGPT as a way to unblock the path forward.
I know very well that I need to parse the final result to get all the pieces together. I find myself often seeing the end, and not how to get there.
The question I MUST ask myself going forward is: What do I need to get to the end?
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Writing comments before coding allows you to break the problem down into small problems.

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.

1 participant