Skip to content

Feat/ga checks (#127)#149

Merged
CIGbalance merged 2 commits intomainfrom
feat/ga_consistency
Apr 8, 2026
Merged

Feat/ga checks (#127)#149
CIGbalance merged 2 commits intomainfrom
feat/ga_consistency

Conversation

@CIGbalance
Copy link
Copy Markdown
Collaborator

  • Add PorkchopPlot, KinematicsRobotArm, VehicleDynamics real-world problems

  • Add global search box, and update webpage.

  • Update reference links to dois.

  • Update README with form link

Added a link to a form for submitting problems, suites, or generators.

  • Remove 'now' from added form text

  • github action that triggers checks

  • fixing syntax

  • only execute when the file changes

  • Add link on the website to the google form for problem/benchmark submission. (Add link on the website to the google form for problem/benchmark submission. #136)

  • Add problems from Tailoring RSP workshop

  • Minor fixes and website generation.

  • Fix field value consistency for searchability.

  • Change links processing for html conversion

  • Also process links starting with www

  • testing tests and gitignore


* Add PorkchopPlot, KinematicsRobotArm, VehicleDynamics real-world problems

* Add global search box, and update webpage.

* Update reference links to dois.

* Update README with form link

Added a link to a form for submitting problems, suites, or generators.

* Remove 'now' from added form text

* github action that triggers checks

* fixing syntax

* only execute when the file changes

* Add link on the website to the google form for problem/benchmark submission. (#136)

* Add problems from Tailoring RSP workshop

* Minor fixes and website generation.

* Fix field value consistency for searchability.

* Change links processing for html conversion

* Also process links starting with www

* testing tests and gitignore

---------

Co-authored-by: ShuaiqunPan <s.pan@liacs.leidenuniv.nl>
Co-authored-by: Shuaiqun (Daniel) Pan <122439592+ShuaiqunPan@users.noreply.github.com>
Co-authored-by: Koen van der Blom <kvdb@cwi.nl>
Co-authored-by: Diederick Vermetten <44877568+Dvermetten@users.noreply.github.com>
Co-authored-by: Koen van der Blom <5031234+kvdblom@users.noreply.github.com>
Co-authored-by: Dvermetten <d.vermetten@gmail.com>
Copilot AI review requested due to automatic review settings April 8, 2026 08:56
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the OPL dataset and the generated GitHub Pages site by adding new benchmark entries, improving link handling/search UX, and introducing a GitHub Action to validate new submissions.

Changes:

  • Add multiple new problems/suites to problems.yaml and update the generated HTML tables/pages accordingly.
  • Update HTML generation to auto-link URLs inside table cells and enable a global search box in the DataTable UI.
  • Add a CI workflow intended to validate utils/new_problem.yaml changes, and update contribution docs with a Google Form submission link.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
yaml_to_html.py Adds URL detection/linkification and adjusts Pandas HTML output settings.
utils/new_problem.yaml Updates the submission template content.
README.md Adds a Google Form link for submitting problems/benchmarks.
problems.yaml Appends new problems/suites and associated metadata.
docs/problems.html Regenerated problems table HTML reflecting new data/link handling.
docs/javascript.html Enables DataTables global search UI element.
docs/index.html Updates landing page copy and regenerated table/scripts.
docs/header.html Adds Google Form link to the site header.
.gitignore Expands ignore patterns (Python/tooling oriented).
.github/workflows/new_problem_check.yml Introduces CI workflow to validate new problem YAML submissions.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

pip install -r utils/requirements.txt

- name: Run New Problem Check
run: |
Copy link

Copilot AI Apr 8, 2026

Choose a reason for hiding this comment

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

This workflow runs utils/validate_yaml.py utils/new_problem.yaml, but validate_yaml.py explicitly errors if name is still template. Since utils/new_problem.yaml is kept as a template in-repo, this check will fail for this PR (and for any future edits that don’t update the template name). Consider either (1) running validation only on PRs where contributors have replaced the template with real entries, (2) adjusting the validator to allow name: template when running in CI, or (3) using a separate submission file path instead of the shared template.

Suggested change
run: |
run: |
if grep -Eq '^[[:space:]]*name:[[:space:]]*template[[:space:]]*$' utils/new_problem.yaml; then
echo "utils/new_problem.yaml is still the template; skipping validation."
exit 0
fi

Copilot uses AI. Check for mistakes.
@CIGbalance CIGbalance merged commit a4b8be1 into main Apr 8, 2026
1 check failed
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