Skip to content

Conversation

@drgrice1
Copy link
Member

This script currently does not work well (if at all). This gets the script working again with better help and options.

Note the following changes:

The script no longer saves ids (course id, set id, user id, etc) as useless hashes. I don't know why it was thought that doing so was a good idea.

The script does not gzip the resulting csv file. You can do that on your own if so desired.

The script does not have an upload option (that uploads to wwdata@52.88.32.79). You can upload it to where you want on your own if so desired.

The unix timestamp for the time at which the script is executed is no longer saved in each row of the CSV file. It is part of the default file name if you want it.

Past answers from gateway quiz versions are also dumped. I think it was intended that they would be dumped before, but it wasn't working.

Past answers from problems with more than one answer in them are now dumped. It certainly was intended that this be done before, but wasn't because of a bug in the script.

@drgrice1
Copy link
Member Author

Note that this is already setup to work with #2295, and will also work without that pull request.

@pstaabp
Copy link
Member

pstaabp commented Jan 24, 2024

Looks good. I notice, this doesn't dump the reduced_scoring_date. I know that doesn't always make sense if the course doesn't have it, but maybe should be included for courses that do.

@drgrice1
Copy link
Member Author

drgrice1 commented Jan 24, 2024

Yeah, the script did not include the reduced scoring date previously, and I didn't add it. An issue with adding that in the region of the csv the dates are in is that it will change the column numbers of the data after it making the format more variable. That will also change the format if someone has been using the data from this script previously with reduced scoring enabled.

@drgrice1
Copy link
Member Author

I added the reduced scoring date to the dumped data. It is always in the data even if it is not defined. In that case the column for it in that row will be empty in the csv file.

@drgrice1 drgrice1 force-pushed the dump_past_answers-rewrite branch 2 times, most recently from fa781ac to c110f64 Compare January 24, 2024 22:30
@drgrice1 drgrice1 force-pushed the dump_past_answers-rewrite branch from c110f64 to d85f2ab Compare February 2, 2024 20:49
@drgrice1 drgrice1 force-pushed the dump_past_answers-rewrite branch 4 times, most recently from 52e4eff to 55e0790 Compare February 13, 2024 12:01
@drgrice1 drgrice1 force-pushed the dump_past_answers-rewrite branch from 55e0790 to 43c343f Compare February 14, 2024 23:10
@drgrice1 drgrice1 force-pushed the dump_past_answers-rewrite branch from 43c343f to 54def09 Compare February 25, 2024 12:14
The script no longer saves ids (course id, set id, user id, etc)  as
useless hashes.  I don't know why it was thought that doing so was a
good idea.

The script does not gzip the resulting csv file.  You can do that on
your own if so desired.

The script does not have an upload option.  You can upload it to where
you want on your own if so desired.

The unix timestamp for the time at which the script is executed is no
longer saved in each row of the CSV file.  It is part of the default
file name if you want it.

Past answers from gateway quiz versions are also dumped.  I think it was
intended that they would be dumped before, but it wasn't working.

Past answers from problems with more than one answer in them are now
dumped.  It certainly was intended that this be done before, but wasn't
because of a bug in the script.
of assuming the admin course is named 'admin'.  It falls back to using
'admin' if that variable is undefined.
@drgrice1 drgrice1 force-pushed the dump_past_answers-rewrite branch from 54def09 to 2f988f6 Compare February 29, 2024 13:50
@Alex-Jordan Alex-Jordan merged commit 57b9e31 into openwebwork:develop Feb 29, 2024
@drgrice1 drgrice1 deleted the dump_past_answers-rewrite branch February 29, 2024 21:25
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.

3 participants