-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove "synthetic/implementation" class functions #3
Comments
I agree, this would indeed be nice. But how would you go on about this? How On 15 November 2016 at 18:43, Irene Lobo Valbuena notifications@github.com
|
To TODO came from me (also to me;-) and I have not studied the code enough
There may be good reason to tie the class to Double, but if feels strange /P On 16 November 2016 at 20:20, simonpf notifications@github.com wrote:
|
I am definitely curious about this because I couldn't find a solution that But in the end every finite dimensional space is isomorphic to R^n so, if On 16 November 2016 at 21:09, Patrik Jansson notifications@github.com
|
That would be the case if the vector space were over ℝ itself, or another field isomorphic to it. There are finite fields (finite in cardinality, not basis cardinality), for instance. Maybe you were referring to Euclidean spaces (?). In any case, 'toDouble'/'fromDouble' wouldn't "belong" in the definition of the 'Field' class since for finite/Galois fields these embeddings (respecting the structure) would not exist. 'Double' is a concrete representation of what we understand ℝ to be but is not exact, in addition. |
Yes, you are right. I didn't consider fields different from On 17 November 2016 at 14:28, Irene Lobo Valbuena notifications@github.com
|
Such as those converting to and from 'Double' in 'Field'. So far, they are seen in:
Initially, the existence of these class functions stemmed mostly from the numerics behind integration of polynomials and other computations of the like. Even though most practical uses of the library might involve ℝn, it would be preferable to keep the classes neat and general.
The text was updated successfully, but these errors were encountered: