Alex Young
Alex Young
This set of patches adds a very basic debian package definition. The package installs to `/usr` rather than `/usr/local`, and puts the documentation in the right place. There are a...
Hi, I've added a #trashable? instance method, which allows you to control whether or not a given record will be saved to the trash_records table based on its attributes. I...
This patch introduces the chatml prompt format (and associated tests) necessary for driving the [Dolphin 2.6](https://huggingface.co/cognitivecomputations/dolphin-2_6-phi-2) fine-tune of [`microsoft/phi-2`](https://huggingface.co/microsoft/phi-2). I'm using the fine-tune rather than the upstream because the prompt...
This is a feature I've been wanting for a while, but haven't seen go past anywhere else. It allows streaming raw audio from `stdin`. It's different from the `stdin` support...
## Description As currently written, `check=all scripts/githooks/check-markdown-format` will not only check files intentionally checked in, it will also check anything under, for instance, `.venv/` or `node_modules/` which are out of...
## Description Minor (but important!) changes to how docker is run, `DOCKER_IMAGE` handling, and some docs. ## Context This change wraps up a few different changes I needed to make...
I usually try to follow a rule of not committing generated files. [Here](https://github.com/nhs-england-tools/repository-template/blob/5a9f3f2e4bd8ff89eb14f4a048c406f469faadb0/docs/developer-guides/Scripting_Docker.md?plain=1#L161), though, we say: ```markdown 6. Commit all changes to these files - `infrastructure/images/cypress/Dockerfile` - `infrastructure/images/cypress/Dockerfile.effective` - `infrastructure/images/cypress/VERSION`...
I don't think this line is right: https://github.com/nhs-england-tools/repository-template/blob/5a9f3f2e4bd8ff89eb14f4a048c406f469faadb0/scripts/docker/docker.lib.sh#L55 It looks like an attempt to clean up intermediate images. However, when I run it on podman on a fairly ordinary dockerfile,...
We apply a `.PHONY: *` rule [here](https://github.com/nhs-england-tools/repository-template/blame/2bba42e993d6f1748e3b4af43e0df0824c193da8/scripts/init.mk#L86). The scope of the `.PHONY` target seems to be global, so by doing this the user can't use file-based dependency logic in the...
## Description The [prior tutorial branch](https://github.com/nhs-england-tools/repository-template/pull/120) involved Docker usage that we don't want to involve as a first contact with the framework. This (minor) rewrite skips the Docker setup, instead...