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

Check and document import warnings #25

Open
griff-rees opened this issue Jul 6, 2023 · 1 comment
Open

Check and document import warnings #25

griff-rees opened this issue Jul 6, 2023 · 1 comment
Labels
documentation Improvements or additions to documentation

Comments

@griff-rees
Copy link
Contributor

griff-rees commented Jul 6, 2023

Successful runs of create_adjacent_tables.py yields warnings of duplicates. Documenting and checking these warnings would help address potential issues with correctness.

@griff-rees griff-rees added the documentation Improvements or additions to documentation label Jul 6, 2023
@griff-rees
Copy link
Contributor Author

Example warnings:

...
Warning: 2 newspapers found with NLP 0000174 -- keeping first
Warning: 2 newspapers found with NLP 0000174 -- keeping first
Warning: 2 newspapers found with NLP 0000174 -- keeping first
Warning: 2 newspapers found with NLP 0000174 -- keeping first
Warning: 2 newspapers found with NLP 0000174 -- keeping first
Warning: 2 newspapers found with NLP 0000174 -- keeping first
Warning: 2 newspapers found with NLP 0000174 -- keeping first
Warning: 2 newspapers found with NLP 0000174 -- keeping first
Warning: 2 newspapers found with NLP 0000174 -- keeping first
Warning: 2 newspapers found with NLP 0000174 -- keeping first
Warning: 2 newspapers found with NLP 0000174 -- keeping first
Warning: 2 newspapers found with NLP 0000174 -- keeping first
Warning: Could not find NLP 0000373 in newspaper data
Warning: 2 newspapers found with NLP 0000153 -- keeping first
Warning: 2 newspapers found with NLP 0000153 -- keeping first
Warning: 2 newspapers found with NLP 0000153 -- keeping first
Warning: 2 newspapers found with NLP 0000153 -- keeping first
Warning: 2 newspapers found with NLP 0000153 -- keeping first
Warning: 2 newspapers found with NLP 0000153 -- keeping first
Warning: 2 newspapers found with NLP 0000153 -- keeping first
Warning: 2 newspapers found with NLP 0000153 -- keeping first
Warning: 2 newspapers found with NLP 0000153 -- keeping first
Warning: 2 newspapers found with NLP 0000153 -- keeping first
Warning: 2 newspapers found with NLP 0000153 -- keeping first
Warning: 2 newspapers found with NLP 0000153 -- keeping first
Warning: Could not find NLP 0000668 in newspaper data
Warning: Could not find NLP 0000669 in newspaper data
Warning: 2 newspapers found with NLP 0000070 -- keeping first
Warning: 2 newspapers found with NLP 0002642 -- keeping first
Warning: 2 newspapers found with NLP 0002642 -- keeping first
Warning: 2 newspapers found with NLP 0002642 -- keeping first
...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

1 participant