Skip to content
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

Display partial information on classical modular form pages #1385

Closed
AndrewVSutherland opened this issue May 17, 2016 · 6 comments
Closed

Display partial information on classical modular form pages #1385

AndrewVSutherland opened this issue May 17, 2016 · 6 comments
Assignees
Labels
CMF Classical modular forms critical Serious problem that directly impacts www.lmfdb.org

Comments

@AndrewVSutherland
Copy link
Member

AndrewVSutherland commented May 17, 2016

Edited per the discussion below

In cases where the coefficients of the Q-expansion are too large to reasonably display, as on

http://beta.lmfdb.org/ModularForm/GL2/Q/holomorphic/23/38/1/ http://beta.lmfdb.org/ModularForm/GL2/Q/holomorphic/23/38/1/a

for example, don't display the q-expansion but just give information about the coefficient field (a link to the field if it is in the LMFDB, and if not, at least its degree and perhaps a defining polynomial for it).

We can instead provide a link for downloading whatever coefficient data we have available for those who want it.

For cases where we only know the dimensions of the Hecke orbits that make up the decomposition of the space and perhaps some other partial information (e.g. the coefficient field or its degree), we could display this and note that the coefficients are not yet available. This would be friendlier then a long list of "Hecke orbit space not in database" entries (we currently hide these space to avoid displaying them).

I am marking this as critical because, as it stands, many of the current pages (not just the two examples above) cannot be reasonably rendered and take very long to load. This is something we could easily fix even before filling out and verifying all the modular form data we would like to have available (see the discussion at #1248).

This more flexible approach would also allows us to have modular forms pages in cases where we have only numerical data but still want to include the L-function in the LMFDB (as noted in #1237 (comment)).

@AndrewVSutherland AndrewVSutherland added CMF Classical modular forms critical Serious problem that directly impacts www.lmfdb.org labels May 17, 2016
@sehlen
Copy link
Contributor

sehlen commented May 17, 2016

It does not make sense to restrict to a_2 as this might vanish. Moreover,
it might not be of the form alpha where Q(alpha) is the coefficient field
in the data that we computed. We have to distinguish several cases and I
would rather vote for hiding the q-expansion by default in those cases.

On Tue, May 17, 2016, 08:56 Andrew Sutherland notifications@github.com
wrote:

In cases where the coefficients of the Q-expansion are too large to
reasonably display, as on

http://beta.lmfdb.org/ModularForm/GL2/Q/holomorphic/23/38/1/
http://beta.lmfdb.org/ModularForm/GL2/Q/holomorphic/23/38/1/a

for example, only display the first two coefficients (e.g. q+_a_2 q^2+...)
and give information about the coefficient field (a link to the field if it
is in the LMFDB, and if not, at least its degree and perhaps a defining
polynomial for it). I note that the coefficient field is not always Q(
a), but when it is not it can usually be compactly described as Q(
_a_2,b) for some reasonably simple b). We could then provide a link
for downloading whatever coefficient data we have available for those who
want it.

For cases where we only know the dimensions of the Hecke orbits that make
up the decomposition of the space and perhaps some other partial
information (e.g. the coefficient field or its degree), we could display
this and note that the coefficients are not yet available. This would be
friendlier then a long list of "Hecke orbit space not in database" entries
(we currently hide these space to avoid displaying them).

I am marking this as critical because, as it stands, many of the current
pages (not just the two examples above) cannot be reasonably rendered and
take very long to load. This is something we could easily fix even before
filling out and verifying all the modular form data we would like to have
available (see the discussion at #1248
#1248).


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#1385

@sehlen
Copy link
Contributor

sehlen commented May 17, 2016

But there is another issue with this, namely that pages take long to load
(before they get the chance to render) because the record is large. Please
assign me and I can offer a fix for both issues today or tomorrow for
people to test.

On Tue, May 17, 2016, 09:03 Stephan Ehlen stephan@die-ehlens.de wrote:

It does not make sense to restrict to a_2 as this might vanish. Moreover,
it might not be of the form alpha where Q(alpha) is the coefficient field
in the data that we computed. We have to distinguish several cases and I
would rather vote for hiding the q-expansion by default in those cases.

On Tue, May 17, 2016, 08:56 Andrew Sutherland notifications@github.com
wrote:

In cases where the coefficients of the Q-expansion are too large to
reasonably display, as on

http://beta.lmfdb.org/ModularForm/GL2/Q/holomorphic/23/38/1/
http://beta.lmfdb.org/ModularForm/GL2/Q/holomorphic/23/38/1/a

for example, only display the first two coefficients (e.g. q+_a_2 q^2+...)
and give information about the coefficient field (a link to the field if it
is in the LMFDB, and if not, at least its degree and perhaps a defining
polynomial for it). I note that the coefficient field is not always Q(
a), but when it is not it can usually be compactly described as Q(
_a_2,b) for some reasonably simple b). We could then provide a link
for downloading whatever coefficient data we have available for those who
want it.

For cases where we only know the dimensions of the Hecke orbits that make
up the decomposition of the space and perhaps some other partial
information (e.g. the coefficient field or its degree), we could display
this and note that the coefficients are not yet available. This would be
friendlier then a long list of "Hecke orbit space not in database" entries
(we currently hide these space to avoid displaying them).

I am marking this as critical because, as it stands, many of the current
pages (not just the two examples above) cannot be reasonably rendered and
take very long to load. This is something we could easily fix even before
filling out and verifying all the modular form data we would like to have
available (see the discussion at #1248
#1248).


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#1385

@AndrewVSutherland
Copy link
Member Author

AndrewVSutherland commented May 17, 2016

@sehlen (#1385 (comment)) Good point, and I think you are right that we don't want to complicate things. So for the moment we could just have two cases

  • Display everything as we do now (coefficients and information about the coefficient field)
  • Display no coefficients (with a download button if they are available) and give information about the coefficient field (its dimension, and a link to its LMFDB page if there is one or a defining polynomial if not)

Does that sound reasonable? (If so I can amend the issue description)

@sehlen
Copy link
Contributor

sehlen commented May 17, 2016

@AndrewVSutherland Absolutely, that's exactly what I wanted to (will) do.

@sehlen
Copy link
Contributor

sehlen commented May 18, 2016

See my proposal at #1386 (comment) and my branch https://github.com/sehlen/lmfdb/tree/cleanup_bugfix_efficiency for testing this.

sehlen added a commit to sehlen/lmfdb that referenced this issue May 18, 2016
work in all cases. Still have to tune the default value for
default_max_height, wich controls when to hide
a Fourier expansion. We now also display the trace and the approximate norm
of the "first" nonvanishing Fourier coefficient.

This fixes LMFDB#1385
although there is certainly room for improvement.
@sehlen
Copy link
Contributor

sehlen commented May 19, 2016

Updated, see my pull request #1409

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CMF Classical modular forms critical Serious problem that directly impacts www.lmfdb.org
Projects
None yet
Development

No branches or pull requests

2 participants