Conversation
…ing the django admin...
* CU-869bu4tum: Add test to make sure that saved model is NOT a wrapper * CU-869bu4tum: Move wrapper to a fully delegating one * CU-869bu4tum: Add a comment regarding setattr * CU-869bu4tum: Add a few more tests for wrapper (i.e getting attributes, properties, methods) * CU-869bu4tum: Fix injection tests * CU-869bu4tum: Add some comments to injection tests * CU-869bu4tum: Fix test time wrapping * CU-869bu4tum: Add a few more comprehensive tests * CU-869bu4tum: Add a few more comprehensive tests for property / method / attribtue getting * CU-869bu4tum: Fix trainer property for wrapper * CU-869bu4tum: Simplify trainer property
#297) * MedCAT v1: fixed relation infer output (entities not outputting start/end char positions. * Lints.
…el roles and add separate client for backend auth (#298) * refactor(medcat-trainer): Externalise VITE_ env vars, introduce separate client for backend auth for clarity, update API migrations, ensure roles can be read from top-level role group. * doc(medcat-trainer): Provide more comprehensive OIDC auth guide * doc(medcat-trainer): Update OIDC Auth guide and installation file * refactor(medcat-trainer): Address code review feedback * chore(medcat-trainer): Potential fix for code scanning alert no. 151: Clear-text logging of sensitive information Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> * refactor(medcat-trainer): Address more code review feedback --------- Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
* feat(medcat-trainer): improved demo screen, model pack selection, new clinical text component, improved demo filtering concept picker * fix(medcat-trainer): CU-8699ryke4: test type fixes * fix(medcat-trainer): pr review changes * fix(medcat-trainer): address auto feedback --------- Co-authored-by: Tom Searle <tom@cogstack.org>
…t as models.py calls insert twice (#299)
…_OIDC is set to true (#301)
* CU-869bvptgq: Simplify relations output somewhat * CU-869bvptgq: Add relation both entity start and end char positions * CU-869bvptgq: Add entity start / end positions to rel-cat * CU-869bvptgq: Add relevant changes to relation datasets * CU-869bvptgq: Minor linting change * CU-869bvptgq: Fix minor linting issue
…dised github actions for docker (#302) * build(medcat-trainer): Use Docker actions for github CI builds * build(medcat-trainer): Create a docker image for every PR * build(medcat-trainer): Consolidate workflows into one file. Updating docker action
…ct page, more options on the page
…from the 'setting' cog'
…around dataset schema
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
tomolopolis
force-pushed
the
admin-page
branch
from
f8edb29 to
6378159
Compare
…cross-site scripting Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
…hrough an exception Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
…uttons. Refactor common styles to _admin.scss
| </script> | ||
|
|
||
| <style scoped lang="scss"> | ||
| @import '@/styles/variables.scss'; |
Collaborator
There was a problem hiding this comment.
Can you put this css into another file?
alhendrickson
approved these changes
Feb 9, 2026
Collaborator
alhendrickson
left a comment
alhendrickson
left a comment
There was a problem hiding this comment.
Trying out saving a model pack, I hit this error, due to this PR.
I'll look at fixing it as I had planned to https://github.com/CogStack/cogstack-nlp/pull/305/changes
Approved this PR after realising it shouldnt be blocked by the above
…nsistency between buttons, action buttons, improve validation notifications, fix cdb_search_filter being set from model pack or concept_db
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Django admin for managing projects is starting to become unwieldy.
A user friendly form of project setup is needed. This limits what can be setup directly to the
Project,Model Packs,DatasetsandUsersvia the app to provide this streamlined project setup exp.