Skip to content

Sync with docs - #18

Merged
kris927b merged 13 commits into
mainfrom
sync-with-docs
Aug 26, 2025
Merged

kris927b merged 13 commits into
mainfrom
sync-with-docs

Conversation

@kris927b

Copy link
Copy Markdown
Contributor

[v1.0.1] - 2025-08-22

Added

  • Added a script for syncing datasheets with our docs site
    • Created simplified main "README" template for syncing
  • Added a simplified language distribution plot. It only uses the "main" language.

Changed

  • Changed the document size distribution plots to use log bins instead of fixed width bins.
  • Changed the dataset sizes plot to make sure all datasets are represented on the y-axis.

@KennethEnevoldsen

Copy link
Copy Markdown
Contributor

Very hard PR to review given all the changes (and the fact that GitHub is very slow to handle the diff). Though generally it looks great (though maybe worth splitting the plots up in a separate PR next time)

Wasn't the DFM site going to be replaced, so I'm unsure if these changes will last.

For the site: does mkdocs handle the frontmatter when it is rendered? (Have you tried building the site?)

@kris927b

Copy link
Copy Markdown
Contributor Author

@KennethEnevoldsen I am aware that this is a big change. And down the line we should separate the changes a bit more.

I am unsure of the timeline for decomissioning the site, so I discussed with @saattrupdan that this might be a good solution for now.

I have tested that the mkdocs works. mkdocs removes the frontmatter, so it looks clean. Below is a few screenshots.
Also I have a PR on the site repo here: danish-foundation-models/site#323

screencapture-localhost-8000-data-cvr-reports-cvr-reports-html-2025-08-26-08_37_43 screencapture-localhost-8000-data-retsinformationdk-retsinformationdk-html-2025-08-26-08_36_55 screencapture-localhost-8000-dfm-data-html-2025-08-26-08_36_02

@KennethEnevoldsen

Copy link
Copy Markdown
Contributor

For us, it works as well (to be honest, I think it is quite nice that the current site is static)

@kris927b

Copy link
Copy Markdown
Contributor Author

@KennethEnevoldsen I will merge this then.

The sheets are also live at foundationmodels.dk

@kris927b
kris927b merged commit 97e4d41 into main Aug 26, 2025
2 checks passed
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.

2 participants