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

Problem statement: Service Placement #1

Open
wants to merge 22 commits into
base: main
Choose a base branch
from

Conversation

noegodinho
Copy link

Problem statement for the Service Placement problem.

Contains a markdown file with the description of the problem, instance and solutions files and their explanations.

@adbjesus
Copy link
Contributor

adbjesus commented Jun 2, 2024

@noegodinho for the license of the missing files (txt files inside data) you may consider putting license information inside the .reuse/dep5 (at the root of the repository) as it makes it easier to apply the same license to several files at once (there is already an example there). In the next version of reuse (in a few weeks I think) it will be possible to have a REUSE.toml file for this in each problem folder (and we will convert to that once it is available), but for now .reuse/dep5 is the only option for "mass" license.

@adbjesus adbjesus self-requested a review June 2, 2024 23:00
@noegodinho
Copy link
Author

Thank you for the guidance @adbjesus. Hopefully this last commit solves the problem.

Copy link
Contributor

@adbjesus adbjesus left a comment

Choose a reason for hiding this comment

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

Sorry for taking a while to reply back. I just went over the whole submission and there a few things that I would suggest changing. Many of the comments are just some minor formatting improvements. However, there are a few critical ones. Still, I think they should be fast to change.

problems/service-placement/problem-statement-sp.md Outdated Show resolved Hide resolved
.reuse/dep5 Show resolved Hide resolved
problems/service-placement/problem-statement-sp.md Outdated Show resolved Hide resolved
problems/service-placement/problem-statement-sp.md Outdated Show resolved Hide resolved
problems/service-placement/problem-statement-sp.md Outdated Show resolved Hide resolved
problems/service-placement/problem-statement-sp.md Outdated Show resolved Hide resolved
problems/service-placement/problem-statement-sp.md Outdated Show resolved Hide resolved
problems/service-placement/support/evaluation.py Outdated Show resolved Hide resolved
problems/service-placement/support/generator.py Outdated Show resolved Hide resolved
problems/service-placement/support/generator.py Outdated Show resolved Hide resolved
noegodinho and others added 18 commits September 19, 2024 11:48
Co-authored-by: Alexandre Jesus <adbjesus@gmail.com>
Co-authored-by: Alexandre Jesus <adbjesus@gmail.com>
Co-authored-by: Alexandre Jesus <adbjesus@gmail.com>
Co-authored-by: Alexandre Jesus <adbjesus@gmail.com>
Co-authored-by: Alexandre Jesus <adbjesus@gmail.com>
Co-authored-by: Alexandre Jesus <adbjesus@gmail.com>
Co-authored-by: Alexandre Jesus <adbjesus@gmail.com>
Co-authored-by: Alexandre Jesus <adbjesus@gmail.com>
Co-authored-by: Alexandre Jesus <adbjesus@gmail.com>
Co-authored-by: Alexandre Jesus <adbjesus@gmail.com>
Co-authored-by: Alexandre Jesus <adbjesus@gmail.com>
Co-authored-by: Alexandre Jesus <adbjesus@gmail.com>
Co-authored-by: Alexandre Jesus <adbjesus@gmail.com>
Co-authored-by: Alexandre Jesus <adbjesus@gmail.com>
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