I think a wiki page describing this should be fine. Just wanted to make sure I use the new functionality properly.

Cheers,
Josh

On May 12, 2008, at 5:59 PM, Ralph Castain wrote:




On 5/12/08 3:49 PM, "Jeff Squyres" <jsquy...@cisco.com> wrote:

Sorry it took so long for a reply; Ralph and I were working on this
code much of the day in an attempt to have it all complete / tidied up
for the teleconf tomorrow.


On May 12, 2008, at 10:04 AM, Josh Hursey wrote:

Er, no. I thought the group had agreed to the main idea last Tuesday
(framework for filtering output).  We were racing against the time-
to-
branch clock and didn't take the time for an RFC after we agreed on
the design.  Do we need to?

I don't think so. But I'd just kinda like a more formal description of
what this fix is and it's implications on how the developers are
expected to use it going forward since this is altering the coding
standards.

Fair enough, will do.

Since this one was kinda weird, do you want an after-the-fact RFC, or
a page on the wiki? I'm partial to the latter; it'll be more durable.

The side effect of eliminating duplicate error messages is new / was not discussed last Tuesday -- I can put out an RFC for that if you'd
like, but the benefit is so obvious that I didn't think it would be
controversial.

Don't get me wrong, I'm not arguing the benefit just that I'd like to
know what is expected of me as a developer after this change.

That's perfectly reasonable.  In short: s/opal_show_help/
orte_show_help/ in the ORTE and OMPI layers, and you're done (which we already did throughout the code base). Use orte_show_help in the ORTE
and OMPI layers in the future.  I think this information should go on
the wiki.

Just to complete that, you also should:

s/opal_output/orte_output
s/OPAL_OUTPUT/ORTE_OUTPUT
s/OPAL_OUTPUT_VERBOSE/ORTE_OUTPUT_VERBOSE

throughout ORTE and OMPI layers in the future.

This has also been done in the current code base.


Finally, per a conversation that I had with Terry earlier today, I
added a new MCA parameter that will turn off the show_help message
aggregation.  It defaults to aggregation enabled, but you can disable
it with:

     ... --mca orte_base_help_aggregation 0 ...

This will show *all* show_help messages, regardless of duplication.
Terry was worried that aggregating the same (filename, tuple) messages
may actually mask different errors because we allow %s expansion in
the message.

Re-examining George's mail in this thread, I think he may have had
similar concerns, but I didn't grok that at the time.


_______________________________________________
devel mailing list
de...@open-mpi.org
http://www.open-mpi.org/mailman/listinfo.cgi/devel

Reply via email to