Skip to content

Conversation

@mart-r
Copy link
Collaborator

@mart-r mart-r commented Oct 30, 2025

This updates the scripts downloader.

The help for the new CLI is as follows:

% python -m medcat download-scripts --help
usage: python -m medcat download-scripts [-h] [--overwrite-url OVERWRITE_URL] [--overwrite-tag OVERWRITE_TAG]
                                         [--log-level {DEBUG,INFO,WARNING,ERROR}]
                                         [destination]

Download medcat-scripts

positional arguments:
  destination           The destination folder for the scripts

options:
  -h, --help            show this help message and exit
  --overwrite-url OVERWRITE_URL
                        The URL to download and extract from. This is expected to refer to a .zip file that has a `medcat-scripts` folder.
  --overwrite-tag OVERWRITE_TAG
                        The tag to use from GitHub
  --log-level {DEBUG,INFO,WARNING,ERROR}
                        The log level for fetching

This means you can now specify a URL to download for.
For example, since there is currently not release with medcat-scripts, with the dev version this would download nothing because it won't find the medcat-scripts folde within the repo. However, you specify the overwrite URL to mitigate this:

python -m medcat download-scripts --overwrite-url https://github.com/CogStack/cogstack-nlp/archive/refs/heads/main.zip

Or, you can just specify the tag:

python -m medcat download-scripts --overwrite-tag main

@tomolopolis
Copy link
Member

Copy link
Collaborator

@alhendrickson alhendrickson left a comment

Choose a reason for hiding this comment

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

Epic, cheers for the time on this

@mart-r
Copy link
Collaborator Author

mart-r commented Oct 31, 2025

Note: the MedCAT main workflow be modified to remove the medcat-scripts tag creation.

@mart-r mart-r merged commit 8f0aebc into main Nov 5, 2025
24 of 28 checks passed
@mart-r mart-r deleted the feat/medcat/CU-869b09dk4-update-scripts-dl branch November 5, 2025 13:55
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.

4 participants