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

Fix bug related to issue #103 #108

Merged
merged 2 commits into from
Feb 18, 2019
Merged

Fix bug related to issue #103 #108

merged 2 commits into from
Feb 18, 2019

Conversation

swewers
Copy link
Contributor

@swewers swewers commented Feb 18, 2019

With Sage 8.6, residue fields of valuations on function fields are now
function fields themselves, if this makes sense. However, the constant
base field may not be a true finite field (this is trac:26103). This
resulted in an error message when applying the method
make_function_field.

Now make_function_field creates a new function field with the right
kind of constant base field if the input is already a function field.

With Sage 8.6, residue fields of valuations on function fields are now 
function fields themselve, if this makes sense. However, the constant 
base field may not be a *true* finite field (this is trac:26103). This 
resulted in an error message when applying the method 
``make_function_field``.

Now ``make_function_field`` creates a new function field with the right 
kind of constant base field if the input is already a function field.
@swewers
Copy link
Contributor Author

swewers commented Feb 18, 2019

At the moment it is not possible to pass all test for 8.3 and 8.4 and the current version.

@swewers swewers closed this Feb 18, 2019
@swewers swewers reopened this Feb 18, 2019
@swewers
Copy link
Contributor Author

swewers commented Feb 18, 2019

At the moment it is not possible to pass all test for the current version of Sage, and for 8.3 and 8.4. I ignore this issue for the moment, and merge.

@swewers swewers merged commit e1a0d23 into master Feb 18, 2019
@swewers swewers deleted the new_try branch February 18, 2019 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant