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

Negative cycles #33

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

Negative cycles #33

wants to merge 14 commits into from

Conversation

MichielStock
Copy link
Owner

No description provided.

@MichielStock
Copy link
Owner Author

  • rename your notebook to something more telling
  • don't use capital letters in function or variable names
  • you might use Graphs.jl to plot some examples
  • check the blog "Dijkstra in disguise" for more examples and pointers, can be found in the folder on shortest paths on Ufora.
  • Try to find a couple of interesting examples why you can explain the algorithm on. Check the project of Douwe on maximum flow for an excellent example.
  • You don't have to provide a lot of code, just explain it well

Good luck.

@shvhoye
Copy link

shvhoye commented Jan 31, 2022

STMO review 2022:

  • I like the figures and animations, text is clear
  • Love how tidy the code is
  • The only thing I could think of that might improve the notebook would be a table of contents (TOC) at the beginning + a TOC of everything that is in the appendix and functions, just to get an overview
  • Sad it can't make me rich though

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.

3 participants