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

Peerfeedback model project #9

Open
Vithushaviknes opened this issue May 13, 2024 · 0 comments
Open

Peerfeedback model project #9

Vithushaviknes opened this issue May 13, 2024 · 0 comments

Comments

@Vithushaviknes
Copy link

1. The most elegant solution in the project was:
In general, the way you have incorporated classes is done very nicely. The code here is pretty compact and efficient, without many unnecessary steps. In addition, this makes the project easier to read, since much of the more complex code is in the py file.

2. The hardest section of code in the project to understand was:
The simulation part, especially the ‘simulate_before_s’ and ‘simulate_after_s’ functions can be pretty hard to understand, maybe these functions could benefit from some more explanations in the beginning, describing what the code does.

3. This part of the project could be better documented:
In general, the code in the py.file could be better documented, especially the simulation codes as mentioned previously.

4. An idea for an improvement/clarification could be:
Since your code for all of your figures are pretty similar, you could create a function for them too in your py.file. This would make your project even easier to read and understand.

5. An idea for an extension could be:
Your figures are all visually appealing and easy to understand. A way to improve them could be by making interactive plots for the different parameter values. In this way one figure would be enough to come across with your conclusions and you could show how your results change with even more values for the parameters. Otherwise you have done a really good job with the project!

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

No branches or pull requests

1 participant