Skip to content

Conversation

@hherde
Copy link

@hherde hherde commented Feb 8, 2026

Thanks for helping me learn about Just! I added some text to the parent readme explaining what Just is and that it needs to be installed to set up to use the commands in the readme.

Thanks for helping me learn about Just! I added some text to the parent readme explaining what Just is and that it needs to be installed to set up to use the commands in the readme.
@hherde
Copy link
Author

hherde commented Feb 8, 2026

Lol we should add that we need wget as well for MacOS

@hherde
Copy link
Author

hherde commented Feb 8, 2026

Added LunDMX variation

@hherde hherde changed the title Update README.md: Add Just information Update README.md: Add Just information + Add LunDMX supprot Feb 8, 2026
@hherde hherde changed the title Update README.md: Add Just information + Add LunDMX supprot Update README.md: Add Just information + Add LunDMX support Feb 8, 2026
- Using the [Aileron](https://www.fontsquirrel.com/fonts/aileron?filter%5Bfamily_size%5D=12) font

## Set Up
Automations run with `just` command here use the [Just system](https://just.systems). Please consider installing Just first. You will also need `wget` (see eg [this gist](https://gist.github.com/scisco/4c63a2659a6ea33eb987)).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe it would be better to point at our own documentation in https://ldmx-software.github.io/developing/getting-started.html#install-just ?

Copy link
Member

@tomeichlersmith tomeichlersmith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It brings me so much joy to see other folks working on this 🥲

  • Does MacOS have curl by default? I could switch the font download to use curl instead of wget.
  • An alternative un design is to shift the L left. This would require more tweaking (I couldn't get it to work easily/quickly), but it should be possible. Thoughts?

dm-text-color: auto,
prefix: none,
suffix: none,
un: none, // For adding the "un" in LunDMX
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Put argument documentation up in the comment area above #let logo. (You'll see the other arguments there as well).

// point that electron should stop relative to target
// ends in the upper left corner of the D as if the D is the calorimeters
// depends on the font and size
//let beam-endpoint = (0.7cm, +0.35cm)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably leftover from development

Suggested change
//let beam-endpoint = (0.7cm, +0.35cm)

Comment on lines +115 to +126
(0,0),
if prefix == none [] else if type(prefix) == str {
align(top + right, text(size: 15pt, weight: "light", prefix))
} else {
prefix
},
height: text-box-height,
width: if prefix == none { min-prefix-width } else { auto },
name: <prefix>
),
node(
(rel: target-shift, to: <L>),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As far as I can tell, this stuff is the same as whats on main, but has been reformatted. I'm guessing your text editor is struggling to understand the Typst syntax - I can undo this manually if you want, but you can also look at installing a Typst syntax/format extension.

shape: rect,
),
node(

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems leftover from development

Suggested change

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants