<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<style>
body {
font-family: "Helvetica Neue", Helvetica, Arial, sans-serif;
padding:1em;
margin:auto;
background:#fefefe;
}
h1, h2, h3, h4, h5, h6 {
font-weight: bold;
}
h1 {
color: #000000;
font-size: 28pt;
}
h2 {
border-bottom: 1px solid #CCCCCC;
color: #000000;
font-size: 24px;
}
h3 {
font-size: 18px;
}
h4 {
font-size: 16px;
}
h5 {
font-size: 14px;
}
h6 {
color: #777777;
background-color: inherit;
font-size: 14px;
}
hr {
height: 0.2em;
border: 0;
color: #CCCCCC;
background-color: #CCCCCC;
display: inherit;
}
p, blockquote, ul, ol, dl, li, table, pre {
margin: 15px 0;
}
a, a:visited {
color: #4183C4;
background-color: inherit;
text-decoration: none;
}
#message {
border-radius: 6px;
border: 1px solid #ccc;
display:block;
width:100%;
height:60px;
margin:6px 0px;
}
button, #ws {
font-size: 12 pt;
padding: 4px 6px;
border-radius: 5px;
border: 1px solid #bbb;
background-color: #eee;
}
code, pre, #ws, #message {
font-family: Monaco;
font-size: 10pt;
border-radius: 3px;
background-color: #F8F8F8;
color: inherit;
}
code {
border: 1px solid #EAEAEA;
margin: 0 2px;
padding: 0 5px;
}
pre {
border: 1px solid #CCCCCC;
overflow: auto;
padding: 4px 8px;
}
pre > code {
border: 0;
margin: 0;
padding: 0;
}
#ws { background-color: #f8f8f8; }
.bloop_markdown table {
border-collapse: collapse;
font-family: Helvetica, arial, freesans, clean, sans-serif;
color: rgb(51, 51, 51);
font-size: 15px; line-height: 25px;
padding: 0; }
.bloop_markdown table tr {
border-top: 1px solid #cccccc;
background-color: white;
margin: 0;
padding: 0; }
.bloop_markdown table tr:nth-child(2n) {
background-color: #f8f8f8; }
.bloop_markdown table tr th {
font-weight: bold;
border: 1px solid #cccccc;
margin: 0;
padding: 6px 13px; }
.bloop_markdown table tr td {
border: 1px solid #cccccc;
margin: 0;
padding: 6px 13px; }
.bloop_markdown table tr th :first-child, table tr td :first-child {
margin-top: 0; }
.bloop_markdown table tr th :last-child, table tr td :last-child {
margin-bottom: 0; }
.bloop_markdown blockquote{
border-left: 4px solid #dddddd;
padding: 0 15px;
color: #777777; }
blockquote > :first-child {
margin-top: 0; }
blockquote > :last-child {
margin-bottom: 0; }
code, pre, #ws, #message {
word-break: normal;
word-wrap: normal;
}
hr {
display: inherit;
}
.bloop_markdown :first-child {
-webkit-margin-before: 0;
}
code, pre, #ws, #message {
font-family: Menlo, Consolas, Liberation Mono, Courier, monospace;
}
.send { color:#77bb77; }
.server { color:#7799bb; }
.error { color:#AA0000; }</style>
</head>
<body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;">
<div class="bloop_markdown">
<p>Hi Matt,</p>
<p>Some notes about this issue:</p>
<ol>
<li>
<p>Correction to my previous email: the <code>XYZMonitor()</code> routines were not touched/modified in this feature branch. I misspoke about that. We modified the
<code>XYZView()</code> only, and indeed these are the ones that appear to have some different indentations in the output files.</p>
</li><li>
<p>I’m seeing incorrect tab levels in only two ex12 tests: <code>ex12_tri_p1_adapt_0</code> and
<code>ex12_tri_p1_adapt_1</code>. Turns out <code>SNESReasonView()</code> did not have the earlier tab level changes correctly propagated into it. I’ve attached a
<code>git format-patch</code> that fixes this (I didn’t wanna push directly to <code>
next</code> without approval). But the remaining tab level changes are consistent with the
<code>XYZView()</code> changes we’ve made. So I would recommend updating the output files after applying the patch (again, I didn’t want to touch the outputs without your approval first).</p>
</li><li>
<p>Your first email on this mentioned FAS solver errors, but all the <code>ex12_fas_%</code> tests are passing for me locally. Can you confirm that these are working for you as well? If not, which test machine should I run this on to reproduce your errors?</p>
</li><li>
<p>I’ve got some number of the ex12 tests crashing on my machine. <code>ex12_restart-b</code> reports HDF5 errors, while
<code>ex12_mg_newton_coarse_0</code>, <code>ex12_mg_newton_coarse_1</code> and <code>
ex12_mg_newton_coarse_2</code> report “inconsistent data” with regards to dof mapping. These just might have to do with my local configuration but I still wanted to put it on your radar just in case.</p>
</li></ol>
<p>Thanks,</p>
<p></p>
</div>
<div class="bloop_original_html"><style>body{font-family:Helvetica,Arial;font-size:13px}</style>
<div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">
<br>
</div>
<div id="bloop_sign_1522802853906365952" class="bloop_sign">—
<div>Alp Dener</div>
</div>
<br>
<p class="airmail_on">On April 3, 2018 at 6:24:42 PM, Matthew Knepley (<a href="mailto:knepley@gmail.com">knepley@gmail.com</a>) wrote:</p>
<blockquote type="cite" class="clean_bq"><span>
<div>
<div></div>
<div>
<title></title>
<div dir="ltr">
<div class="gmail_extra">
<div class="gmail_quote">On Tue, Apr 3, 2018 at 7:16 PM, Dener, Alp <span dir="ltr">
<<a href="mailto:adener@anl.gov" target="_blank">adener@anl.gov</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word;line-break:after-white-space">
<div id="m_6527039185982089683bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px;color:rgba(0,0,0,1.0);margin:0px;line-height:auto">
Sure thing!</div>
<div id="m_6527039185982089683bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px;color:rgba(0,0,0,1.0);margin:0px;line-height:auto">
<br>
</div>
<div id="m_6527039185982089683bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px;color:rgba(0,0,0,1.0);margin:0px;line-height:auto">
Matt, just to give you a bit of detail, the Monitor() calls to ASCII viewers were changed to respect the tab levels for the parent objects instead of manually pushing and popping tabs on the printouts. The code also tries to preserve the existing tab levels
of the viewer though — that is, we first save the existing tab level, then indent monitor printouts from the root (0 tab) using the object’s tab level, and then set the viewer back to whatever tab level it was at before the monitor is called. So in theory,
it shouldn’t be changing the behavior of the viewer outside of the monitor. It is entirely possible I introduced a bug somewhere and that’s what I’m going to look for right now, but in the meantime please let me know if there’s a use-case or a test I can use
for debugging as well.</div>
</div>
</blockquote>
<div><br>
</div>
<div>The easiest thing to do is run all the SNES ex12 tests in next and look at the indentation errors. I am now pushing fixes for all other errors, so</div>
<div>by the time you run there should be nothing else.</div>
<div><br>
</div>
<div> Thanks,</div>
<div><br>
</div>
<div> Matt</div>
<div> </div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word;line-break:after-white-space">Thanks,<br>
<div id="m_6527039185982089683bloop_sign_1522797124911343104" class="m_6527039185982089683bloop_sign">
—
<div><span class="HOEnZb"><font color="#888888">Alp Dener</font></span></div>
</div>
<div>
<div class="h5"><br>
<p class="m_6527039185982089683airmail_on">On April 3, 2018 at 6:11:12 PM, Smith, Barry F. (<a href="mailto:bsmith@mcs.anl.gov" target="_blank">bsmith@mcs.anl.gov</a>) wrote:</p>
<blockquote type="cite" class="m_6527039185982089683clean_bq">
<div>
<div><span><br>
Alp,<br>
<br>
Can you please take a look at this?<br>
<br>
Thanks<br>
<br>
Barry<br>
<br>
<br>
> On Apr 3, 2018, at 4:46 PM, Matthew Knepley <<a href="mailto:knepley@gmail.com" target="_blank">knepley@gmail.com</a>> wrote:<br>
><br>
> The recent indentation fix (I think) has broken the FAS solver indentation.<br>
><br>
> Matt<br>
><br>
> --<br>
> What most experimenters take for granted before they begin their experiments is infinitely more interesting than any results to which their experiments lead.<br>
> -- Norbert Wiener<br>
><br>
> <a href="https://www.cse.buffalo.edu/~knepley/" target="_blank">https://www.cse.buffalo.edu/~<wbr>knepley/</a><br>
<br>
</span></div>
</div>
</blockquote>
</div>
</div>
</div>
</blockquote>
</div>
<br>
<br clear="all">
<div><br>
</div>
--<br>
<div class="gmail_signature" data-smartmail="gmail_signature">
<div dir="ltr">
<div>
<div dir="ltr">
<div>What most experimenters take for granted before they begin their experiments is infinitely more interesting than any results to which their experiments lead.<br>
-- Norbert Wiener</div>
<div><br>
</div>
<div><a href="http://www.caam.rice.edu/~mk51/" target="_blank">https://www.cse.buffalo.edu/~knepley/</a><br>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</span></blockquote>
</div>
<div class="bloop_markdown">
<p></p>
</div>
</body>
</html>