Opened 4 months ago

Closed 4 months ago

Last modified 3 months ago

#30286 closed enhancement (fixed)

Graph: last sporadic distance-regular graphs

Reported by: gh-Ivo-Maffei Owned by:
Priority: major Milestone: sage-9.2
Component: graph theory Keywords: drg, coset_graph
Cc: dimpase Merged in:
Authors: Ivo Maffei Reviewers: Dima Pasechnik
Report Upstream: N/A Work issues:
Branch: 73c4fed (Commits) Commit:
Dependencies: #30260, #30253 Stopgaps:

Description

Added a few more distance-regular graphs.

Change History (7)

comment:1 Changed 4 months ago by gh-Ivo-Maffei

  • Branch set to public/graphs/30286
  • Cc dimpase added
  • Commit set to 68da709f5f06485c9c497218faef5e286a2e3021
  • Status changed from new to needs_review

This is ready for review, but we need #30277 to be done so that #30240 can go.

Last 10 new commits:

​1978c7dadded a few sporadic distance regular graphs
​384fb27moved file to generators; fixed tests
​85b4d55added 4 more sporadic drg
​4c4f69badded remaining Witt graphs
​9c81329renamed locally GQ function and added references
​b88df6badded reference blocks
​ecde7cafix wrong rebase merge conflict resolution
​e897a6bMerge branch 'drg_sporadic2' into drg_sporadic3
​9b11d8dadded the last sporadic drg
​68da709added references; added name to graph catalog

comment:2 follow-up: Changed 4 months ago by dimpase

"Linst-Schrijver graph" should be "van Lint-Schrijver graph" (Dutch names with van have v, while Flemish have V, and van must not be skipped :-))

comment:3 Changed 4 months ago by git

  • Commit changed from 68da709f5f06485c9c497218faef5e286a2e3021 to 73c4feda56bf58a2f116dfb0376fd75feddd59f3

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

​59299e6fix some docstrings
​574960dadded gap_packages flag to atlasrep
​9db5fd3Merge branch 'drg_sporadic1' into drg_sporadic2
​a87fad1added gap_packages flag to atlasrep
​73c4fedMerge ticket 30260 into ticket 30286

comment:4 Changed 4 months ago by dimpase

  • Reviewers set to Dima Pasechnik
  • Status changed from needs_review to positive_review

ok, good.

comment:5 Changed 4 months ago by vbraun

  • Branch changed from public/graphs/30286 to 73c4feda56bf58a2f116dfb0376fd75feddd59f3
  • Resolution set to fixed
  • Status changed from positive_review to closed

comment:6 in reply to: ↑ 2 ; follow-up: Changed 3 months ago by dimpase

  • Commit 73c4feda56bf58a2f116dfb0376fd75feddd59f3 deleted

Replying to dimpase:

"Linst-Schrijver graph" should be "van Lint-Schrijver graph" (Dutch names with van have v, while Flemish have V, and van must not be skipped :-))

this was not quite fixed, I still see LintSchrijver instead of vanLintSchrijver in graphs.*

comment:7 in reply to: ↑ 6 Changed 3 months ago by dimpase

Replying to dimpase:

Replying to dimpase:

"Linst-Schrijver graph" should be "van Lint-Schrijver graph" (Dutch names with van have v, while Flemish have V, and van must not be skipped :-))

this was not quite fixed, I still see LintSchrijver instead of vanLintSchrijver in graphs.*

you can fix it on #30312, for instance

Note: See TracTickets for help on using tickets.