Webwork Script - Modify output for PL script

[Parsa-Rajabi]

There are currently some errors when PL script is ran on the .md files:

There is an error in this problem:
    - File path: source_ww/output_md/College_Physics_Urone/5.Friction_Drag_and_Elasticity/5-01.Friction/NU_U17_05_01_002.md
    - Error: string indices must be integers
source_ww/output_md/College_Physics_Urone/5.Friction_Drag_and_Elasticity/5-01.Friction/NU_U17_05_01_006.md
Check the heading levels, is there one that doesn't belong? Or is the heading level incorrect? For e.g., it should be ### Answer Section (this is not necessarily where the issue is).
There is an error in this problem:
    - File path: source_ww/output_md/College_Physics_Urone/5.Friction_Drag_and_Elasticity/5-01.Friction/NU_U17_05_01_006.md
    - Error: list index out of range
source_ww/output_md/College_Physics_Urone/5.Friction_Drag_and_Elasticity/5-01.Friction/NU_U17_05_01_016.md
Check the heading levels, is there one that doesn't belong? Or is the heading level incorrect? For e.g., it should be ### Answer Section (this is not necessarily where the issue is).

Errors:

• string indices must be integers
• list index out of range

Look into the root cause of the errors and resolve them

[firasm]

I just did some quick hacking and I think I see the issue:

In the file NU_U17-33-02-002.md, the level two heading doesn't look formatted correctly:

# {{ params.vars.title }}## Part 1 
The ratio of the strong to the weak force and the ratio of the strong force to theelectromagnetic force become (1) under circumstances where they are unified. Whatare the ratios of the strong force to those two forces under normal circumstances? 

It should be:

# {{ params.vars.title }}

## Part 1 
The ratio of the strong to the weak force and the ratio of the strong force to theelectromagnetic force become (1) under circumstances where they are unified. Whatare the ratios of the strong force to those two forces under normal circumstances? 

[firasm]

Also in the same file I can see in Lines 39 and 40:

      # Randomize Variables
      $SE = 1/(10**-2);
      $SW = 1/(10**-13);

I think those $ aren't valid python names

We also probably should replace all ** to E

[firasm]

Ok I think the largest issue is the YAML export.

Remember that |- means that "everything after it should be interpreted as a multi-line string" so we only want that for things like imports and generate.

If we do it for server, it will not parse the sub-levels as separate dictionary keys:

So we need to get rid of |- in server and in the parts:

[firasm]

TBD needs to be commented out as anything in generate will get run

[firasm]

One other oddity: it looks like the YAML section starts with part2 rather than part1

[firasm]

For the problem type, type: Numerical should be: type: number-input

[firasm]

After I fix those relatively minor issues, IT WORKS!!!

I'm so excited! 🎉

[firasm]

P.S. I've logged all the changes I had to make to the md file in this commit

[Parsa-Rajabi]

I just did some quick hacking and I think I see the issue:

In the file NU_U17-33-02-002.md, the level two heading doesn't look formatted correctly:

# {{ params.vars.title }}## Part 1 
The ratio of the strong to the weak force and the ratio of the strong force to theelectromagnetic force become (1) under circumstances where they are unified. Whatare the ratios of the strong force to those two forces under normal circumstances? 

It should be:

# {{ params.vars.title }}

## Part 1 
The ratio of the strong to the weak force and the ratio of the strong force to theelectromagnetic force become (1) under circumstances where they are unified. Whatare the ratios of the strong force to those two forces under normal circumstances? 

Fixed

[Parsa-Rajabi]

For the problem type, type: Numerical should be: type: number-input

Fixed

[Parsa-Rajabi]

Also in the same file I can see in Lines 39 and 40:

      # Randomize Variables
      $SE = 1/(10**-2);
      $SW = 1/(10**-13);

I think those $ aren't valid python names

We also probably should replace all ** to E

Fixed. Removed $ and replaced ** with E

[Parsa-Rajabi]

TBD needs to be commented out as anything in generate will get run

already completed

[Parsa-Rajabi]

Sever in YAML

Ok I think the largest issue is the YAML export.

Remember that |- means that "everything after it should be interpreted as a multi-line string" so we only want that for things like imports and generate.

If we do it for server, it will not parse the sub-levels as separate dictionary keys:

• Addressed all server items except generate: https://github.com/open-resources/scripts/commit/81a4c31b883eea2ce48bed79106a038415db6abf
• Still need to identify how and why generate does not output results as intended. Completed via: https://github.com/open-resources/scripts/commit/179d7d64467e8b988648bf311897353a81256a55

[Parsa-Rajabi]

Part Subsections

So we need to get rid of |- in server and in the parts:

One other oddity: it looks like the YAML section starts with part2 rather than part1

• Still need to identify the root cause of part#s and ensure |- is removed from them
• Fixed part numbering via https://github.com/open-resources/scripts/commit/179d7d64467e8b988648bf311897353a81256a55
• Still looking into |-

[Parsa-Rajabi]

All items are complete now!

Latest changes include: https://github.com/open-resources/scripts/commit/67ae7f02e2cb2b2e98d00d9e0a7262e752183ddc

• Each question has its own folder
• All question images are being copied to the question folder for PL
• Part # issue has been fixed