Skip to content

Conversation

@JasonMarechal25
Copy link
Contributor

@JasonMarechal25 JasonMarechal25 commented Oct 24, 2025

End to End test for 6.2 : feature 13 : Export investment problem
Given the modeler system is described with an optim-config
Modeler should generate master, subproblems and structure file

We test that problems data are the same (number of rows, columns, ub, lb, type, etc) using xpress solver. Limit: We can't work with problems too big.
We test that structure files contains the same data

Todos, pending 6.2c

  • Fix master name (should be "master"
  • Handle subproblem names. Either use names and append ".mps" when reading them or use file name directly.
  • Use proper output directory. Use context.output_path, verify if correct

@pull-request-size pull-request-size bot added size/L and removed size/S labels Oct 24, 2025
@tbittar tbittar force-pushed the feature/add-modeler-invest-test branch from 0314c52 to b4c2f1a Compare October 24, 2025 14:11
Base automatically changed from feature/add-modeler-invest-test to develop October 28, 2025 08:21
@sonarqubecloud
Copy link

objective-contributions:
- id: invest_objective
location: master
- id: operational_objective
Copy link
Member

Choose a reason for hiding this comment

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

The input yaml files contain typos, please update your branch when the rest is merged

@@ -0,0 +1,168 @@
from behave import *
import xpress as xp
Copy link
Member

Choose a reason for hiding this comment

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

xpress requires a license and is nonfree, maybe we could find another python package for MPS parsing ?

Copy link
Member

Choose a reason for hiding this comment

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

@flomnes flomnes closed this Dec 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants