Merging Common Cartridge Export (LTI 1.1) into develop/nutmeg.master. #239
Open
mystica-l wants to merge 14 commits into
Open
Merging Common Cartridge Export (LTI 1.1) into develop/nutmeg.master. #239mystica-l wants to merge 14 commits into
mystica-l wants to merge 14 commits into
Conversation
Added --cc-lti flag that redirects from xml_exporter to imscc_exporter in export_olx.py, new file created to start implementing imscc function, starting with basic iterator for sequentials
…ent_settings.xml file next steps, work on creating identifiers and linking them, look at trying to extract points, clean up code, remove unecessary functions for original openedx exporter in imscc_exporter.py
…e new TestExportManager class All the necessary work for imscc was merged together into one class to keep things organized and concise, additionally, moved code to create assignment_setting.xml files into a function separate from the primary export command
ztraboo
force-pushed
the
feature.nutmeg/mystica-l/common-cartridge-export
branch
from
f471379 to
caad21d
Compare
…ourse_settings folder Removed a lot of the hard coded assignment parameters for the xml, found most were not necessary
Need to update imsmanifest_xml to take the new identifiers as assignment folders are built out first
…xternal tool xml file export Next steps include finishing out module_meta.xml file, probably one of the more challenging files to create
…ncorrect schema values Next, need to incorporate module headers Also fixed a lot of typos and added more detailed documentation
…roups is iterable Added safeguard that prevents passing multiple courses without cc-lti flag
assignment folders imsmanifest and module_meta now generates using multiple courses, changed identifier dictionary keys of sequentials and chapters to use a serialized object with custom hash function Update for use of a new serialized object for chapters and sequentials due a weird issue where iterating through the same list of courselike keys in a different function led to slightly different chapters and sequentials causing for non-consistent keys, maybe due to memory of the objects? Naming conventions were also added such as adding the tag at the end of the course key to the beginning of modules, etc.
Need to check if this system also works with qualtrics surveys Update: Works with qualtrics surveys
…ted new export_imscc file
ztraboo
force-pushed
the
feature.nutmeg/mystica-l/common-cartridge-export
branch
from
caad21d to
b510d8b
Compare
ztraboo
force-pushed
the
feature.nutmeg/mystica-l/common-cartridge-export
branch
from
b510d8b to
2f95a01
Compare
ztraboo
reviewed
Mar 20, 2025
| # re pattern for extracting the values between the plus signs | ||
| between_pluses = r'(?<=\+)(.*?)(?=\+)' | ||
| # re pattern for the 'FAA-ACS-AM-IA-ACE' that all courses seem to have | ||
| course_type = r'([A-Za-z]{3}-[A-Za-z]{3}-[A-Za-z]{2}-[A-Za-z]{2}-[A-Za-z]{3})' |
There was a problem hiding this comment.
@mystica-l
This only works for things like FAA-ACS-AM-IB-ACD for example. It will not work for EV-ST-IEV or other things like SFT108. I'm going to remove this check and we'll just use the full Course Number for now. At the moment, there is no consistency in how we name these values.
ztraboo
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Common cartridge export feature accessed by usage of export_imscc management command.
Able to export multiple courses at once by having their course codes follow one after another.
Able to export as external tools only for LMS platforms that don't use assignments using --external-tool-only flag.