Skip to content

Remove reducedScoring from PG. #650

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

Closed
wants to merge 1 commit into from

Conversation

somiaj
Copy link
Contributor

@somiaj somiaj commented Feb 5, 2022

As discussed in #648 PG should report raw scores to webwork, and webwork should then adjust the score accordingly. This removes all reducedScoring computations from PG and the variable sub_recorded_score will no longer be set by PG, thus PG will only return the record_score to webwork.

This needs to be merged with openwebwork/webwork2#1578.

  PG should report raw scores to webwork, and webwork should then
  adjust the score accordingly. This removes all reducedScoring
  computations from PG and the variable sub_recorded_score will
  no longer be set by PG, thus PG will only return the record_score
  to webwork..
@drgrice1 drgrice1 force-pushed the remove-reduced-from-pg branch from 54ae10d to ee6fdec Compare February 10, 2022 03:29
@somiaj
Copy link
Contributor Author

somiaj commented Feb 10, 2022

It appears that reduced scoring is also included in some problems inside the OPL. grep -r sub_recorded_score returns various files that are using reduced scoring that will also need to be updated to remove reduced scoring from PG. The file list I get is:

Contrib/CUNY/CityTech/Precalculus/setVectors_-_Magnitude_and_Direction/ggb-magnitude-direction.pg
Contrib/CUNY/CityTech/Precalculus/setVectors_-_Components/ggb-vector-components.pg
Contrib/CUNY/CityTech/Precalculus/setVectors_-_Components/ggb-unit-vector-components.pg
Contrib/CUNY/CityTech/Precalculus/setVectors_-_Unit_Vectors/ggb-magnitude-angle.pg
Contrib/CUNY/CityTech/CollegeAlgebra_Trig/setGeogebra/parabola-roots-multians-incremental.pg
Contrib/CUNY/CityTech/CollegeAlgebra_Trig/setGeogebra/line-intercepts-multians-inc/line-intercepts-multians-inc.pg
Contrib/CUNY/CityTech/CollegeAlgebra_Trig/setGeogebra/circle-center-multians-inc/circle-center-multians-inc-test.pg
Contrib/CUNY/CityTech/CollegeAlgebra_Trig/setGeogebra/circle-center-multians-inc/circle-center-multians-inc.pg
Pending/CUNY/CityTech/Precalculus/setVectors_-_Magnitude_and_Direction/ggb-magnitude-direction.pg
Pending/CUNY/CityTech/Precalculus/setVectors_-_Components/ggb-vector-components.pg
Pending/CUNY/CityTech/Precalculus/setVectors_-_Components/ggb-unit-vector-components.pg
Pending/CUNY/CityTech/Precalculus/setVectors_-_Unit_Vectors/ggb-magnitude-angle.pg
Pending/CUNY/CityTech/CollegeAlgebra_Trig/setGeogebra/parabola-roots-multians-incremental.pg
Pending/CUNY/CityTech/CollegeAlgebra_Trig/setGeogebra/line-intercepts-multians-inc/line-intercepts-multians-inc.pg
Pending/CUNY/CityTech/CollegeAlgebra_Trig/setGeogebra/circle-center-multians-inc/circle-center-multians-inc-test.pg
Pending/CUNY/CityTech/CollegeAlgebra_Trig/setGeogebra/circle-center-multians-inc/circle-center-multians-inc.pg

@drgrice1
Copy link
Member

Those are all @drdrew42's problems. So ask him about those.

@drgrice1
Copy link
Member

It looks like @drdrew42 will just need to delete the assignment to the sub_recorded_score. It shouldn't be needed for his problems. He is also assigning to the recorded_score, which should be enough. @drdrew42 is hacking on pg's internals to make those problems work.

@drdrew42
Copy link
Member

@drgrice1 is correct, assigning values to sub_recorded_score in these problems is not necessary.

@pstaabp
Copy link
Member

pstaabp commented Oct 5, 2022

PR #709 has superseded this by removing the reducedScoring functionality from PG.

@somiaj
Copy link
Contributor Author

somiaj commented Oct 5, 2022

Closing. Outdated and has been superseded by #709.

@somiaj somiaj closed this Oct 5, 2022
@somiaj somiaj deleted the remove-reduced-from-pg branch November 21, 2023 20:36
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.

4 participants