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

Cleaned file pokemon_species_flavor_text.csv #963

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

AndryCola
Copy link
Contributor

I cleaned this file from broken chars and random new lines.

@Naramsim
Copy link
Member

Hi, can you describe your addition? The diff is too large

@AndryCola
Copy link
Contributor Author

AndryCola commented Nov 24, 2023

praticamente in alcune righe ( come ad esempio nella prima ) il testo non solo ha un carattere che non riesce a visualizzare, ma va anche a campo in mezzo al testo ( vedi foto 1 ) e questo mi ha dato tantissimi problemi per l'import su mysql del csv.
per cui ho rimosso in queste situazioni l'andata a capo e ho rimosso quel carattere codificato male.

in realtà ho notato che nel flavor_text ci sono anche delle virgole e siccome il divisore è appunto la virgola mi ha dato problemi anche su quello, però ho visto che mettendo il punto e virgola la build non funziona per cui sono tornato indietro in tal senso.

<---->
Practically in some lines (such as in the first one) the text not only has a character that it cannot display, but also goes to the field in the middle of the text (see photo 1) and this gave me a lot of problems for importing csv to mysql.

So I removed in these situations the go to the end and removed that poorly coded character.

I actually noticed that in the flavor_text there are also commas and since the divider is precisely the comma it gave me problems on that too, but I saw that putting the semicolon the build does not work so I went back in that direction.
<---->

photo 1:
Screenshot 2023-11-24 alle 09 42 19

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.

2 participants