Opened 7 years ago

Last modified 3 years ago

#19426 needs_work enhancement

AsymptoticRing: convert Orders of symbolic ring

Reported by: Daniel Krenn Owned by:
Priority: major Milestone: sage-8.2
Component: asymptotic expansions Keywords:
Cc: Benjamin Hackl Merged in:
Authors: Eran Assaf Reviewers: Benjamin Hackl
Report Upstream: N/A Work issues: rebase
Branch: u/assaferan/asymptoticring__convert_orders_of_symbolic_ring (Commits, GitHub, GitLab) Commit: 90152f3b60b3d756fe8088ecc0f9277dc195d25d
Dependencies: #19083, #19425 Stopgaps:

Status badges

Description (last modified by Clemens Heuberger)

Implement conversion from Order(...) to asymptotic rings.

(There seems to be a bug in SR (#19425) which has to be solved first.)

Change History (10)

comment:1 Changed 7 years ago by Clemens Heuberger

Description: modified (diff)

comment:2 Changed 7 years ago by Clemens Heuberger

Cc: Benjamin Hackl added
Dependencies: #19083, #19424#19083, #19425

comment:3 Changed 7 years ago by Daniel Krenn

Keywords: beginner added

comment:4 Changed 5 years ago by Eran Assaf

Branch: u/assaferan/asymptoticring__convert_orders_of_symbolic_ring

comment:5 Changed 5 years ago by Eran Assaf

Authors: Eran Assaf
Commit: 45ec3d759f5146c65f0be1bd001ca22a2385b6c0
Status: newneeds_review

New commits:

bd4a461Changed brial installation to support cygwin 64-bit with the new limits.h
60c5606Fixed Ticket #19426 - Implemented conversion from Order(...) to asymptotic rings.
cd2f1a4Added conversion from Symbolic Order to Asymptotic Ring.
45ec3d7Added an appropriate doctest

comment:6 Changed 5 years ago by Benjamin Hackl

Milestone: sage-6.10sage-8.2
Reviewers: Benjamin Hackl
Status: needs_reviewneeds_work

Hi! Your basic approach looks good, however, you should merge the latest beta (8.2.beta8) into your branch and fix the resulting merge conflict with your adaption of the brial configuration. (This issue has been fixed in #24860, which is merged in beta8.)

As a suggestion: you should keep fixes for unrelated problems (like in this case the brial problem) separate from the code for this ticket (basically, separate fixes deserve separate tickets).

I'll review this as soon as the conflict is resolved.

comment:7 Changed 5 years ago by git

Commit: 45ec3d759f5146c65f0be1bd001ca22a2385b6c090152f3b60b3d756fe8088ecc0f9277dc195d25d

Branch pushed to git repo; I updated commit sha1. New commits:

90152f3merged latest beta code with mine

comment:8 Changed 5 years ago by Eran Assaf

Status: needs_workneeds_review

Hi! I've fixed the conflict and committed the changes.

In principal, I agree with your suggestion. The problem was I could not build sage on my PC without fixing this problem, and it took a while until I could get everything to work (trac user, git, etc.).

Anyway, hopefully it can be reviewed now. Thanks, Eran

comment:9 Changed 3 years ago by gh-RelationaleDB

Status: needs_reviewneeds_work
Work issues: rebase

merge conflict

comment:10 Changed 3 years ago by Dave Morris

Keywords: beginner removed

Removing the "beginner" tag from old tickets. Some could be returned to beginner-friendly status by adding a comment about what needs to be done. Some others might be easy for an experienced developer to finish.

Note: See TracTickets for help on using tickets.