Opened 5 years ago

Last modified 5 months ago

#21343 needs_review enhancement

deal with bad chosen representations of k-regular sequences

Reported by: dkrenn Owned by:
Priority: major Milestone: sage-7.4
Component: combinatorics Keywords:
Cc: Merged in:
Authors: Daniel Krenn Reviewers:
Report Upstream: N/A Work issues:
Branch: u/dkrenn/k-regular-warning (Commits, GitHub, GitLab) Commit: 9b5902236508b1704b481d8fbd5c4fadfe66389d
Dependencies: #21204 Stopgaps:

Status badges

Description

Deal with the troubles coming from bad chosen representations of k-regular sequences, where mu[0]*right != right. (This makes troubles as leading zeros of digit expansions should not influence the result.)

Change History (17)

comment:1 Changed 5 years ago by dkrenn

  • Dependencies set to #21204

comment:2 Changed 5 years ago by dkrenn

  • Branch set to u/dkrenn/k-regular-warning

comment:3 Changed 5 years ago by dkrenn

  • Branch u/dkrenn/k-regular-warning deleted
  • Status changed from new to needs_review
  • Summary changed from bad chosen representations of k-regular sequences to deal with bad chosen representations of k-regular sequences

comment:4 Changed 5 years ago by dkrenn

  • Branch set to u/dkrenn/k-regular-warning

comment:5 Changed 5 years ago by git

  • Commit set to 5b7b85a7fd7d059c1c1e85c1689d5fe6d0fc9eed

Branch pushed to git repo; I updated commit sha1. Last 10 new commits:

352751eMerge tag '7.4.beta2' into t/21295/sequences/recognizable
914640fMerge branch 't/21295/sequences/recognizable' into t/21203/sequences/k-regular
c0ade99Merge branch 't/21203/sequences/k-regular' into t/21318/sequences/rec-basic-arith
35807dfMerge branch 't/21318/sequences/rec-basic-arith' into t/21319/sequences/rec-hash
a2fe6f3Merge branch 't/21318/sequences/rec-basic-arith' into t/21325/sequences/k-regular-subseq
e4cec17minor correction in doc
3a4e849Merge branch 't/21318/sequences/rec-basic-arith' into t/21325/sequences/k-regular-subseq
c8a86dbMerge branch 't/21325/sequences/k-regular-subseq' into t/21319/sequences/rec-hash
0dae02bMerge branch 't/21319/sequences/rec-hash' into t/21204/sequences/k-regular-guess
5b7b85aMerge branch 't/21204/sequences/k-regular-guess' into t/21343/k-regular-warning

comment:6 Changed 5 years ago by git

  • Commit changed from 5b7b85a7fd7d059c1c1e85c1689d5fe6d0fc9eed to a716121e47ed35e4b140235b3ed2fc45113869a3

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

1d64e84some_elements: pass keyword arguments to element constructor
e02018boverride some_elements in kRegularSequenceSpace and activate healing
18c5268correct typo in docstring
a716121Merge branch 't/21319/sequences/rec-hash' into t/21343/k-regular-warning

comment:7 follow-up: Changed 5 years ago by cheuberg

  • Status changed from needs_review to needs_work

failing doctests (cf. patchbot), python3 plugin

comment:8 Changed 5 years ago by git

  • Commit changed from a716121e47ed35e4b140235b3ed2fc45113869a3 to e25b8d227be3e0821b4b659d185a03cf65cdecdb

Branch pushed to git repo; I updated commit sha1. Last 10 new commits:

4da9796Merge branch 't/21203/sequences/k-regular' into t/21318/sequences/rec-basic-arith
13b536ePython3: absolut import
5919c4dMerge branch 't/21203/sequences/k-regular' into t/21318/sequences/rec-basic-arith
9086b64Merge branch 't/21318/sequences/rec-basic-arith' into t/21325/sequences/k-regular-subseq
fc6c3f5Python3: xrange
714276cMerge branch 't/21325/sequences/k-regular-subseq' into t/21319/sequences/rec-hash
457aefdPython3: fix iteritems
fcb53a3fix doctest output of TestSuite (new tests were added)
bde8e67Merge branch 't/21319/sequences/rec-hash' into t/21204/sequences/k-regular-guess
e25b8d2Merge branch 't/21204/sequences/k-regular-guess' into t/21343/k-regular-warning

comment:9 in reply to: ↑ 7 Changed 5 years ago by dkrenn

Replying to cheuberg:

failing doctests (cf. patchbot), python3 plugin

Fixed.

comment:10 Changed 5 years ago by dkrenn

  • Status changed from needs_work to needs_review

comment:11 Changed 4 years ago by git

  • Commit changed from e25b8d227be3e0821b4b659d185a03cf65cdecdb to dde17007540561f19baecefe6d973912a7439a1c

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

c2bf1b8Merge tag '8.1' into u/dkrenn/sequences/recognizable
fef79b2Merge branch 'u/dkrenn/sequences/recognizable' into u/dkrenn/sequences/k-regular
8eda76eMerge branch 'u/dkrenn/sequences/k-regular' into u/dkrenn/sequences/rec-basic-arith
0c83557Merge branch 'u/dkrenn/sequences/rec-basic-arith' into u/dkrenn/sequences/k-regular-subseq
e30a7fbMerge branch 'u/dkrenn/sequences/k-regular-subseq' into u/dkrenn/sequences/rec-hash
622c3f6Merge branch 'u/dkrenn/sequences/rec-hash' into u/dkrenn/sequences/k-regular-guess
dde1700Merge branch 'u/dkrenn/sequences/k-regular-guess' into u/dkrenn/k-regular-warning

comment:12 Changed 3 years ago by git

  • Commit changed from dde17007540561f19baecefe6d973912a7439a1c to 3b0f9d0aaddbabca73a19fe027b0b7cf0733cc8a

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

5dbdcbbMerge tag '8.7' into u/dkrenn/sequences/recognizable
d17def8Merge branch 'u/dkrenn/sequences/recognizable' into u/dkrenn/sequences/k-regular
2b998baMerge branch 'u/dkrenn/sequences/k-regular' into u/dkrenn/sequences/rec-basic-arith
1f8c2a1Merge branch 'u/dkrenn/sequences/rec-basic-arith' into u/dkrenn/sequences/k-regular-subseq
a87520eMerge branch 'u/dkrenn/sequences/k-regular-subseq' into u/dkrenn/sequences/rec-hash
4e7f615Merge branch 'u/dkrenn/sequences/rec-hash' into u/dkrenn/sequences/k-regular-guess
3b0f9d0Merge branch 'u/dkrenn/sequences/k-regular-guess' into u/dkrenn/k-regular-warning

comment:13 Changed 3 years ago by git

  • Commit changed from 3b0f9d0aaddbabca73a19fe027b0b7cf0733cc8a to 848565f03490ec7e5677db1b015296a53afef451

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

21ed597Trac #21343: fix doctest some_elements
848565fTrac #21343: rename to "degenerate" and co

comment:14 Changed 7 months ago by git

  • Commit changed from 848565f03490ec7e5677db1b015296a53afef451 to f776785803038ba37e9f513220d22a03b8a30fe0

Branch pushed to git repo; I updated commit sha1. Last 10 new commits:

a117f51Merge branch 't/21203/sequences/k-regular' into t/21318/sequences/rec-basic-arith
0f8fb2eTrac #21318: fixup due to recent changes in dependencies
4e967fdMerge branch 't/21318/sequences/rec-basic-arith' into t/21325/sequences/k-regular-subseq
1ead378Merge branch 't/21325/sequences/k-regular-subseq' into t/21319/sequences/rec-hash
15b0405Trac #21319: fixup due to changes in dependencies
e726278Merge branch 't/21319/sequences/rec-hash' into t/21204/sequences/k-regular-guess
dcc7cd1Trac #21204: cherry-pick to avoid merge conflict
c0519f0Trac #21204: fixup code and tests
8488b55Merge branch 't/21204/sequences/k-regular-guess' into t/21343/k-regular-warning
f776785Trac #21343: adapt to removed transpose-property in dependency

comment:15 Changed 7 months ago by dkrenn

tranpose was removed due to the discussion started at ticket:21295#comment:32 issue 6. This lead to a non-minor change here, as dealing with regenerated sequences and transposing sequences needs to go hand-in-hand during construction. I now remember, that this was the original motivation for having transpose in the constructor. However, I have now created a solution here adapting to this situation.

comment:16 Changed 5 months ago by git

  • Commit changed from f776785803038ba37e9f513220d22a03b8a30fe0 to 9b5902236508b1704b481d8fbd5c4fadfe66389d

Branch pushed to git repo; I updated commit sha1. Last 10 new commits:

d4e1d1fTrac #21318: rmul/lmul preserve identity for multiplying by 1
e66ff5eTrac #21325: fixup test .subsequence being identity
c70f8dbTrac #21318: rmul/lmul preserve identity for multiplying by 1
030162aMerge branch 't/21318/sequences/rec-basic-arith' into t/21325/sequences/k-regular-subseq
cff550eMerge branch 't/21325/sequences/k-regular-subseq' into t/21319/sequences/rec-hash
93c6687Trac #21319: fix punctuation
5ce5276Merge branch 't/21319/sequences/rec-hash' into t/21204/sequences/k-regular-guess
00047ceTrac #21204: fix punctuation
fb5fb62Merge branch 't/21204/sequences/k-regular-guess' into t/21343/k-regular-warning
9b59022Trac #21343: fix punctuation

comment:17 Changed 5 months ago by dkrenn

Merged dependencies; ready for review.

Note: See TracTickets for help on using tickets.