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

Lai rgalrong #1331

Merged
merged 9 commits into from
Sep 27, 2023
Merged

Lai rgalrong #1331

merged 9 commits into from
Sep 27, 2023

Conversation

LinguList
Copy link
Contributor

@LinguList LinguList commented Sep 23, 2023

Pull request checklist

  • add new concept list
  • add new metadata
  • add new Concepticon concept sets
    • checked whether the new concept(s) can be applied to existing lists with
      concepticon notlinked --gloss "NEW_GLOSS"
  • add new Concepticon concept relations
  • refine existing Concepticon concept set mappings
  • refine Concepticon glosses
  • refine Concepticon concept relations
  • refine Concepticon concept definitions
  • retire data

Additional information

Concept list must be added for a lexibank dataset, where we currently draft the revision.

@LinguList
Copy link
Contributor Author

@MuffinLinwist and @CarolinHu would you be available to review this PR quickly? We would like to resubmit at some point soon, so it would be helpful to have this already covered. We do not expect many problems, since the concept list is mainly based on Sagart's list from 2019.

@MuffinLinwist
Copy link
Collaborator

@MuffinLinwist and @CarolinHu would you be available to review this PR quickly? We would like to resubmit at some point soon, so it would be helpful to have this already covered. We do not expect many problems, since the concept list is mainly based on Sagart's list from 2019.

Yes! I'll add it as part of the tasks for the next two days..

@LinguList
Copy link
Contributor Author

@laiyunfan, you can see the current data in EDICOTR with a link that makes it easy to browse the data when using this link:

https://digling.org/edictor/?file=lairgyalrong.tsv&preview=500&basics=DOCULECT|CONCEPT|TOKENS|COGIDS&publish=true

I think we may argue that this is enough for the time being?

@LinguList
Copy link
Contributor Author

I'd then make a version for the current state of the data, and that would be it?

@laiyunfan
Copy link
Collaborator

Yes, I think that would be it. Then I will do a final check of the manuscript and the response, and will finally submit it.

Copy link
Collaborator

@MuffinLinwist MuffinLinwist left a comment

Choose a reason for hiding this comment

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

@LinguList and @laiyunfan, my review of this PR is finished. Just minor comments on some mappings.

Lai-2023-291-10 158 red 156 RED 1007 red
Lai-2023-291-11 14 black 163 BLACK 1005 black
Lai-2023-291-12 16 to blow (of wind) 175 BLOW (OF WIND) 1738 to blow (of wind)
Lai-2023-291-13 17 the body hair (hair or fur) 189 HAIR (BODY) 266 the body hair (hair or fur) // the hair
Copy link
Collaborator

Choose a reason for hiding this comment

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

Perhaps, HAIR OR FUR is better for this one.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The Chinese concept is maximally unspecified, but in this case, the base list links already to body hair, so we better just follow that mapping.

Lai-2023-291-53 170 the sand 671 SAND 44 the sand
Lai-2023-291-54 43 early 672 EARLY 1018 early
Lai-2023-291-55 134 the mouth 674 MOUTH 84 the mouth
Lai-2023-291-56 250 the shit 676 SHIT (DEFECATE) 151 the shit
Copy link
Collaborator

Choose a reason for hiding this comment

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

The current mapping refers to the action. I believe FAECES (EXCREMENT) would be a more suitable mapping for this gloss.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, this is a wrong mapping.

Lai-2023-291-81 161 right 1019 RIGHT 711 right
Lai-2023-291-82 425 to yawn 1028 YAWN to yawn
Lai-2023-291-83 76 good 1035 GOOD 1052 good
Lai-2023-291-84 80 the hair (of the head) 1040 HAIR 75 the hair (of the head)
Copy link
Collaborator

Choose a reason for hiding this comment

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

HAIR (HEAD) would be a more suitable mapping.

Lai-2023-291-84 80 the hair (of the head) 1040 HAIR 75 the hair (of the head)
Lai-2023-291-85 117 light (of weight) 1052 LIGHT (WEIGHT) 1015 light (of weight)
Lai-2023-291-86 233 the wheat 1077 WHEAT 410 the wheat
Lai-2023-291-87 359 the work 1086 WORK (ACTIVITY) the work
Copy link
Collaborator

Choose a reason for hiding this comment

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

The current mapping referes to the action. WORK (LABOUR) is a more suitable mapping.

Lai-2023-291-135 84 the head 1256 HEAD 74 the head
Lai-2023-291-136 239 the wing 1257 WING 298 the wing
Lai-2023-291-137 183 to sing 1261 SING 1184 to sing
Lai-2023-291-138 90 high / tall 1265 HIGH 968 high / tall
Copy link
Collaborator

Choose a reason for hiding this comment

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

HIGH OR TALL perhaps is a more suitable mapping.

Lai-2023-291-141 81 the hand 1277 HAND 107 the hand
Lai-2023-291-142 227 to vomit 1278 VOMIT 1535 to vomit
Lai-2023-291-143 379 to fall 1280 FALL to fall
Lai-2023-291-144 312 the fat (grease) 1282 FAT (FROM ANIMALS) the fat (grease)
Copy link
Collaborator

Choose a reason for hiding this comment

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

the fat (grease) has been previously mapped as FAT (ORGANIC SUBSTANCE) in our concept lists.

Lai-2023-291-182 153 to play 1413 PLAY 1703 to play
Lai-2023-291-183 414 to squeeze 1414 SQUEEZE to squeeze
Lai-2023-291-184 213 to think (reflect) 1415 THINK (REFLECT) 1726 to think (reflect)
Lai-2023-291-185 409 to sit 1416 SIT to sit
Copy link
Collaborator

Choose a reason for hiding this comment

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

Entry for HUANG_1992_1820 column is empty. Just to confirm, is this not missing?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

These are new entries. This is correct.

Lai-2023-291-287 45 the earthworm 2350 EARTHWORM 363 the earthworm
Lai-2023-291-288 38 the dream 2374 DREAM 699 the dream
Lai-2023-291-289 93 horizontal 2376 HORIZONTAL 1001 horizontal
Lai-2023-291-290 430 wide 2531 BROAD wide
Copy link
Collaborator

Choose a reason for hiding this comment

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

Perhaps, WIDE or BROAD OR WIDE is a more suitable mapping?

@LinguList
Copy link
Contributor Author

Thanks, @MuffinLinwist, good catches, I'll look into those later.

@LinguList
Copy link
Contributor Author

My updates should have addressed these cases now.

@LinguList
Copy link
Contributor Author

I'll thus merge now. I also fixed an error in Huang, which is useful, thanks @MuffinLinwist !

@LinguList LinguList merged commit b6508d4 into master Sep 27, 2023
1 check passed
@laiyunfan
Copy link
Collaborator

Thank you very much!

@AnnikaTjuka AnnikaTjuka deleted the lai-rgalrong branch March 14, 2024 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants