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

Clean up trade flows #311

Merged
merged 3 commits into from
Feb 4, 2025
Merged

Clean up trade flows #311

merged 3 commits into from
Feb 4, 2025

Conversation

olejandro
Copy link
Member

No description provided.

Copy link

github-actions bot commented Feb 3, 2025

Regression test results on commit 7cfe361

         Benchmark    Time (s)                 GDX Diff     Accuracy       Correct    Additional
------------------  ----------  -----------------------  -----------  ------------  ------------
     DemoS_001-all   2.1   2.5            OK         OK  100.0 100.0    118    118      3      3
     DemoS_002-all   2.5   2.5            OK         OK  100.0 100.0    344    344      3      3
     DemoS_003-all   2.7   3.5            OK         OK  100.0 100.0    633    633      6      6
         DemoS_004   2.9   3.3            OK         OK  100.0 100.0    662    662     12     12
        DemoS_004a   2.9   3.0            OK         OK  100.0 100.0    665    665     12     12
DemoS_004a-ie-test   3.0   3.2            OK         OK  100.0 100.0    667    667     12     12
        DemoS_004b   2.9   3.5            OK         OK  100.0 100.0    665    665     12     12
     DemoS_004-all   2.9   3.3            OK         OK  100.0 100.0    667    667     12     12
     DemoS_005-all   3.5   4.0            13         13   99.7  99.7   1160   1160     12     12
     DemoS_006-all   4.1   4.5            13         13   99.7  99.7   1258   1258     12     12
     DemoS_007-all   5.0   5.7            13         13   99.8  99.8   2155   2155     12     12
  DemoS_007-all-1r   3.9   5.0            11         11   99.8  99.8   1179   1179     12     12
     DemoS_008-all   6.9   8.3            13         13   99.9  99.9   5333   5333     18     18
     DemoS_009-all   7.8   9.1            32         32   99.9  99.9   5807   5807     29     29
     DemoS_010-all   8.0   8.9           826        826   88.4  88.4   6139   6139     29     29
     DemoS_011-all   7.9   8.4           856        854   88.0  88.0   6150   6150     29     29
     DemoS_012-all   8.4   8.3           903        903   88.2  88.2   6306   6306     53     53
  DemoS_special-t1   4.2   4.1  Error runn… Error runn…   92.2  92.2   2108   2108     42     42
      TIMES-IE-all  23.0  22.8          3125       3125   97.5  97.5  42461  42461   2826   2826
      TIMES-IE-NoM  20.6  20.8           357        357   99.3  99.3  40529  40529    632    632
      TIMES-IE-MCB  21.4  20.2          1100       1100   97.6  97.6  42068  42068    632    632
      TIMES-NZ-KEA  25.2  24.0          3918       3869   98.3  98.3  76417  76417    456    409
      TIMES-NZ-TUI  17.3  17.1          3952       3903   98.3  98.3  76043  76043    472    425

2025-02-04 00:08:33.841 |     INFO : Total runtime: 196.15s (main: 188.92s)
2025-02-04 00:08:33.841 |     INFO : Change in runtime (negative == faster): +7.23s (+3.8%)
2025-02-04 00:08:33.841 |     INFO : Change in correct rows (higher == better): +0 (+0.0%)
2025-02-04 00:08:33.842 |     INFO : Change in additional rows: -94 (-1.8%)
2025-02-04 00:08:33.842 |  SUCCESS : No regressions. You're awesome!

This comment will be updated when new commits are added to the PR.

This way the topology takes into account regions specified according to the workbook name.
@olejandro olejandro marked this pull request as ready for review February 4, 2025 00:05
@siddharth-krishna siddharth-krishna merged commit 9d8795c into main Feb 4, 2025
2 checks passed
@siddharth-krishna siddharth-krishna deleted the olex/ire-processes branch February 4, 2025 05:16
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