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

documentation of QQbar #3416

Open
micjoswig opened this issue Feb 23, 2024 · 8 comments
Open

documentation of QQbar #3416

micjoswig opened this issue Feb 23, 2024 · 8 comments
Assignees
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Milestone

Comments

@micjoswig
Copy link
Member

micjoswig commented Feb 23, 2024

The Calcium implementation of QQbar becomes increasingly important for OSCAR. So it needs to be properly documented.

Searching for "QQbar" kind of works but only leads to the Nemo docs.

As I think, QQbar should also be mentioned here.

QQbar is used, e.g., for polyhedral computations in real number fields (with fixed embeddings). This works nicely, but the overall behavior could be surprising for some users. Crucially, QQbar implements a comparison function, which is only defined for real numbers. The implementation is correct as it dies with an error if the user tries to compare complex numbers with nonzero imaginary parts. Yet, this "feature" allows QQbar as a type parameter, e.g., for the Polyhedron class. And this is unexpected, because the algebraic closure of the rationals is not an ordered field.

Again, there is nothing wrong with the implementation. But the docs need to explain this carefully.

@micjoswig micjoswig added enhancement New feature or request backport 1.0.x Should be backported to the release 1.0 branch labels Feb 23, 2024
@micjoswig
Copy link
Member Author

@fieker FYI

@lgoettgens lgoettgens added the documentation Improvements or additions to documentation label Feb 23, 2024
@lgoettgens
Copy link
Member

As a first step, one can add https://github.com/Nemocas/Nemo.jl/blob/master/docs/src/algebraic.md to the navigation bar under "Fields" (similarly as e.g. Padics). Everything else should be adapted in this Nemo doc page.

@micjoswig
Copy link
Member Author

micjoswig commented Feb 23, 2024

As a first step, one can add https://github.com/Nemocas/Nemo.jl/blob/master/docs/src/algebraic.md to the navigation bar under "Fields" (similarly as e.g. Padics).

That is certainly a good idea.

However, it is not quite enough. The reason is that Nemo has a focus on number theory, which makes the focus on reals less important. The whole thing is relevant for polyhedral geometry; and there we need an additional paragraph in the docs somewhere.

But should the polyhedral docs contain a page/paragraph dedicated to QQbar, or should this be written elsewhere (in the OSCAR docs, outside Nemo)?

@tbrysiewicz @lkastner @benlorenz

@thofma
Copy link
Collaborator

thofma commented Feb 23, 2024

Can you clarify if this is about QQbar or embedded number fields (e.g. things like $\mathbf{Q}(\sqrt{2})$?

@micjoswig
Copy link
Member Author

Can you clarify if this is about QQbar or embedded number fields (e.g. things like Q(2)?

If in doubt: both. But QQbar is more important.

@lgoettgens lgoettgens added this to the 1.0 milestone Feb 23, 2024
@lgoettgens lgoettgens removed the backport 1.0.x Should be backported to the release 1.0 branch label Feb 23, 2024
@lgoettgens
Copy link
Member

I created #3423 to tackle #3416 (comment).
And I removed the backport label here since an issue cannot really be backported (instead, used a milestone).

@fingolfin
Copy link
Member

Actually a global const QQBar which is equal to algebraic_closure(QQ) is documented on https://docs.oscar-system.org/dev/Nemo/algebraic/ but it only exists in Nemo, not in Oscar.

We just talked about this, @fieker proposes to remove it from the Nemo docs and perhaps we should just remove this global on Nemo, and tell people to use algebraic_closure(QQ) instead.

@fingolfin
Copy link
Member

In Nemocas/Nemo.jl#1715 @lgoettgens started to prepare Nemo for the removal of QQBar as a global object. We should complete that PR and then do the removal of QQBar...

@fingolfin fingolfin removed the triage label Jun 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants