Asset cleanup #62
Open
Asset cleanup #62
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
We used these files when initially creating the new L2s/L3s, but they're now out of date. We've made subsequent changes directly to all_fields.tsv, which is now at the top of our workflow for updating field text. So for clarity I'm removing these assets, because they contain information that's *almost* the same as in field_meta. I'm putting the notebook I used to create an updated version of field_meta.jsonl that incorporates our final decisions from l2_revisions.xlsx in an analysis subdirectory.
This commit deletes some code and files that we originally used to derive our L0-L1 taxonomy from the MAG taxonomy, and find corresponding Wiki text given a large set of field names; we aren't using this now.
No longer used.
rggelles
approved these changes
Jan 31, 2025
There was a problem hiding this comment.
Long-term if this is still in use I think it would be nice to convert this to a real Python file (notebooks can have weird dependency issues that are harder to work out and are more of a pain to re-run) but not a big deal for now.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Remove unused assets for: