search

UvA-FNWI / UvA-Workflow-Nbgrader-Canvas

MIT License
0 stars 1 forks source link

nbgrader_config.py #35

Open maartenmarx opened 4 years ago

maartenmarx commented 4 years ago

Het zou fijn zijn als er een voorbeeld nbgrader_config.py in deze repo staat.

In mijn versie zat een kleine fout in de regex voor zip collect. Hier is de juiste regex:

Apply this regular expression to the extracted file filename (absolute path)

c.FileNameCollectorPlugin.named_regexp = ( r'.?(?P\d{7,8})(\d{6})?(?P(Deeltoets|AssignmentWeek|Hertentamen).?).ipynb')

(het haakje rond de filenaam moet NA de .*? staan (die pakt het getal mee).

jaspervanderheide commented 4 years ago

Ja ik zal een voorbeeld er in zetten.

Probleem met jouw oplossing is dat AssignmentWeek1(1).ipynb niet goed gepakt wordt, denk ik (check dit even anders mis je cijfers!!) . (Deeltoets\d|AssignmentWeek\d|Hertentamen) zou misschien wel werken? Dan pak je correct het cijfer mee. Maar ook dat hangt af van de naamgeving, en gaat bijvoorbeeld fout als de toets simpelweg Deeltoets heet.

From: maartenmarx notifications@github.com Sent: Tuesday, November 19, 2019 3:13:48 PM To: UvA-FNWI/UvA-Workflow-Nbgrader-Canvas UvA-Workflow-Nbgrader-Canvas@noreply.github.com Cc: jaspervdh96 jaspervdh96@hotmail.com; Assign assign@noreply.github.com Subject: Re: [UvA-FNWI/UvA-Workflow-Nbgrader-Canvas] nbgrader_config.py (#35)

Assigned #35https://github.com/UvA-FNWI/UvA-Workflow-Nbgrader-Canvas/issues/35 to @jaspervdh96https://github.com/jaspervdh96.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHubhttps://github.com/UvA-FNWI/UvA-Workflow-Nbgrader-Canvas/issues/35?email_source=notifications&email_token=AEWBKZTUNPXFS2VZUY3S6JLQUPYBZA5CNFSM4JPD6WN2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOU6KSJUA#event-2811569360, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEWBKZTCCE65CIYL5U4THB3QUPYBZANCNFSM4JPD6WNQ.

maartenmarx commented 4 years ago

Aah, dus daarom had je dat gedaan ;-) En ik maar handmatig al die klojos hun toetsnaam veranderen.

Ja, jouw oplossing is het beste en netste: (Deeltoets\d|AssignmentWeek\d|Hertentamen)