Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Assignment 1 #1

Open
wants to merge 23 commits into
base: master
Choose a base branch
from
Open

Assignment 1 #1

wants to merge 23 commits into from

Conversation

abendo
Copy link

@abendo abendo commented Sep 30, 2020

No description provided.

@ereator
Copy link
Collaborator

ereator commented Oct 6, 2020

I cannot accept this assignment because of the following issues:

Major Issue:

  • The code is not buildable: CI reports a build error

Minor issues:

  • Changed files that should not have been changed (e.g. ICamera.h, appveyor.yml, .gitignore, etc)
  • No rendered images were submitted

@abendo
Copy link
Author

abendo commented Oct 6, 2020 via email

@ereator
Copy link
Collaborator

ereator commented Oct 7, 2020

Dear Albrit Bendo,

the requirements that the code must be able to built with the CI (major issue) and about changing only those places of the code where indicated with the text

--- --- PUT YOUR CODE HERE ---

were communicated on the lecture.

In addition I have just took another look at your submission. It looks like you have submitted the solution for Assignment 1 CG course Fall 2019 for the current assignment. The current Assignment was simplified and is different from the Assignment of 2019 course. But you changed the code in those places where you were not expected to change it, and thus you made the assignment mode complex and difficult - you changed it back to the Assignment from 2019 course. This is a clear indication of plagiarism to me.

So, again, I can't accept your submission.

With the best regards,

Sergey Kosov

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