Opened 7 years ago

Closed 7 years ago

#16236 closed enhancement (fixed)

Five mutually orthogonal latin squares of order 12

Reported by: ncohen Owned by:
Priority: major Milestone: sage-6.3
Component: combinatorics Keywords: design, mols
Cc: vdelecroix, knsam, brett Merged in:
Authors: Nathann Cohen Reviewers: Vincent Delecroix
Report Upstream: N/A Work issues:
Branch: 2d7da8a (Commits, GitHub, GitLab) Commit: 2d7da8a150862462eaadd3e261ecc81abbe712c9
Dependencies: #16241 Stopgaps:

Status badges

Description (last modified by ncohen)

Wuuuhuuuuuuuu !! Another construction ! Given by Brendan McKay ;-)

Nathann

Change History (24)

comment:1 Changed 7 years ago by ncohen

  • Branch set to u/ncohen/16236
  • Status changed from new to needs_review

comment:2 Changed 7 years ago by git

  • Commit set to 3442d2075e4fd20a16390013c5b4a111c5803312

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

​3442d20trac #16236: Five mutually orthogonal latin squares of order 12

comment:3 Changed 7 years ago by ncohen

  • Description modified (diff)

comment:4 Changed 7 years ago by git

  • Commit changed from 3442d2075e4fd20a16390013c5b4a111c5803312 to 491ce08b3a40cd6b00c34ffdfeb539851136cc94

Branch pushed to git repo; I updated commit sha1. This was a forced push. Last 10 new commits:

​ddda559trac #16277: Merged with updated #16272
​7570b39trac #16277: From availability to existence and broken doctests
​5e8b2aftrac #16277: removes a doctests that appears several times already
​acf8988trac #16277: Merged with updated #16231
​0f3a755trac #16235: A pair of orthogonal latin squares of order 10
​316b379trac #16235: Merged with updated #16277
​e59d6cbtrac #16241: New MOLS shared by Ian Wanless
​e08b0betrac #16241: missing links and tests
​fb26a1etrac #16241: Merged with updated #16235
​491ce08trac #16236: Five mutually orthogonal latin squares of order 12

comment:5 Changed 7 years ago by ncohen

  • Cc vdelecroix knsam brett added

Updated !

comment:6 Changed 7 years ago by vdelecroix

  • Keywords design mols added

adding keywords...

comment:7 Changed 7 years ago by git

  • Commit changed from 491ce08b3a40cd6b00c34ffdfeb539851136cc94 to e655b363ae30694fa5f36e6b0edbdddcfb95f0cf

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

​d34b012trac #16272: Merged with updated #16227
​5cab81ctrac #16286: Allow k=None in MOLS/TD/OA
​4970da4trac #16286: Merged with updated #16272
​cb2e272trac #16235: Merged with updated #16286
​1a13ff8trac #16241: New MOLS shared by Ian Wanless
​2af2acftrac #16241: missing links and tests
​7e77f90trac #16241: Broken doctests
​e655b36trac #16236: Five mutually orthogonal latin squares of order 12

comment:8 Changed 7 years ago by vdelecroix

  • Status changed from needs_review to needs_work

Hi Nathann,

I guess this one needs also needs to be rebased on #16286 (that is a dependency of #16235).

Vincent

comment:9 follow-up: Changed 7 years ago by ncohen

  • Dependencies changed from #16235 to #16241
  • Status changed from needs_work to needs_review

Nonono actually this one is above #16241

comment:10 in reply to: ↑ 9 Changed 7 years ago by vdelecroix

Replying to ncohen:

Nonono actually this one is above #16241

Anyway, there is some rebase to do as there are some merging conflict with #16241.

comment:11 Changed 7 years ago by vdelecroix

  • Status changed from needs_review to needs_work

comment:12 Changed 7 years ago by git

  • Commit changed from e655b363ae30694fa5f36e6b0edbdddcfb95f0cf to b51626696a83e0072ae5c9a2ef42fc505f1da37f

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

​a97434ftrac #16286: more doc, more tests
​aeab437trac #16235: merge the updated #16286
​daa3835trac #16235: case n=1 + doc
​31a53f2trac #16235: update the MOLS table
​83b0d2ctrac #16241: Merged with updated #16235
​c212bf9trac #16241: Broken doctests
​b516266trac #16236: Merged with updated #16241

comment:13 Changed 7 years ago by ncohen

  • Status changed from needs_work to needs_review

comment:14 Changed 7 years ago by git

  • Commit changed from b51626696a83e0072ae5c9a2ef42fc505f1da37f to f5339fabafd4d2b3ffe56b1e68bb68d75ec19902

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

​f5339fatrac #16236: Broken doctests

comment:15 follow-up: Changed 7 years ago by vdelecroix

Hi Nathann,

Don't you have better reference rather than "given by Brendan McKay"?

I thought that we moved TD6_12 from orthogonal_arrays to database... what happend?

Vincent

comment:16 in reply to: ↑ 15 ; follow-up: Changed 7 years ago by ncohen

Yo !

Don't you have better reference rather than "given by Brendan McKay"?

Well.... I sent him an email, he gave me the matrices .... :-P

I thought that we moved TD6_12 from orthogonal_arrays to database... what happend?

Yeah I thought I did too... Perhaps one of the conflicts, who knows ? :-D

Anyway the good news is that we did not end up with a weird situation where it has not been moved WHILE some code is trying to load it from database.py. At least it works. Well, we will fix that in a later patch. I hope that Volker will release the next beta soon, because right now the Design code is a bit "foggy". It exists as a set of differences on several trac tickets, but I would like to have it in Sage and be sure that there are no conflicts anymore :-D

Nathann

comment:17 in reply to: ↑ 16 ; follow-up: Changed 7 years ago by vdelecroix

  • Status changed from needs_review to positive_review

Replying to ncohen:

Yo !

Don't you have better reference rather than "given by Brendan McKay"?

Well.... I sent him an email, he gave me the matrices .... :-P

I thought that we moved TD6_12 from orthogonal_arrays to database... what happend?

Yeah I thought I did too... Perhaps one of the conflicts, who knows ? :-D

Anyway the good news is that we did not end up with a weird situation where it has not been moved WHILE some code is trying to load it from database.py. At least it works. Well, we will fix that in a later patch. I hope that Volker will release the next beta soon, because right now the Design code is a bit "foggy". It exists as a set of differences on several trac tickets, but I would like to have it in Sage and be sure that there are no conflicts anymore :-D

Me too!!

comment:18 in reply to: ↑ 17 Changed 7 years ago by ncohen

Thanks for the review !

Nathann

comment:19 Changed 7 years ago by vbraun_spam

  • Milestone changed from sage-6.2 to sage-6.3

comment:20 Changed 7 years ago by vbraun

Reviewer name

comment:21 Changed 7 years ago by ncohen

  • Reviewers set to Vincent Delecroix

comment:22 Changed 7 years ago by git

  • Commit changed from f5339fabafd4d2b3ffe56b1e68bb68d75ec19902 to 2d7da8a150862462eaadd3e261ecc81abbe712c9
  • Status changed from positive_review to needs_review

Branch pushed to git repo; I updated commit sha1 and set ticket back to needs_review. Last 10 new commits:

​5e8b2aftrac #16277: removes a doctests that appears several times already
​acf8988trac #16277: Merged with updated #16231
​485e880trac #16277: Merged with updated #16248
​411a759trac #16286: Merged with updated #16277
​11eff2ctrac #16235: Merged with 6.2
​5a0e3fetrac #16235: Merged with #16231
​c0b13c4trac #16235: Merged with updated #16286
​9fb0f62trac #16241: useless if condition in MOLS constructor
​67ab2d2trac #16241: Merged with updated #16235
​2d7da8atrac #16236: Merged with updated #16241

comment:23 Changed 7 years ago by ncohen

  • Status changed from needs_review to positive_review

comment:24 Changed 7 years ago by vbraun

  • Branch changed from u/ncohen/16236 to 2d7da8a150862462eaadd3e261ecc81abbe712c9
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.