Simon-Initiative / course-digest

Tool to produce a summary or digest of OLI course package contents
MIT License
2 stars 0 forks source link

[PERFORMANCE] Performance and usage optimizations #91

Closed darrensiegel closed 2 years ago

darrensiegel commented 2 years ago

This PR does two things:

  1. Moves the "file existence verification" step outside of loop that visits every reference in an org. This was causing the course digest to take several minutes to run when faced with a large course that has many organizations.
  2. Simplifies the command line operation, supplying useful default to all but the inputDir. Now, to invoke the "convert" operation against a course, you can simply do:
npm run start -- --operation convert --inputDir /Users/darrensiegel/dev/courses/chemistry/chem1/trunk