Hi Igor,<br><br>Thanks for the patch!<br><br>I just applied your patch to Osmosis trunk before receiving this latest email.  I've applied it without making any changes, it all looks good to me.<br><br>I'm not too fussed about the default of warning.  As you say, it preserves backwards compatibility which is generally a good thing.  I'm happy to break compatibility if there's a good reason (eg. unintentional data corruption), but I don't think dropping a bound element fits into that category. <br>
<br>Can you please update the development version of the wiki usage page with your changes?<br><a href="http://wiki.openstreetmap.org/wiki/Osmosis/Detailed_Usage_0.40">http://wiki.openstreetmap.org/wiki/Osmosis/Detailed_Usage_0.40</a><br>
<br>Brett<br><br><div class="gmail_quote">2011/5/9 Igor Podolskiy <span dir="ltr"><<a href="mailto:igor.podolskiy@vwi-stuttgart.de">igor.podolskiy@vwi-stuttgart.de</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Hi Dominik, hi all,<div class="im"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
So, basically, I tested your patch sucessfully for the case:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
2. If both sources have a bound, then a bound corresponding to the<br>
unions of bounds 0 and 1 is emitted to the merged stream.<br>
</blockquote></blockquote></div>
cool, thanks for the response, nice to know it worked out for you.<div class="im"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
In case 3b - when a bound is "removed" -, there are three<br>
</blockquote></blockquote>
possibilities controlled by the new boundRemovedAction keyword argument:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
* ignore: don't do anything and continue<br>
* warn: emit a warning to the log but continue (default)<br>
* fail: stop processing altogether<br>
</blockquote>
<br>
Thinking twice about this, I would perhaps consolidate that to<br>
default failure (with detailed explanations in the log how to<br>
overcome this problem, i.e. how to specify the following keyword<br>
argument) plus  a keyword argument "ignoreBounds" which would allow<br>
continuing (equivalent to your "* ignore" case). This would ensure<br>
that the user looks at the issue and ensures correct functioning or<br>
fixing the input maps. - just my suggestion. Maybe case 1 should be<br>
covered by this behavior, too - not sure.<br>
</blockquote></div>
Well... I'm not so sure about that. The thing is, many people don't really care about the <bound> being preserved - well, I didn't care, for example, till you reported the bug. And if the patch with the default behavior of "fail" makes it into a new version of Osmosis, it will suddenly break a lot of pipelines which have been working fine. So I'd let it at a default "warn" for backward compatibility reasons if for nothing else.<div class="im">
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
In any case, thanks a lot for your great work - the whole thing<br>
works  nicely for me now.<br>
</blockquote></div>
You're welcome :)<br>
<br>
Best regards<br>
Igor<br>
<br>
_______________________________________________<br>
dev mailing list<br>
<a href="mailto:dev@openstreetmap.org" target="_blank">dev@openstreetmap.org</a><br>
<a href="http://lists.openstreetmap.org/listinfo/dev" target="_blank">http://lists.openstreetmap.org/listinfo/dev</a><br>
</blockquote></div><br>