Skip to content

Fix scipy warnings - Optimal transport and LP intro - #322

Merged
Smit-create merged 2 commits into
mainfrom
i-321
Apr 11, 2023
Merged

Fix scipy warnings - Optimal transport and LP intro#322
Smit-create merged 2 commits into
mainfrom
i-321

Conversation

@Smit-create

@Smit-create Smit-create commented Mar 15, 2023

Copy link
Copy Markdown
Member

Fixes #321

@Smit-create Smit-create changed the title Fix scipy warnings - Optimal transport lecture Fix scipy warnings - Optimal transport and LP intro Mar 15, 2023
@Smit-create Smit-create added the maintenance Housekeeping: refactors, tooling, infra, style, env upkeep label Mar 15, 2023
@github-actions

Copy link
Copy Markdown

@mmcky

mmcky commented Apr 11, 2023

Copy link
Copy Markdown
Contributor

thanks @Smit-create I have noticed the output is now different and doesn't say Optimization terminated successfully. Is this b/c the output from the package has changed?

@Smit-create

Copy link
Copy Markdown
Member Author

Is this b/c the output from the package has changed?

The method used in the backend is changed now to highs which previously was revised simplex. So, I guess the logs were method specific, but I can see this message in the output:

           con: array([0., 0., 0.])
 crossover_nit: 0
         eqlin:  marginals: array([ 75072.24525694,   4648.8252293 , -20000.        ])
  residual: array([0., 0., 0.])
           fun: 141018.2434979269
       ineqlin:  marginals: array([-24927.75474306, -50000.        ])
  residual: array([0., 0.])
         lower:  marginals: array([0., 0., 0., 0., 0., 0., 0.])
  residual: array([       inf,        inf,        inf,        inf,        inf,
              inf, 0.00147003])
       message: 'Optimization terminated successfully. (HiGHS Status 7: Optimal)'
           nit: 0
         slack: array([0., 0.])
        status: 0
       success: True
         upper:  marginals: array([     0.        ,      0.        ,      0.        , -95072.24525694,
       -24648.8252293 ,      0.        ,      0.        ])
  residual: array([       inf,        inf,        inf, 0.        , 0.        ,
       0.16502827,        inf])
             x: array([ 1.37644176,  1.29852997,  1.22502827,  0.        ,  0.        ,
       -0.16502827,  0.00147003])

@mmcky

mmcky commented Apr 11, 2023

Copy link
Copy Markdown
Contributor

Oh I can see it now about half way down. It used to be much cleaner output :-)

@mmcky
mmcky self-requested a review April 11, 2023 10:30

@mmcky mmcky left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

thanks @Smit-create this is looking good to me.

@Smit-create
Smit-create merged commit e22fab7 into main Apr 11, 2023
@Smit-create
Smit-create deleted the i-321 branch April 11, 2023 11:02
mmcky added a commit that referenced this pull request Nov 4, 2025
- Install quantecon-book-theme from PR #322
- Now testing both sphinx-exercise PR #81 and theme PR #322
- Related: QuantEcon/quantecon-book-theme#322
mmcky added a commit that referenced this pull request Nov 5, 2025
- Change from PR branch to released version 0.10.1
- Theme PR #322 has been merged and released
- Still testing sphinx-exercise PR #81
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Housekeeping: refactors, tooling, infra, style, env upkeep

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Warnings from scipy

2 participants