Skip to content

Comments

Final PR for Sem 2: Graded and Discriminating Essence + Refactor#35

Merged
ndangtt merged 105 commits intostacs-cp:mainfrom
vincepick:main
May 18, 2025
Merged

Final PR for Sem 2: Graded and Discriminating Essence + Refactor#35
ndangtt merged 105 commits intostacs-cp:mainfrom
vincepick:main

Conversation

@vincepick
Copy link
Collaborator

Includes full functionality for Graded and Discriminating instance generation using Essence problem models. This PR includes support for problems with and without a repair model, a significant refactor of the "wrapper.py" file, as well as numerous additional features.

vincepick and others added 30 commits November 8, 2024 20:58
…ation compatibility, added new helpers to essence_pipeline, and seem to have completed all the parameter handling for the essence runs
…ction, theoretically fully implemented but has yet to be run, has no support for the oracle, and needs further confirmation
@vincepick
Copy link
Collaborator Author

vincepick commented May 9, 2025

For a more detailed account of changes and justifications, please see the associated report.

The above testing pipeline passes, indicating both the newly implemented and existing functionalities should be present. There is a current issue with the Docker file pipeline I believe due to this being a merge request from the fork, but it isn't an actual error and should work again once this is merged.

The experiments directory was deleted from the main branch for an easier submission. However, if one wishes to see example runs, please see the discrim_teardown branch in my fork of AutoIG, which still contains a full experiments directory. This original branch final_essence still does contain the submission_outputs directory which contains the outputs used within the report, it was deleted from here to make the PR easier to review.

Below is the directory tree of all files edited to enable the new functionality. Click on any file to view the changes in this PR.

.
|-- .github/workflows/
│   |-- run_pr_tests.yml
│   |-- run_push_tests.yml
|-- scripts/
│   |-- evaluators/
│   │   |-- essence_discriminating.py
│   │   |-- essence_graded.py
│   |-- essence_pipeline_utils.py
│   |-- wrapper.py
│   |-- collect_results.py
│   |-- extract-md5sum.sh
│   |-- setup.py
│   |-- wrapper_helpers.py
│   |-- conf.py
│   |-- scenario.R
│   |-- testScripts
|-- DEV_README.md
|-- Dockerfile

@vincepick vincepick marked this pull request as ready for review May 12, 2025 22:06
@ndangtt ndangtt merged commit e09f5ae into stacs-cp:main May 18, 2025
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