Opened 7 years ago
Closed 7 years ago
#16248 closed enhancement (fixed)
Some trivial Orthogonal Arrays (when n=1)
Reported by:  ncohen  Owned by:  

Priority:  major  Milestone:  sage6.3 
Component:  combinatorics  Keywords:  design, orthogonal array 
Cc:  vdelecroix, knsam  Merged in:  
Authors:  Nathann Cohen  Reviewers:  Vincent Delecroix 
Report Upstream:  N/A  Work issues:  
Branch:  5dc4a25 (Commits, GitHub, GitLab)  Commit:  5dc4a25b48a0900cf2242ba10f6bf5b568b2c6a5 
Dependencies:  #16272  Stopgaps: 
Description
When n=1
, the orthogonal array [[0]*k]
is a valid answer.
I met this as a cornercase of some constructions I am working on...
Nathann
Change History (17)
comment:1 Changed 7 years ago by
 Branch set to u/ncohen/16248
 Status changed from new to needs_review
 Summary changed from Some trivial Orthogonal Arrays to Some trivial Orthogonal Arrays (when n=1)
comment:2 Changed 7 years ago by
 Commit set to 26423171f7e3daab8c4164b2bcba485df875dbdd
comment:3 Changed 7 years ago by
 Status changed from needs_review to positive_review
Hi Nathann,
I love easy zerology tickets!
comment:4 Changed 7 years ago by
 Keywords design orthogonal array added
 Reviewers set to Vincent Delecroix
comment:5 Changed 7 years ago by
Ahahah. Well, I prefer when they are not graphrelated. I hate to spoil a clean graph algorithm with one thousand stupid "if" :P
Nathann
comment:6 Changed 7 years ago by
 Milestone changed from sage6.2 to sage6.3
comment:7 Changed 7 years ago by
 Status changed from positive_review to needs_work
Merge conflict in src/sage/combinat/designs/orthogonal_arrays.py
comment:8 Changed 7 years ago by
 Commit changed from 26423171f7e3daab8c4164b2bcba485df875dbdd to 36a705b6f5377da844955a7821e53d3beeb33803
Branch pushed to git repo; I updated commit sha1. Last 10 new commits:
e62fae8  corrected doctests + new doctests

4d6e964  trac #16227: Replace exception with booleans in the doctests

5cfee91  trac #16227: Merged with updated #15310

a46446f  trac #16231: Equivalence between OA/TD/MOLS

a9dce70  trac #16231: Merged with updated #16227

8d8b928  more documentation to orthogonal_arrays.py

8257178  remove MOLS construction for prime powers + doc

d051cf4  ValueError > EmptySetError in latin_squares

9e5e94f  trac #16231: Merged with updated #16227

36a705b  trac #16248: Merged with #16231

comment:9 Changed 7 years ago by
 Dependencies set to #16231
 Status changed from needs_work to positive_review
Last 10 new commits:
e62fae8  corrected doctests + new doctests

4d6e964  trac #16227: Replace exception with booleans in the doctests

5cfee91  trac #16227: Merged with updated #15310

a46446f  trac #16231: Equivalence between OA/TD/MOLS

a9dce70  trac #16231: Merged with updated #16227

8d8b928  more documentation to orthogonal_arrays.py

8257178  remove MOLS construction for prime powers + doc

d051cf4  ValueError > EmptySetError in latin_squares

9e5e94f  trac #16231: Merged with updated #16227

36a705b  trac #16248: Merged with #16231

comment:10 Changed 7 years ago by
I hope that this was where your conflict came from O_O;;;
Nathann
comment:11 Changed 7 years ago by
 Commit changed from 36a705b6f5377da844955a7821e53d3beeb33803 to 410200e3c025e1ed7da44f0bf602c24ae2f9d855
 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:
0812a73  trac #16281: Simplification

61dc86b  16281: long comment for the construction of the projective plane

51daa7f  trac #16281: correct a doctest

e090f92  trac #16272: merge #16281

9a221be  trac #16272: fix doctests

5074eee  trac #16272: finer doctest to test the output of transversal_design

d81f265  trac #16272: ultimate doctest

47798d2  trac #16272: simplifying the structure of orthogonal_array

d34b012  trac #16272: Merged with updated #16227

410200e  trac #16248: Merged with #16272

comment:12 Changed 7 years ago by
 Dependencies changed from #16231 to #16231, #16272
comment:13 Changed 7 years ago by
 Commit changed from 410200e3c025e1ed7da44f0bf602c24ae2f9d855 to 5dc4a25b48a0900cf2242ba10f6bf5b568b2c6a5
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
5dc4a25  trac #16248: Merged with #16272

comment:14 Changed 7 years ago by
 Status changed from needs_review to positive_review
comment:15 Changed 7 years ago by
I moved this above #16272. Looks like with git you'd better work with totally linear dependencies or the number of commits on each ticket will JUMP when you merged everything together _
Nathann
comment:16 Changed 7 years ago by
 Dependencies changed from #16231, #16272 to #16272
comment:17 Changed 7 years ago by
 Branch changed from u/ncohen/16248 to 5dc4a25b48a0900cf2242ba10f6bf5b568b2c6a5
 Resolution set to fixed
 Status changed from positive_review to closed
Branch pushed to git repo; I updated commit sha1. New commits:
trac #16248: Some trivial Orthogonal Arrays