Skip to content
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

Bug: Synonym sync: Redundant whitespace #738

Merged
merged 1 commit into from
Jan 10, 2025
Merged

Bug: Synonym sync: Redundant whitespace #738

merged 1 commit into from
Jan 10, 2025

Conversation

joeflack4
Copy link
Contributor

@joeflack4 joeflack4 commented Dec 23, 2024

Resolves #733

Overview

Now JOINing Mondo and source on the lowercasing + redundant-whitespace-removed variation of synonym strings. Also for -added, will now be removing redundant whitespace from synonyms.

Pre-merge checklist

Documentation

Was the documentation added/updated under docs/?

  • Yes
  • No, updates to the docs were not necessary after careful consideration

QC

Was the full pipeline run before submitting this PR using sh run.sh make build-mondo-ingest on this branch (after
docker pull obolibrary/odkfull:dev), and no errors occurred?

New Packages

Were any new Python packages added?

Were any other non-Python packages added?

PR Review and Conversations Resolved

Has the PR been sufficiently reviewed by at least 1 team member of the Mondo Technical team and all threads resolved?

  • Yes

- Update: Now JOINing Mondo and source on the lowercasing + redundant-whitespace-removed variation of synonym strings.
- Update: For -added, will now be removing redundant whitespace from synonyms.
@joeflack4 joeflack4 self-assigned this Dec 23, 2024
@joeflack4 joeflack4 linked an issue Dec 23, 2024 that may be closed by this pull request
@joeflack4 joeflack4 added the bug Something isn't working label Dec 23, 2024
Copy link
Contributor

@twhetzel twhetzel left a comment

Choose a reason for hiding this comment

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

Looks good!

@joeflack4
Copy link
Contributor Author

@matentzn Just need your approval on this PR now. Pretty simple task: it just removes redundant whitespace from synonyms. This was causing an issue where we were not able to JOIN between Mondo and source terms sometimes.

@twhetzel
Copy link
Contributor

@joeflack4 do you have a specific question on this PR you are waiting on Nico for? Remember he is busy on other projects for Q1.

@joeflack4
Copy link
Contributor Author

@twhetzel Ah, you mentioned that. But I assumed he'd still be reviewing. Nothing specific. Do we want to do 1 reviewer only until some time in Feb, Mar, or April?

@matentzn
Copy link
Member

I would suggest for now at least to only ask a review of me if Trish is uncertain about something.. I have a terrible month in terms of competing priorities and while I am committed to always do everything, things will be severely delayed..

In the same vein it would be good if super high priority tasks (reviews, issues, questions) are always pushed to me on slack.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: Synonym sync: Redundant whitespace
3 participants