[PATCH] Re: New Bug Reports/Feature Requests
Dirk Hohndel
dirk at hohndel.org
Wed Mar 2 09:25:07 PST 2016
On Wed, Mar 02, 2016 at 09:12:43AM -0800, Linus Torvalds wrote:
> On Wed, Mar 2, 2016 at 5:15 AM, Lubomir I. Ivanov <neolit123 at gmail.com> wrote:
> >
> > hmm, that looks quite off to me considering the scale (the level of
> > precision) of the ordinate.
>
> Hmm. Yes. I don't know how much we really care in the end, but
> especially now that we actually show it in the cylinder size, it's not
> good.
>
> In fact, with this patch now applied, the AL80, that used to look like
>
> 80(77)cuft
>
> in the size field, now looks like
>
> 80(80)cuft
>
> which is pointless and wrong.
>
> That "let's get the right value for the most common cylinder in the
> known universe" was a big part of trying to be more exact here, and
> now we don't do that any more.
>
> I like that Robert's function can handle other gases than air, but I
> do think it's too far off from reality to be good.
>
> Or we just need to stop showing the corrected cylinder size, because
> right now it's useless and very wrong.
So since Robert's formula /should/ be the right way to calculate the
compensation factors, let's figure out what about it is broken and use
"matches the wikipedia data" as a measuring stick for that.
I don't want to walk away from having the "correct" sizes - that's the
point of why we were doing all this. It's good that we show our calculated
values as this makes it obvious that the current formula isn't quite
correct.
/D
More information about the subsurface
mailing list