Skip to content

ML Export for Amsterdam#1612

Closed
ramonavic wants to merge 2 commits into
mainfrom
feature/add-ml-amsterdam-export
Closed

ML Export for Amsterdam#1612
ramonavic wants to merge 2 commits into
mainfrom
feature/add-ml-amsterdam-export

Conversation

@ramonavic

@ramonavic ramonavic commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Description

Create a copy of ml.csv export but with more specific filtering and slugs instead of category names.
This is more aligned with the training algorithm of Amsterdam.

Checklist

  • Keep the PR, and the amount of commits to a minimum
  • The commit messages are meaningful and descriptive
  • The change/fix is well documented, particularly in hard-to-understand areas of the code / unit tests
  • Are there any breaking changes in the code, if so are they discussed and did the team agreed to these changes
  • Check that the branch is based on main and is up to date with main
  • Check that the PR targets main
  • There are no merge conflicts and no conflicting Django migrations
  • PR was created with the "Allow edits and access to secrets by maintainers" checkbox checked

How has this been tested?

  • Provided unit tests that will prove the change/fix works as intended
  • Tested the change/fix locally and all unit tests still pass
  • Code coverage is at least 85% (the higher the better)
  • No iSort, Flake8 and SPDX issues are present in the code

@ramonavic ramonavic force-pushed the feature/add-ml-amsterdam-export branch from 6cdcf88 to bcbd7ad Compare June 29, 2026 15:04
@ramonavic ramonavic closed this Jun 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant