-
Notifications
You must be signed in to change notification settings - Fork 4
Development: Port new gender contracts
#2638
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
ge94zec
wants to merge
144
commits into
main
Choose a base branch
from
feat/2637-port-new-gender-contracts
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
144 commits
Select commit
Hold shift + click to select a range
f4a0117
fix: added basicInfoFormValueSignal() as a tracked dependency so the …
ge94zec 40d1edf
fix: run compliance analysis before translation for faster feedback
ge94zec edbea60
Merge branch 'main' into bugfix/2321-title-compliance-error-not-shown
ge94zec a7392d8
fix: double-trigger for translate -> race condition
ge94zec 437788f
Merge remote-tracking branch 'origin/bugfix/2321-title-compliance-err…
ge94zec 652bb5e
Added new categories in compliance sidebar
ge94zec 22c7fe1
added new categories in sidebar
ge94zec a184f9d
Merge branch 'main' into bugfix/2321-title-compliance-error-not-shown
ge94zec 343c8cc
fix highlights across reload
ge94zec 7f5a227
Merge remote-tracking branch 'origin/bugfix/2321-title-compliance-err…
ge94zec b0840ab
prettier
ge94zec 4a13ca2
esLint
ge94zec 07b036c
Merge branch 'main' into feat/2345-add-new-compliance-categories-in-s…
ge94zec 55ba4dd
Merge branch 'main' into feat/2345-add-new-compliance-categories-in-s…
ge94zec 21e240e
esLint
ge94zec 0c46d89
Merge remote-tracking branch 'origin/feat/2345-add-new-compliance-cat…
ge94zec e5d7de4
fix translations
ge94zec f1e62d4
prettier
ge94zec f241af0
Merge branch 'main' into bugfix/2321-title-compliance-error-not-shown
ge94zec 1b847bd
change requests
ge94zec 3c99c9f
Merge branch 'main' into feat/2345-add-new-compliance-categories-in-s…
ge94zec 08537ab
Merge branch 'main' into feat/2345-add-new-compliance-categories-in-s…
az108 fba7bed
Merge branch 'main' into feat/2345-add-new-compliance-categories-in-s…
ge94zec ee05ead
Merge branch 'main' into bugfix/2321-title-compliance-error-not-shown
ge94zec 5c6f1e2
Merge branch 'main' into feat/2345-add-new-compliance-categories-in-s…
ge94zec 497111a
change requests
ge94zec 53dd4e5
Merge remote-tracking branch 'origin/feat/2345-add-new-compliance-cat…
ge94zec dcf5420
Merge branch 'main' into bugfix/2321-title-compliance-error-not-shown
ge94zec 08152f4
change requests
ge94zec ccdca07
eslint
ge94zec 87ab90e
change requests
ge94zec dca0e2a
Merge branch 'main' into bugfix/2321-title-compliance-error-not-shown
ge94zec 807e8d8
Merge branch 'main' into feat/2345-add-new-compliance-categories-in-s…
ge94zec 9e23fd7
change requests
ge94zec 1acbc29
Merge remote-tracking branch 'origin/bugfix/2321-title-compliance-err…
ge94zec 7fe140f
Merge branch 'main' into feat/2345-add-new-compliance-categories-in-s…
ge94zec e08cc5b
Merge remote-tracking branch 'origin/main' into bugfix/2321-title-com…
ge94zec cd8b599
Merge branch 'feat/2345-add-new-compliance-categories-in-sidebar' int…
ge94zec 8bb854e
feat: merge compliance add new categories
ge94zec 0107acc
updated editor
ge94zec a8bf9bd
Merge branch 'main' into bugfix/2321-title-compliance-error-not-shown
ge94zec cc271f7
Merge branch 'main' into bugfix/2321-title-compliance-error-not-shown
ge94zec 1f09ad4
Merge branch 'main' into bugfix/2321-title-compliance-error-not-shown
ge94zec 2842223
Merge branch 'main' into bugfix/2321-title-compliance-error-not-shown
ge94zec 6e1c011
Merge branch 'main' into bugfix/2321-title-compliance-error-not-shown
ge94zec f958bc2
Merge branch 'main' into bugfix/2321-title-compliance-error-not-shown
ge94zec 5ac21ff
feat: move gender decoder analysis to server side
ge94zec c9f0d10
refactor: restructure server-side bias analysis and reduce excessive …
ge94zec 203ca29
- processingOrder: genderDecoder -> original compliance and translate…
ge94zec 8b22df9
Merge remote-tracking branch 'origin/main' into bugfix/2379-fix-exces…
ge94zec 9a02e9b
updated openapi
ge94zec 5eddb05
Merge remote-tracking branch 'origin/main' into bugfix/2379-fix-exces…
ge94zec 158eec0
chore: update OpenAPI spec and generated client
github-actions[bot] 8b7c4d4
fixed client and server tests
ge94zec 7327454
Merge remote-tracking branch 'origin/bugfix/2379-fix-excessive-api-re…
ge94zec 46b5c94
fixed client test
ge94zec 04a26a6
fixed server test
ge94zec 817ab36
Merge branch 'main' into bugfix/2379-fix-excessive-api-requests-trigg…
ge94zec 343b52b
- prettier
ge94zec 72c15c5
Merge remote-tracking branch 'origin/bugfix/2379-fix-excessive-api-re…
ge94zec ee58a1f
add doc
ge94zec c3fe888
refactor: remove GenderBiasAnalysisResource and GenderBiasAnalysisRe…
ge94zec 7252ffa
Merge branch 'main' into bugfix/2379-fix-excessive-api-requests-trigg…
ge94zec 9a2db45
Merge remote-tracking branch 'origin/main' into bugfix/2379-fix-exces…
ge94zec 9f0aa5d
refactor: replace coding with enum GenderCategory
ge94zec 0627177
\`Bugfix\`: Replace double Job load with a targeted biased-issues query
az108 c94c107
Merge branch 'main' into bugfix/2379-fix-excessive-api-requests-trigg…
az108 da449e8
fix editor
ge94zec a36dd36
Merge remote-tracking branch 'origin/main' into bugfix/2379-fix-exces…
ge94zec eb73e71
moved BiasWordList and GenderCategory
ge94zec 7d9245b
prettier
ge94zec 29c625b
Merge branch 'main' into bugfix/2379-fix-excessive-api-requests-trigg…
ge94zec cbe0b64
`Bugfix`: Store biased issues in a Set so Hibernate stops rewriting t…
az108 762be05
\`Bugfix\`: Carry the biased-issues Set type through the repo and DTO…
az108 be9f574
refactor complianceScore and tests
ge94zec e0f9c72
Merge remote-tracking branch 'origin/bugfix/2379-fix-excessive-api-re…
ge94zec 2b77652
refactor complianceScore
ge94zec 1e8ae67
prettier
ge94zec 9c85baa
Merge branch 'main' into bugfix/2379-fix-excessive-api-requests-trigg…
ge94zec 032eefe
Merge branch 'main' into bugfix/2379-fix-excessive-api-requests-trigg…
ge94zec e4cd714
Merge branch 'main' into bugfix/2379-fix-excessive-api-requests-trigg…
ge94zec e3634b0
- refactored namings and BiasedIssues to Set
ge94zec d5c024b
Merge remote-tracking branch 'origin/main' into bugfix/2379-fix-exces…
ge94zec 3bfef56
chore: update OpenAPI spec and generated client
github-actions[bot] 6ff3240
- fixed test names
ge94zec 509a39a
Merge remote-tracking branch 'origin/bugfix/2379-fix-excessive-api-re…
ge94zec bdfb33a
- fixed AiResourceTest
ge94zec 914b244
prettier
ge94zec f1ed30b
fix test
ge94zec 8011d44
fix test
ge94zec 775de1b
prettier
ge94zec 71d4abb
Fix gender bias scoring persistence per job
ge94zec 2174c4d
chore: update OpenAPI spec and generated client
github-actions[bot] 880e9e7
Adjust tests and prettier
ge94zec 536fbc2
Merge remote-tracking branch 'origin/bugfix/2379-fix-excessive-api-re…
ge94zec a3ecf73
lint
ge94zec 0f65c7a
fix gender bias scoring, blank descriptions, and review feedback
ge94zec 8b5da77
merge main
ge94zec 8e0f3c6
fix(ai): persist and return multilingual job analysis consistently
ge94zec 84eb1d8
chore: update OpenAPI spec and generated client
github-actions[bot] 705e89e
fix server and client tests
ge94zec 05e731b
Merge remote-tracking branch 'origin/bugfix/2379-fix-excessive-api-re…
ge94zec 506a38b
feat: add genderDecoder UI in sidebar
ge94zec 8e5d39e
Merge branch 'main' into bugfix/2379-fix-excessive-api-requests-trigg…
ge94zec b9fe961
fix: synchronize autosave, translation and compliance analysis
ge94zec aad62bf
chore: update OpenAPI spec and generated client
github-actions[bot] 74ec58d
- fix server tests
ge94zec 1cc7ad7
Merge remote-tracking branch 'origin/bugfix/2379-fix-excessive-api-re…
ge94zec a25e72d
Merge branch 'main' into bugfix/2379-fix-excessive-api-requests-trigg…
ge94zec 811f5d0
fix: address review feedback for gender bias analysis
ge94zec c975f3b
prettier
ge94zec c1a3ef9
chore: update OpenAPI spec and generated client
github-actions[bot] 581b926
test: drive editor tests through public inputs and template events
ge94zec 73e5155
Merge remote-tracking branch 'origin/bugfix/2379-fix-excessive-api-re…
ge94zec dee54f3
chore: update OpenAPI spec and generated client
github-actions[bot] b2edf95
fix client test
ge94zec 0c27f27
Merge remote-tracking branch 'origin/bugfix/2379-fix-excessive-api-re…
ge94zec 7ea50d7
fix server test
ge94zec 33b2fc2
optimized tests
ge94zec f84b5d0
fix: prevent AI compliance analysis without user consent
ge94zec 232dbb9
fix client tests
ge94zec c52b9f8
prettier
ge94zec 431156e
prettier
ge94zec 1f1ac8e
test: compare parallel AI processing performance on test server
ge94zec ed0bcc8
Revert analysis and translation to sequential processing
ge94zec 487bbcc
Merge branch 'main' into feat/2378-improve-genderdecoder-accuracy-usi…
ge94zec 89f7d0e
feat: add gender decoder insights and filtering to AI sidebar
ge94zec 46eb372
Merge remote-tracking branch 'origin/feat/2378-improve-genderdecoder-…
ge94zec 3e7d3a0
`fix(ui): tighten gender bias filter typing and pill count coverage`
ge94zec 8ebe8c4
Replace JobRepository.findByIdForAiUpdate with the inherited findById
ge94zec b55e5c0
fix(ui): correct gender bias highlights and loading feedback
ge94zec 9031293
rename test
ge94zec 23324c8
fix lighthouse scan
ge94zec bc60579
- add JavaDoc to GenderCategory
ge94zec f3e59e6
Merge branch 'main' into bugfix/2379-fix-excessive-api-requests-trigg…
ge94zec b085d5b
`General`: Move the rule based gender bias analysis out of the ai pac…
ge94zec a9bc453
Merge branch 'main' into feat/2378-improve-genderdecoder-accuracy-usi…
ge94zec 7c06146
Merge remote-tracking branch 'origin/main' into bugfix/2379-fix-exces…
ge94zec 09e2a5c
chore: update OpenAPI spec and generated client
github-actions[bot] 90d3110
Merge branch 'feat/2378-improve-genderdecoder-accuracy-using-ai' into…
ge94zec bafc4e9
refactor: port gender decoder UI to biased issue contracts
ge94zec d34eb9a
prettier
ge94zec d6e384a
Merge remote-tracking branch 'origin/main' into feat/2637-port-new-ge…
ge94zec 82d9587
fix: prevent partial-word gender highlights in the editor
ge94zec File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
50 changes: 50 additions & 0 deletions
50
src/main/resources/config/liquibase/changelog/00000000000053_add_biased_issues_to_jobs.xml
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,50 @@ | ||
| <?xml version="1.0" encoding="utf-8"?> | ||
| <databaseChangeLog | ||
| xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" | ||
| xmlns="http://www.liquibase.org/xml/ns/dbchangelog" | ||
| xsi:schemaLocation="http://www.liquibase.org/xml/ns/dbchangelog http://www.liquibase.org/xml/ns/dbchangelog/dbchangelog-latest.xsd"> | ||
|
|
||
| <changeSet id="053a_add_biased_issues_to_jobs" author="melissa"> | ||
| <preConditions onFail="MARK_RAN"> | ||
| <not> | ||
| <tableExists tableName="job_biased_issues"/> | ||
| </not> | ||
| </preConditions> | ||
| <createTable tableName="job_biased_issues"> | ||
| <column name="job_id" type="CHAR(36)"> | ||
| <constraints nullable="false" foreignKeyName="fk_biased_issues_job" references="jobs(job_id)" deleteCascade="true"/> | ||
| </column> | ||
| <column name="language" type="varchar(10)"/> | ||
| <column name="word" type="varchar(255)"/> | ||
| <column name="type" type="varchar(50)"/> | ||
| </createTable> | ||
| </changeSet> | ||
|
|
||
| <changeSet id="053b_add_unique_constraint_biased_issues" author="aniruddh"> | ||
| <preConditions onFail="MARK_RAN"> | ||
| <dbms type="mysql"/> | ||
| <not> | ||
| <uniqueConstraintExists tableName="job_biased_issues" constraintName="uk_biased_issues_job_lang_word_type"/> | ||
| </not> | ||
| </preConditions> | ||
| <addUniqueConstraint | ||
| tableName="job_biased_issues" | ||
| columnNames="job_id, language, word, type" | ||
| constraintName="uk_biased_issues_job_lang_word_type"/> | ||
| </changeSet> | ||
|
|
||
| <changeSet id="053c_rename_gender_bias_score_to_ai_score" author="melissa"> | ||
| <preConditions onFail="MARK_RAN"> | ||
| <columnExists tableName="jobs" columnName="gender_bias_score"/> | ||
| <not> | ||
| <columnExists tableName="jobs" columnName="ai_score"/> | ||
| </not> | ||
| </preConditions> | ||
| <renameColumn | ||
| tableName="jobs" | ||
| oldColumnName="gender_bias_score" | ||
| newColumnName="ai_score" | ||
| columnDataType="integer"/> | ||
| </changeSet> | ||
|
|
||
| </databaseChangeLog> | ||
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
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This whole file can go — it's a leftover from before
00000000000052_add_biased_issues_to_jobs.xmlmerged to main.It is byte-identical to the merged 052, apart from the changeset id prefixes:
It is also not registered in
master.xml— the diff doesn't touch that file at all — so it never runs.Those two facts happen to cancel out into "harmless": as-is it is dead weight. But if someone later notices the missing include and "fixes" it, this would re-attempt
createTable job_biased_issuesand thegender_bias_score->ai_scorerename on a schema where 052 already did both. ThepreConditions onFail="MARK_RAN"guards would absorb it, so still no breakage — just a second changeset that does nothing. Cleanest is to delete the file rather than leave that trap.job_biased_issuesandai_scoreare already on main via 052, so nothing here is needed.