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

Implement Mod for Network Design Problem #37

Closed
wants to merge 4 commits into from

Conversation

stevedwards
Copy link

Description

Implements #36

For a new Mod contribution, please include the issue number for the Mod proposal and check through the list below.

Checklist

  • Implementation:
    • Implementation of the Mod in the gurobi_optimods installable package
    • Tests for the Mod implementation in tests/
    • Docstrings for public API, correctly linked using sphinx-autodoc
  • Documentation page:
    • Problem specification with description tab and optimzation model tab
    • Example of the input data format (use gurobi_optimods.datasets for loading data)
    • Runnable code example
    • Presentation of solutions

@gglockner
Copy link

I wanted to comment on the idea of networkx vs Pandas:

  • networkx is the preferred API for graph algorithms. Anyone working on pure network applications is likely to use networkx.
  • pandas is the preferred tool for data scientists and is far more popular than networkx.

So which API should be chosen should depend on who is the target for the tool.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


edwards seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@simonbowly simonbowly closed this Jul 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants