-
Notifications
You must be signed in to change notification settings - Fork 67
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
Meta lattice issue #557
Comments
This comment was marked as resolved.
This comment was marked as resolved.
thofma
pushed a commit
that referenced
this issue
Mar 2, 2022
This comment was marked as resolved.
This comment was marked as resolved.
We have this method it is called |
Lll_gram is harder to find and reduces a gram matrix. I'd suggest to future
proof by adding reduce without requiring a specific algorithm
…On Thu, 28 Mar 2024, 10:53 Simon Brandhorst, ***@***.***> wrote:
I guess some tasks in the previous list are done ( (6) coverage for
hermitian lattices, (12) constructor of lattice with given generators, (15)
isdefinite function for hermitian spaces, (16) check if a pseudo matrix
is of full rank). I add some new tasks that I might do soon:
* [ ] Modify `mass(::ZLat)` to return the mass of the genus of the entry argument.
I disagree here. There mass of L is 1/|aut(L)| this change should be
reverted.
* [ ] Add a method `lll_gram(::ZLat)` for arbitrary `ZLat` which calls the appropriate methods depending on the definiteness of the entry argument.
We have this method it is called lll(::ZLat). Maybe it is hard to
discover?
—
Reply to this email directly, view it on GitHub
<#557 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA36CV663GQBRYDNOM7Q7GDY2PK7DAVCNFSM5MPZBRR2U5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TEMBSGQ4DAMRUGE3A>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Agreed. There is |
On Thu, Mar 28, 2024 at 04:49:13AM -0700, Simon Brandhorst wrote:
Agreed. There is `reduce` and `reduction` `saturate` and `saturation` is there any consensus on the horizon?
I think reduce (as in map_reduce) is the julia way, Nouns are probabli
magma induced...
… --
Reply to this email directly or view it on GitHub:
#557 (comment)
You are receiving this because you commented.
Message ID: ***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
gram =
). If the basis is not specified, then the lattice is free and the provided matrix is the Gram matrix of the ambient space. We retire thegram_ambient_space
keyword. (@thofma)LocMultGrpModSquMap
out of Misc to its "proper" place (@thofma).AbsLat
using restriction of scalarsAbsLat
using the more efficient pseudo HNF algorithmslattice(V::AbsSpace, gens::Vector)
prime_power_form
seems to be buggy or some docu/input checks are missingZlattice(;gram)
and remove the@req issymmetric(gram)
since this is checked already in the creation of the quadratic spacediagonal_with_transform(::AbsSpace)
sincediagonal
does not return the transformation.isdefinite
needs to check whether E is totally imaginary in the hermitian case (@thofma)B
has full rank and throw an error if not.direct_sum
"alias" fororthogonal_sum
.orthogonal_sum
toorthgonal_sum_with_maps
and allowVararg{}
as well asVector
.mass(::ZLat)
to return the mass of the genus of the entry argument.lll_gram(::ZLat)
for arbitraryZLat
which calls the appropriate methods depending on the definiteness of the entry argument.lll_gram_indefinite_*
to their transpose to comply with Hecke conventions.lll_gram_indefinite_*
methods.QuadForm
and add doctests.is_isometric_with_isometry
is (randomly) very slow for indefinite quadratic spaces. See the tests that are disabled. If one runs them often enough, it will get stuck. Possible related to large output fromis_isotropic_with_vector
.M_1
be the sublattice spanned by the shortest vectors ofL_1=L
andL_{i+1}
be the orthogonal complement ofM_i
inL_i
andM_{i+1}
the sublattice ofL_{i+1}
spanned by its shortest vectors. Then the output should be the union of the shortest vectors of theM_i
. This should serve as the default input to_set_nice_monomorphism!(G::MatrixGroup, short_vectors; closed=false)
in OSCARThe text was updated successfully, but these errors were encountered: