Skip to content

Rename BBOB to BBOB_test in problems.yaml #162

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.

Sign up for GitHub

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

Closed
CIGbalance wants to merge 2 commits into from
Closed

Rename BBOB to BBOB_test in problems.yaml #162

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
1a031ce
Rename BBOB to BBOB_test in problems.yaml
CIGbalance Apr 21, 2026
d350164
Auto-update HTML files
CIGbalance Apr 21, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions docs/index.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ <h2>OPL &ndash; Optimisation problem library</h2>
</thead>
<tbody>
<tr>
<td>BBOB</td>
<td>BBOB_test</td>
<td></td>
<td>suite</td>
<td>1</td>
Expand Down Expand Up @@ -911,7 +911,7 @@ <h2>OPL &ndash; Optimisation problem library</h2>
<td>no</td>
<td>artificial</td>
<td><a href="https://doi.org/10.1145/3449726.3459521">https://doi.org/10.1145/3449726.3459521</a></td>
<td>None</td>
<td></td>
Copy link

Copilot AI Apr 21, 2026

Choose a reason for hiding this comment

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

As with docs/problems.html, docs/index.html is produced by yaml_to_html.py. These None→empty edits appear to be output-level changes not explained by the PR’s stated goal; updating the generator/YAML and regenerating will keep the docs deterministic and avoid future reintroductions.

Suggested change
<td></td>
<td>None</td>

Copilot uses AI. Check for mistakes.
</tr>
<tr>
<td>Onemax+Sphere / DeceptiveTrap+RotatedEllipsoid</td>
Expand All @@ -926,7 +926,7 @@ <h2>OPL &ndash; Optimisation problem library</h2>
<td>no</td>
<td>artificial</td>
<td><a href="https://doi.org/10.1145/3449726.3459521">https://doi.org/10.1145/3449726.3459521</a></td>
<td>None</td>
<td></td>
Copy link

Copilot AI Apr 21, 2026

Choose a reason for hiding this comment

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

Same generated-output concern here: prefer addressing missing-value rendering in yaml_to_html.py (or in problems.yaml) and regenerating, rather than patching individual HTML table cells.

Copilot uses AI. Check for mistakes.
</tr>
<tr>
<td>InverseDeceptiveTrap+RotatedEllipsoid / DeceptiveTrap+RotatedEllipsoid</td>
Expand All @@ -941,7 +941,7 @@ <h2>OPL &ndash; Optimisation problem library</h2>
<td>no</td>
<td>artificial</td>
<td><a href="https://doi.org/10.1145/3449726.3459521">https://doi.org/10.1145/3449726.3459521</a></td>
<td>None</td>
<td></td>
Copy link

Copilot AI Apr 21, 2026

Choose a reason for hiding this comment

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

Same issue as the other None→empty changes: if the docs are intended to be regenerated, manual edits here will likely be lost. Updating the generator/YAML and regenerating will keep the output consistent.

Copilot uses AI. Check for mistakes.
</tr>
<tr>
<td>PorkchopPlotInterplanetaryTrajectory</td>
Expand Down Expand Up @@ -1050,7 +1050,7 @@ <h2>OPL &ndash; Optimisation problem library</h2>
</tr>
<tr>
<td>TulipaEnergy</td>
<td>Determine the optimal investment and operation decisions for different types of assets in the energy system (production, consumption, conversion, storage, and transport), while minimizing loss of load.</td>
<td>Determine the optimal investment and operation decisions&nbsp;&nbsp;for different types of assets in the energy system (production, consumption, conversion, storage, and transport), while minimizing loss of load.</td>
Copy link

Copilot AI Apr 21, 2026

Choose a reason for hiding this comment

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

Embedding &nbsp;&nbsp; in the generated table content makes the docs harder to maintain and can cause non-wrapping whitespace in narrow views. Prefer fixing the underlying YAML text (remove accidental double spaces) or normalizing whitespace in yaml_to_html.py, then regenerate docs/index.html.

Suggested change
<td>Determine the optimal investment and operation decisions&nbsp;&nbsp;for different types of assets in the energy system (production, consumption, conversion, storage, and transport), while minimizing loss of load.</td>
<td>Determine the optimal investment and operation decisions for different types of assets in the energy system (production, consumption, conversion, storage, and transport), while minimizing loss of load.</td>

Copilot uses AI. Check for mistakes.
<td>Problem Suite</td>
<td>1</td>
<td>scalable</td>
Expand Down
10 changes: 5 additions & 5 deletions docs/problems.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
</thead>
<tbody>
<tr>
<td>BBOB</td>
<td>BBOB_test</td>
<td></td>
<td>suite</td>
<td>1</td>
Expand Down Expand Up @@ -885,7 +885,7 @@
<td>no</td>
<td>artificial</td>
<td><a href="https://doi.org/10.1145/3449726.3459521">https://doi.org/10.1145/3449726.3459521</a></td>
<td>None</td>
<td></td>
Copy link

Copilot AI Apr 21, 2026

Choose a reason for hiding this comment

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

The HTML docs in docs/ are generated from problems.yaml via yaml_to_html.py. This PR changes the rendered output beyond the stated rename (e.g., removing literal None display), but the generator script isn’t updated here; these manual edits are likely to drift or be overwritten next time yaml_to_html.py is run. Consider fixing this at the source by adjusting yaml_to_html.py to render missing values consistently (e.g., normalize None/NA to empty strings) and then regenerate/commit the HTML outputs.

Copilot uses AI. Check for mistakes.
</tr>
<tr>
<td>Onemax+Sphere / DeceptiveTrap+RotatedEllipsoid</td>
Expand All @@ -900,7 +900,7 @@
<td>no</td>
<td>artificial</td>
<td><a href="https://doi.org/10.1145/3449726.3459521">https://doi.org/10.1145/3449726.3459521</a></td>
<td>None</td>
<td></td>
Copy link

Copilot AI Apr 21, 2026

Choose a reason for hiding this comment

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

Same as above: this None→empty change looks like a generated-output difference rather than a source change. To keep docs/problems.html reproducible, update yaml_to_html.py (or the YAML data) and regenerate instead of patching individual cells in the HTML.

Copilot uses AI. Check for mistakes.
</tr>
<tr>
<td>InverseDeceptiveTrap+RotatedEllipsoid / DeceptiveTrap+RotatedEllipsoid</td>
Expand All @@ -915,7 +915,7 @@
<td>no</td>
<td>artificial</td>
<td><a href="https://doi.org/10.1145/3449726.3459521">https://doi.org/10.1145/3449726.3459521</a></td>
<td>None</td>
<td></td>
Copy link

Copilot AI Apr 21, 2026

Choose a reason for hiding this comment

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

Same issue here: editing generated HTML cell content directly is brittle. Prefer to update the YAML source (e.g., make the field truly empty) or update yaml_to_html.py to handle missing values, then regenerate the docs.

Copilot uses AI. Check for mistakes.
</tr>
<tr>
<td>PorkchopPlotInterplanetaryTrajectory</td>
Expand Down Expand Up @@ -1024,7 +1024,7 @@
</tr>
<tr>
<td>TulipaEnergy</td>
<td>Determine the optimal investment and operation decisions for different types of assets in the energy system (production, consumption, conversion, storage, and transport), while minimizing loss of load.</td>
<td>Determine the optimal investment and operation decisions&nbsp;&nbsp;for different types of assets in the energy system (production, consumption, conversion, storage, and transport), while minimizing loss of load.</td>
Copy link

Copilot AI Apr 21, 2026

Choose a reason for hiding this comment

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

Replacing plain whitespace with &nbsp;&nbsp; makes the rendered content harder to maintain and can affect line wrapping/responsiveness. If the goal is to remove an accidental double-space in the source text, it’s better to fix the underlying YAML text (or normalize whitespace during HTML generation) and regenerate the docs, rather than embedding non-breaking spaces in the HTML output.

Suggested change
<td>Determine the optimal investment and operation decisions&nbsp;&nbsp;for different types of assets in the energy system (production, consumption, conversion, storage, and transport), while minimizing loss of load.</td>
<td>Determine the optimal investment and operation decisions for different types of assets in the energy system (production, consumption, conversion, storage, and transport), while minimizing loss of load.</td>

Copilot uses AI. Check for mistakes.
<td>Problem Suite</td>
<td>1</td>
<td>scalable</td>
Expand Down
2 changes: 1 addition & 1 deletion problems.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
- name: BBOB
- name: BBOB_test
Copy link

Copilot AI Apr 21, 2026

Choose a reason for hiding this comment

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

BBOB_test is the only suite/problem name in problems.yaml that uses an underscore; other names consistently use hyphens or spaces. If this is intended to be a public-facing identifier, consider renaming to match the established naming pattern (e.g., BBOB-test or BBOB (test)) to avoid an inconsistent entry and potential downstream parsing/slug issues.

Suggested change
- name: BBOB_test
- name: BBOB-test

Copilot uses AI. Check for mistakes.
Copy link

Copilot AI Apr 21, 2026

Choose a reason for hiding this comment

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

PR title suggests only a rename in problems.yaml, but this PR also changes generated documentation HTML (including unrelated whitespace/missing-value rendering). Consider updating the PR title/description to reflect the full scope, or splitting the docs-regeneration/formatting changes into a separate commit/PR for easier review.

Copilot uses AI. Check for mistakes.
suite/generator/single: suite
objectives: '1'
dimensionality: scalable
Expand Down