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

Format the author as a multi-line list_Youtube #526

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

SeverusYixin
Copy link
Collaborator

@SeverusYixin SeverusYixin commented Nov 27, 2024

Format the author field as a multi-line list to assist data_normalizer.py in recognizing and normalizing authors' names more effectively.

This PR solved problem:

- authors: "Fabig, Gunar, Jannasch, Anett, Okafornta, Chukwuebuka, Boissonnet, Tom,\

This PR related to #512
This PR closes #512

@SeverusYixin
Copy link
Collaborator Author

Hi @haesleinhuepf , would you mind helping me review this too, after the update the output looks like #524

Thank you!

@haesleinhuepf
Copy link
Member

I'm happy to review this. But before: would you mind writing a sentence or two about what this PR brings? I don't want to guess the purpose from the code.

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.

Implement a new normalization rule for the updated data format.
2 participants