-
Notifications
You must be signed in to change notification settings - Fork 175
Modify the GAP banner to show a rendition of the logo #6128
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
|
Delightful! As you hinted, I've tried a few different things myself, and the best I could come up with was: after struggling with getting the lines down. I'd certainly be in favour of the current pull request. |
| Print( """ . - O """, "\n" ); | ||
| Print( """ / / \ """, gap, " ", GAPInfo.BuildVersion, sysdate, "\n" ); | ||
| Print( """ O---O | """, "Architecture: ", GAPInfo.Architecture, "\n" ); | ||
| Print( """ \ \ / """, "Web: https://www.gap-system.org", "\n" ); | ||
| Print( """ ` - O """, "\n" ); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| Print( """ . - O """, "\n" ); | |
| Print( """ / / \ """, gap, " ", GAPInfo.BuildVersion, sysdate, "\n" ); | |
| Print( """ O---O | """, "Architecture: ", GAPInfo.Architecture, "\n" ); | |
| Print( """ \ \ / """, "Web: https://www.gap-system.org", "\n" ); | |
| Print( """ ` - O """, "\n" ); | |
| Print( """ ,-> (@) """, "\n" ); | |
| Print( """ v / ^ """, gap, " ", GAPInfo.BuildVersion, sysdate, "\n" ); | |
| Print( """ (@)-(@) ) """, "Architecture: ", GAPInfo.Architecture, "\n" ); | |
| Print( """ ^ \ v """, "Web: https://www.gap-system.org", "\n" ); | |
| Print( """ `-> (@) """, "\n"); |
Ill throw my hat into the ring too:
,-> (@)
v / ^ GAP 4.16dev-18-gd3d75c4 built on 2025-09-23 23:00:44+0100
(@)-(@) ) Architecture: x86_64-pc-linux-gnu-default64-kv10
^ \ v Web: https://www.gap-system.org
`-> (@)
Configuration: gmp 6.2.1, GASMAN
Loading the library and packages ...
Packages: AClib 1.3.3, Alnuth 3.2.1, AtlasRep 2.1.9, AutoDoc 2025.05.09, AutPGrp 1.11.1, Browse 1.8.21, CaratInterface 2.3.7, CRISP 1.4.8, Cryst 4.1.30, CrystCat 1.1.10,
CTblLib 1.3.11, FactInt 1.6.3, FGA 1.5.0, Forms 1.2.13, GAPDoc 1.6.7, genss 1.6.9, IO 4.9.3, IRREDSOL 1.4.4, LAGUNA 3.9.7, orb 5.0.1, PackageManager 1.6.3,
Polenta 1.3.11, Polycyclic 2.17, PrimGrp 4.0.0, RadiRoot 2.9, recog 1.4.4, ResClasses 4.7.3, SmallGrp 1.5.4, Sophus 1.27, SpinSym 1.5.2, StandardFF 1.0, TomLib 1.2.11,
TransGrp 3.6.5, utils 0.92
Try '??help' for help. See also '?copyright', '?cite' and '?authors'
gap>
Using escape codes like @mtorpey I can get the following:
In addition to the wrapping issues for colors I guess we also need to consider what happens if a terminal doesn't support them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whoops did not mean to start a review!
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
|
If we're still at the brainstorming stage, I could mention that we have a lot more of unicode at our disposal, which could improve things. Looking at Max's design above, we can use box-drawing characters and geometric shapes to smarten it up: (I've also incorporated the vertical "G A P" letters, which look good in the real logo.) Then again, there's something more playful and fun about pure ascii which is lost here. :) Note that the current logo uses box-drawing characters, and ascii-only displays have a fallback: and we could have a similar approach. |
|
I do like @mtorpey version as well, although the black&white version less so. @mtorpey could you also provide a render of a pure ASCII version, say with We are also considering to flip/rotate the logo, like here https://github.com/gap-system/gap-logo/tree/mh/rotated-60?tab=readme-ov-file#preview so you could also do that |
|
I can make a pull request if that'd be helpful. P.S. Code is at mtorpey@dba5b23 |








First iteration -- based on the sketches in https://github.com/gap-system/gap-logo/blob/main/gap-logo-ascii.txt but using a "rotated" / "flipped" version of the logo in anticipation of us changing the logo orientation (clearly we could also flip it back).
I am sure this could be improved. Perhaps e.g. @mtorpey would be interested in that ;-).
Here is a "preview" of the current state: