[Dune] grid.geometry is a bit restrictive

[Aleksejs Fomins]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Dear Christian,

It says that I am not a registered user on a flyspray.

I was trying to log in here
https://www.dune-project.org/flyspray/

with my dune-wiki username and password...

Am I doing this right?
Aleksejs



On 27/02/15 15:44, Christian Engwer wrote:
> On Fri, Feb 27, 2015 at 03:41:52PM +0100, Aleksejs Fomins wrote:
>> Dear Christian,
>>
>> So do I understand correctly that you suggest to use getRealImplementation on user side to get the extra methods of the underlying grid.geometry() implementation? I think this would work too
>>
>> Anyway, I think it would be a good idea to open a feature request. I will explain what methods I provide and why I believe they are useful to the user, and then you people can decide on whether you want to add some of them to the main interface or keep them to the auxiliary interface.
>>
>> Could you please briefly explain how to open a feature request?
> 
> Add a flyspray issue and mark it as "feature request".
> 
> Christian
> 
>>
>> Thanks,
>> Aleksejs
>>
>>
>> On 27/02/15 15:34, Christian Engwer wrote:
>>> Dear Aleksejs,
>>>
>>> On Fri, Feb 27, 2015 at 12:06:16PM +0100, Aleksejs Fomins wrote:
>>>> Dear Dune,
>>>>
>>>> I just realised that CurvilinearGeometry has quite a few new user methods compared to linear geometries, for example, access to interpolation vertices, interpolation polynomials and functions thereof. The wrapper interface of grid.geometry class does not allow for adding these additional methods, so they will need to be accessed some other way.
>>>>
>>>> I have decided to provide a grid method grid.basegeometry(entity),
>>>> which would directly return the Dune::CurvilinearGeometry class, and
>>>> not its grid wrapper. I realise that this goes against the paradigm of
>>>> "protecting the users from themselves", but I do not see a good
>>>> alternative at the moment.
>>>
>>> There is the method grid.getRealImplementation(...) which returns the
>>> implementation hidden in the facades class. My advice is not to add your
>>> new method on the grid, as it is just a more specific version of the
>>> getRealImplementation method.
>>>
>>> Besides this... why do you need these additional methods? If you
>>> consider them necessary, please open a feature-request and let
>>> everybody discuss this topic. I havent worked a lot with higher order
>>> transformations, but I never had the need to access these additional
>>> information from user code.
>>>
>>> Christian
>>>
>>>>
>>>> Regards,
>>>> Aleksejs
>>>>
>>>> _______________________________________________
>>>> Dune mailing list
>>>> Dune at dune-project.org
>>>> http://lists.dune-project.org/mailman/listinfo/dune
>>>>
>>>
>>
> 
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.12 (GNU/Linux)

iQIcBAEBAgAGBQJU8IT8AAoJEDkNM7UOEwMZfZYQALrjdJMUALjdOeSr7ZLhK5G2
XUcvTQKrJLViNWm6K8VHvZbV3+2sxph6lRSpQUmAHFcA1cCfAUgvH8C1gzPSAdGs
sOyEGe6rRR+o2TCvBHIZVg0amcwKNjtQLsckxYAZ6i2ITUC3kVJyn4vc4aGvrsLY
1HGyIfNiwtaCqV0NptftNaBTvZqT2kzBDEIkSzR7nfY78C48TCGDFfJsdjuHLQ+y
S0Q4Bs+sCBW/3/5q3/CFt3bNAgNOcYjqR4fKrAJf4pb7UcTRzFeEjay7/LzM+37W
Hm6v74J4L4cU4eW0ItBLBpLJfHNDzDm94nqpA4Ior5NyJU6nU3qpG2vS5NRf5UU4
d26cKYY3U4AjeTW6sIP10mTMfuSLCLVs7Yq6zIVCbFvFDi845WGjL17fj8wheG93
C18iX92QrLNOn92FOizeQR4Vvr9oLFYdtFmThVNN8ffJXqWeI5lymEy7KQfqBJcj
el0FOtgbWIH07dmQqHDFaKNZKmbXAqsamwoJyjom+pRH1/bPSi5P/EzZpWxaPUsX
5+oXKHYMntqGb6t3tPxwjq+iq9JxyW5jklm9vHoipW8cx9bePV9ETXwP8w0bQbfc
OrA9ejDMXUVwFAGiRbLFoo+69JLvKXDiBCCfhqvMWZbDxSQYU+1El1f9a2GTjKcA
F9YHbIg6AvvhcivcLSbj
=PzBP
-----END PGP SIGNATURE-----