<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">Pierre,<br>
<br>
Yes - the CoinAssert should be commented. In fact it was - 9
months ago with change 2217.<br>
<br>
// set non basic free variables to fake
bounds<br>
// I don't think we should ever get here<br>
// yes we can if basis goes singular twice in
succession!<br>
//CoinAssert(!("should not be here"));<br>
<br>
John Forrest<br>
<br>
<br>
On 25/01/17 23:45, Pierre Moulon wrote:<br>
</div>
<blockquote
cite="mid:coin-or%2FClp%2Fcommit%2F1d65c46b4d4fe4c047fb6ae0b6e46a376dc8bee0%2F20619040@github.com"
type="cite">
<p><a moz-do-not-send="true" href="https://github.com/jjhforrest"
class="user-mention">@jjhforrest</a> I confirm that have the
CoinAssert as a comment is valuable.<br>
I use some LP feasibility test in some Bisection loops for
computer vision problems and updating to CLP: 1.16.10 make on of
my unit test fails.</p>
<p>Here the summary of the discussion<br>
<a moz-do-not-send="true"
href="https://github.com/openMVG/openMVG/issues/740"
class="issue-link js-issue-link"
data-url="https://github.com/openMVG/openMVG/issues/740"
data-id="202647551" data-error-text="Failed to load issue
title" data-permission-text="Issue title is private">openMVG/openMVG#740</a></p>
<p
style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>
You are receiving this because you were mentioned.<br>
Reply to this email directly, <a moz-do-not-send="true"
href="https://github.com/coin-or/Clp/commit/1d65c46b4d4fe4c047fb6ae0b6e46a376dc8bee0#commitcomment-20619040">view
it on GitHub</a>, or <a moz-do-not-send="true"
href="https://github.com/notifications/unsubscribe-auth/AGycHAgz5H6iMOzOrmZ--VumnJqEIsxwks5rV961gaJpZM4LuJHv">mute
the thread</a>.<img moz-do-not-send="true" alt=""
src="https://github.com/notifications/beacon/AGycHInMyASqQJmWIAXWJuXFXsQS8B1Hks5rV961gaJpZM4LuJHv.gif"
height="1" width="1"></p>
<div itemscope="" itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope=""
itemtype="http://schema.org/ViewAction">
<link itemprop="url"
href="https://github.com/coin-or/Clp/commit/1d65c46b4d4fe4c047fb6ae0b6e46a376dc8bee0#commitcomment-20619040">
<meta itemprop="name" content="View Commit">
</div>
<meta itemprop="description" content="View this Commit on
GitHub">
</div>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/coin-or/Clp","title":"coin-or/Clp","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/coin-or/Clp"}},"updates":{"snippets":[{"icon":"PERSON","message":"@pmoulon on 1d65c46: @jjhforrest I confirm that have the CoinAssert as a comment is valuable.\r\nI use some LP feasibility test in some Bisection loops for computer vision problems and updating to CLP: 1.16.10 make on of my unit test fails.\r\n\r\nHere the summary of the discussion\r\nhttps://github.com/openMVG/openMVG/issues/740"}],"action":{"name":"View Commit","url":"https://github.com/coin-or/Clp/commit/1d65c46b4d4fe4c047fb6ae0b6e46a376dc8bee0#commitcomment-20619040"}}}</script>
</blockquote>
<p><br>
</p>
</body>
</html>