-
Notifications
You must be signed in to change notification settings - Fork 317
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
AutoPR for paper information correction #4147
Merged
Changes from 101 commits
Commits
Show all changes
106 commits
Select commit
Hold shift + click to select a range
aa79252
Auto pr for paper information correction (#4138)
yufei118liu f93544f
Auto pr (#4146)
yufei118liu 6831fe1
Merge branch 'master' into autopr
mjpost 6404262
Add new correction form and paper link to form
mjpost 7cad0c6
black
mjpost 4325c92
Define missing variable
mjpost 8fc8bc3
Merge branch 'master' into autopr
mjpost afd9952
Add issue title; make id conditional
mjpost 228162c
Merge remote-tracking branch 'origin/master' into autopr
mjpost 5571e85
Remove non-working hidden template
mjpost 755c471
Fix inline variable expansion
mjpost 635222f
Correct template to JSON
mjpost ebfeafb
Partial bulk edit script
mjpost b727d5f
black
mjpost f17bacb
Remove first attempt
mjpost 62b9787
Remove alert, fix string
mjpost 949d051
Remove unused keys from hugo stub
mjpost 6a6adfa
Add {title,booktitle,abstract}_raw
mjpost d2bb10b
Use raw fields in form
mjpost 904fc44
Format authors as single lines
mjpost 4017fbd
Remove variable
mjpost b2f37cc
black
mjpost 490b352
Fix variable
mjpost 4189ad5
Remove custom function
mjpost d711d8e
Update template
mjpost 0d3d24d
Remove booktitle_raw
mjpost 3a3a690
Revert ingestion requestion change
mjpost 71703aa
black
mjpost 514f3dc
Testing metadata correction locations
mjpost 855b099
black
mjpost 200bf64
Playing with correction link location
mjpost ee2da1e
black
mjpost f18f44d
Fiddling
mjpost 450144c
Change button color
mjpost 65161fe
Add metadata issue annotator
mjpost 5409a8a
black
mjpost 212c530
Loosen JSON check
mjpost 01e45cc
Merge remote-tracking branch 'origin/master' into autopr
mjpost a58abce
Simplify workflow
mjpost 5ac541e
Merge branch 'master' into autopr
mjpost a942394
Remove manual metadata correction template
mjpost 1b14581
black
mjpost 04cd465
core.setFailed -> console.log
mjpost f1cc6ea
Split into steps
mjpost 257f431
Merge remote-tracking branch 'origin/master' into autopr
mjpost 3837995
Add JSON block syntax
mjpost 7d48671
Add LLM-based issue validation
mjpost 9fe9644
Progress on bulk script
mjpost 5955694
black
mjpost 494194a
Refactoring a bit in preparation for the dialog
mjpost 136fd8a
Added example dialog script
mjpost 7b57813
First-pass LLM validator working
mjpost 8ea1a2f
Add LLM metadata validator
mjpost 7d89733
Merge branch 'llm-validator' into autopr
mjpost fdf9d7d
whitespace
mjpost 64d5adf
Nice example but dragging doesn't work
mjpost d61e0f4
More finessing
mjpost 6057899
Merge remote-tracking branch 'origin/master' into autopr
mjpost 7e09025
Remove temp files
mjpost b150ed9
Restore dl
mjpost d28c334
Add code for display dialog
mjpost 76239c2
Added missing styles
mjpost de2dbdc
Minor rewording
mjpost 8f57839
black
mjpost 265b01b
rename
mjpost fef5c2b
Restore additional entries
mjpost 1eacc43
Add license header
mjpost a06a988
Add pen icon
mjpost e0de401
Remove duplicate authorsContainer
mjpost fafe50a
Use evidenced dialog closer
mjpost 3940b9b
Added dummy static file used for layout testing
mjpost 038fe39
black
mjpost 30e0e7b
black
mjpost 3f4aa3c
Move styling propers
mjpost 1845849
Only submit changes
mjpost a87bc3e
Fiddle with responsiveness
mjpost 89262c5
Try to fit drag control with first name
mjpost 283aa96
Trying again
mjpost 8e2bd47
Debugging grouping, add alerts
mjpost d2f55ed
More compact grabber
mjpost 0902946
Fix spacing, remove alerts
mjpost 2cfae96
Restore anth ID in title
mjpost c11e7c2
Update documentation
mjpost 7692326
Bulk script works
mjpost 25ce356
Remove existing authors
mjpost 4b7db8b
Add --dry-run
mjpost 2ce4c75
Add author node in correct place
mjpost 29269e0
Properly handle updated subtags
mjpost f449cc7
black
mjpost 37cd0b0
Merge branch 'master' into autopr
mjpost 0253ffe
Add author IDs to comparison
mjpost b81970a
Preserve only explicit IDs
mjpost 4252ded
Add flag to close old issues
mjpost fe50ccd
Update note
mjpost 3099ce7
Remove test code
mjpost 62181e2
Restore deleted lines
mjpost 68b9c04
Added workflow to remove approval status
mjpost 7e592c9
Look for "approved" label
mjpost 85e9ce7
Cleanup
mjpost 26b29e7
black
mjpost f6e173f
Use try block around each issue
mjpost f4c2d3e
clean up header
mjpost 6503e48
Add missing anthology id key to javascript
mjpost 50fe7f9
typos
mjpost 5540e45
Fix XML file paths
mjpost 599ef1e
Remove LLM checker
mjpost 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
This file was deleted.
Oops, something went wrong.
This file contains 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 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 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,38 @@ | ||
name: Reset Approval on Edit | ||
|
||
on: | ||
issues: | ||
types: | ||
- edited | ||
|
||
jobs: | ||
reset-approval: | ||
if: contains(join(github.event.issue.labels.*.name, ','), 'correction') || contains(join(github.event.issue.labels.*.name, ','), 'metadata') | ||
runs-on: ubuntu-latest | ||
steps: | ||
- name: Leave a comment and reset approval status | ||
uses: actions/github-script@v6 | ||
with: | ||
script: | | ||
const { issue, repository } = context.payload; | ||
const owner = repository.owner.login; | ||
const repo = repository.name; | ||
const approvedLabel = "approved"; | ||
|
||
// Check if the issue has the "approved" label and remove it | ||
if (issue.labels.some(label => label.name === approvedLabel)) { | ||
await github.issues.removeLabel({ | ||
owner, | ||
repo, | ||
issue_number: issue.number, | ||
name: approvedLabel | ||
}); | ||
|
||
// Add a comment to notify about the edit | ||
await github.issues.createComment({ | ||
owner, | ||
repo, | ||
issue_number: issue.number, | ||
body: "Approval status has been reset after the issue was edited." | ||
}); | ||
} |
This file contains 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 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.
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.
The phrase 'If you request an author name correction for yourself, please help ensure your name is correct for future publications by setting it correctly in Softconf/OpenReview.' should be 'If you request an author name correction for yourself, please ensure your name is correct for future publications by setting it correctly in Softconf/OpenReview.' for clarity.
Copilot is powered by AI, so mistakes are possible. Review output carefully before use.