Update Review Data Parsing Scripts #676
Merged
+39
−6
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.
While uploading the review data from semesters 2024A/B, I ran into a few errors due to some new formatting. The changes are summarized below:
RWORKREQUIRED
. When we create ReviewBit objects, we want to ensure that there is at least some data present that we are saving. The assumption is that if we are not given a value for these fields in the summary dump, there are no values to be saved.val is None
, but our parser is not catching these values, so we want to also check for the string"null"
.DD-MON-RR
, which required parser support. TheRR
year format isn't supported directly through Python'sdatetime
module, so we added an additional function to find the four digit year it corresponds with. More information on this conversion here.