Skip to content

Update load_mined_publications.py - fix encoding and add more options#356

Merged
dglemos merged 5 commits intoEBI-G2P:mainfrom
dglemos:update_load_mined_pub_encoding
Mar 16, 2026
Merged

Update load_mined_publications.py - fix encoding and add more options#356
dglemos merged 5 commits intoEBI-G2P:mainfrom
dglemos:update_load_mined_pub_encoding

Conversation

@dglemos
Copy link
Copy Markdown
Collaborator

@dglemos dglemos commented Mar 3, 2026

Description


JIRA Ticket

[Link to the JIRA ticket]


Contributor Checklist

  • The code compiles and runs as expected
  • Relevant unit tests are added or updated
  • All unit tests are passing
  • Code follows the G2P Coding Guidelines
  • Documentation (code comments, confluence, etc.) has been updated as needed

Reviewer Checklist

Please follow the Code Review Guidelines when reviewing this PR.

  • I have followed all review guidelines

@dglemos dglemos changed the title Update load_mined_publications.py - fix encoding Update load_mined_publications.py - fix encoding and add more options Mar 10, 2026
@dglemos dglemos marked this pull request as ready for review March 11, 2026 10:50
@dglemos dglemos requested a review from seeta-ramaraju March 11, 2026 10:50
@dglemos dglemos marked this pull request as draft March 11, 2026 10:51
@dglemos dglemos marked this pull request as ready for review March 11, 2026 11:44
@dglemos dglemos marked this pull request as draft March 11, 2026 13:23
@dglemos dglemos marked this pull request as ready for review March 11, 2026 18:32
Copy link
Copy Markdown
Collaborator

@seeta-ramaraju seeta-ramaraju left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dglemos dglemos merged commit 3274fed into EBI-G2P:main Mar 16, 2026
3 checks passed
@dglemos dglemos deleted the update_load_mined_pub_encoding branch March 16, 2026 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants