MarkUs Blog

MarkUs Developers Blog About Their Project

Archive for February, 2013

Punchlines – UCOSP Winter 2013 – Week 6

without comments

Marc

Status:

  • Completed 1 major folder in the refactoring of Prototype over to jQuery (students folder). Other progress can be seen here

Roadblocks:

  • Existing AJAX call structure is confusing when trying to integrate with jQuery. I do not quite understand how the RJS files interact with the AJAX calls made in our application. Will have to talk to prior team to understand what is going on or talk to Daryn.

Next week:

  • Meet up with Daryn for a hopefully efficient code sprint working out any problems encountered thus far. Keep working on refactoring.

Ian

Status:

  • Made it so adding a test script will make a criterion for it with the same name as the script. I also got the marks from running the test script to be applied to the criteria.

Roadblocks:

  • There’s some dependencies to fix up. (eg. if a test script is removed then it should delete the associated criterion)

Next week:

  • Continue implementation of test generated marks.

Daryn

Status:

  • Currently working on .rjs file compatibility with jQuery
  • Pending solution: take jquery-ujs’s rails.js, which is an unobtrusive adapter for jQuery to rails and works with jQuery to use files such as .rjs type which supposedly only work with prototype.js

Roadblocks:

  • Resolve .rjs type file usage
  • Testing changes

Next week:

  •  Will meet up with Marc to resolve issues on Friday

Nick

Status:

  •  Finished working on Config UI.

Roadblocks:

  • Found an issue: If you upload a test script, run a test on a student, then delete that test script, you can’t view the test results anymore.

Next week:

  •  Start working on test results in grading interface.

Alysha

Status:

  • Tested new fix for issue 941. Updated blueprints for schema change. From my testing, all functionality is the same as pre-schema change. Unit and functional tests do not fail on any tests that were not failing before the change.

Roadblocks:

  • Waiting for Mike’s pull request to go through so I can fix conflicts before creating pull request for my issue.

Next week:

  • Write blog post about database schema change. Log and start working on bugs found.

Mike Stewart

Status:

  • Working on setting up the grade entry graders tab. Have got the UI started, and now working on functionality.

Roadblocks:

  • None.

Next week:

  • Travelling next week over March Break, but hope to get more of the Graders tab working.

Oussama

Status:

  • Working on modifying grade entry forms so that only records for students assigned to a certain grader are displayed, currently, graders can see all students by default
  • contemplating two design choices:
    1) add an extra column in the table that shows the group name the student is in
    2) add a drop-down menu above the table that displays the relevant groups of students assigned to the current grader, so that graders may display one group of students at a time

Roadblocks:

  • None so far

Next week:

  •  Continue working on above-mentioned feature
  • Will have to coordinate closely with Mike-Stewart during the development of this feature since he is working on giving admins the ability of assigning graders to groups of students

Mike Wu

Status:

  • Submitted pull request #1011 for issue #938.
  • Decided to work on the UI side of the remark feature after discussion with Alysha.
  • Began disussion with Karen regarding the UI for remark.

Roadblocks:

  • Would like to have someone review pull request #1011 as Alysha is waiting on this change. Thanks in advance!

Next week:

  • Start working on the UI for remark.

Daniel

Status:

Roadblocks:

Next week:

  • Continue work on assignments, keep looking into extending the response functionality.

Written by Daniel St. Jules

February 26th, 2013 at 9:59 pm

Posted in Status Reports

Punchlines – UCOSP Winter 2013 – Week 5

without comments

Ian

Status:

  • Got functional tests working with a few failure that are consistent with failures in the master branch. Also reviewed the current architecture for rubrics and testing.

Roadblocks:

  • The design of how the tests will mark assignments needs to be discussed.

Next week:

  • Start working on getting automated testing to mark assignments.

 

Nick

Status:

  • Continue working on Config UI page. It’s taking longer than I thought it would.

Roadblocks:

  • The reason is that there’s code using Prototype and I have to convert it to JQuery. I have to rewrite some of the code that was written last term.

Next week:

  • Finish Config UI.

Mike Wu

Status:

  • – Added remark request information on dashboard (#938), putting some finishing touches before committing.

Roadblocks:

  • Not clear whether Alysha’s changes to database schema (#941) would affect my implementation of #938.

Next week:

  • Submit pull request for #938.
  • Plan remark features with Alysha and Karen.

 

Alysha

Status:

  • Testing fix to issue 941.
  • Discovered some problems with the database schema change, primarily from changing to methods that could be returning either the original result or remark result (since methods were based on “latest” results).
  • After discussing with Karen, decided to change the approach to database schema change.

Roadblocks:

  • Since the remark feature was added much later in the development, a lot of areas in the code are still using the original result even though a remark result should be used if it exists.

Next week:

  • Change methods for new database schema to get_original_result and get_remark_result to provide a hopefully less complicated switch from the current implementation.
  • Test.
  • Write a blog post followup.
  • Log issues found throughout testing.

Daniel

Status:

Roadblocks:

  • None.

Next week:

  • Finish up /api/users and basic collection functionality; get filters working as well.

Daryn

Status:

  • Decided to work without prototype.js so we don’t have to work with jQuery.noConflict()
  • Also, by doing so easily determines which pages were dependant on prototype.js since errors would be raised.

Roadblocks:

  • None.

Next week:

  • Continue to cover the view files for any prototype.js dependancies.

 

Mike Stewart

Status:

  • Continuing to work on getting a graders tab set up for grade entry.

Roadblocks:

  • None

Next week:

  • Get a basic graders tab implemented for grade entry, including all views components and basic functionality.

Written by Ian Smith

February 20th, 2013 at 1:01 pm

Posted in Status Reports

Punchlines – ECN PAPPL 2013 OptimMarkus – 2013-02-18

without comments

Claire

  • was assigned to issue #1007
  • worked on issue #1007: examined existing code in order to find the methods needed for this feature

Gaëtan

  • reproduced issue #559 and added a more detailed description – as a comment
  • got started developing to implement the required feature
  • learned how to use the debugger

 

Written by G_Girin

February 18th, 2013 at 4:40 pm

Posted in Developer Essentials

Tagged with , ,

Punchlines – UCOSP Winter 2013 – Week 4

without comments

Marc

Status:

  • Continuing to work on the change over from Prototype to jQuery. Got jQuery, jQuery UI, jQuery Browser Detection implemented. Completed login screen to use a jQuery UI modal.

Roadblocks:

  • None so far.

Next week:

  • Keep working on the changes over to jQuery. Now that I have the login page done, I will start work on the main application and will be working alongside Daryn to make sure we are following the same code conventions.

Ian

Status:

  • fixed majority of test issues. Down from ~130 to ~20.

Roadblocks:

  • Still some issues with the FlexibleCrtieriaController.

Next week:

  • Fix up the rest of the tests.

Daryn

Status:

  • Completing coverage of Dashboard/homepage migrations to jQuery

Roadblocks:

  • None

Next week:

  • Continue the grind of migrations. Going over Views first.

Nick

Status:

  •  Started working on a new Config UI

Roadblocks:

  •  My CSS and Javascript is a bit rusty, but no other roadblocks.

Next week:

  •  Finish working on Config UI.

Alysha

Status:

  • Continuing to work on issue 941. Tested application to look for any bugs caused by schema change. Worked on updating fix so that functionality mimics pre-schema change. Trying to also familiarize myself with the unit/functional tests since they need to be updated with the schema change.

Roadblocks:

  • Realized some issues from swapping all “submission.result” references with new method “submission.get_result”. The former always returned the original result whereas the latter could return either the original or remark (since the new method returns the newest result). This changes some functionality/causes some bugs and needs to be rechecked.

Next week:

  • Recheck affected areas to verify where differences in functionality/bugs are coming from and fix.
  • Need to look into how blueprints work to update unit/functional test for new schema.

Mike Stewart

Status:

  •  Discussed grade entry requirements with Oussama.
  • Began implementing Graders tab view using reused code from assignments.

Roadblocks:

  • Need to touch base with Karen to confirm requirements.

Next week:

  • Get Graders tab view set up.

Oussama

Status:

  • Examined existing grade entry functionality and code
  • Found that large parts of the UI code can be recycled
  • Still unclear about some issues regarding the nature of the work flow, since grade entry sheets exist already
  • Connected with Mike

Roadblocks:

  • None

Next week:

  •  Need to talk with Karen about a few design decisions

Mike Wu

Status:

  • Started on issue 938, but couldn’t go very far due to midterm preparations and Lunar New Year activities. (Happy New Year everyone! =])

Roadblocks:

  • Pull 998 (adding assignment statistics to dashboard) is waiting for code review.

Next week:

  • Continue with issue 938.
  • Discuss with Alysha and Karen regarding developing remark feature.

Daniel

Status:

  • Continued work on issue 1002 (https://github.com/MarkUsProject/Markus/issues/1002). Was able to test the existing API, and proposed re-organization and additional features. Completed a list of the API routes I’d like to complete this term, and included details of common parameters and other points of interest. Began re-organizing the /api/users controller.

Roadblocks:

  • Hoping for input/criticism on the proposed design. Ideas for other features as well. Also, I’m trying to figure out the best way to allow filters in the URI for collections.

Next week:

  • Continue work on /api/users and create/document a rough model for error handling in the API.

Written by Alysha Kwok

February 13th, 2013 at 3:23 am

Posted in Status Reports

Punchlines – ECN PAPPL 2013 OptimMarkus – 2013-02-11

without comments

Claire

Gaëtan

 

Written by G_Girin

February 12th, 2013 at 12:25 pm

Posted in Developer Essentials

Tagged with , ,

Punchlines – UCOSP Winter 2013 – Week 3

without comments

Mike Wu

Status
– Submitted a pull request #998(https://github.com/MarkUsProject/Markus/pull/998), followed by two additional commits: updating French translations and creating a migration.
– Started looking at #938(https://github.com/MarkUsProject/Markus/issues/938).

Roadblocks
– Learned how to create a migration for changing database schema in #915(https://github.com/MarkUsProject/Markus/issues/915).

Next week
– Continue working on #938(https://github.com/MarkUsProject/Markus/issues/938).
– Revise pull #998(https://github.com/MarkUsProject/Markus/pull/998) as needed.

Alysha

Status
– Continuing to work on issue 941(https://github.com/MarkUsProject/Markus/issues/941). Located and switched out all instances of submission.result/submission.remark_result for the appropriate method. Resolved majority of issues that occurred after changing the schema.

Roadblocks
– After changing the submission schema, a foreign key is no longer being set for the remark_result_id column in the Submissions table. I sent an email to the developers list to gain some insight. Working on determining the best way to approach this. Results are used in many aspects of the application. I need to thoroughly test the application to ensure all areas that were affected have been updated.

Next week
– Determine a fix to populate the remark_result_id in the database with the new schema. Test application for any errors that may have been overlooked. Write a blog post to follow-up on this post(http://blog.markusproject.org/?p=4614) which was about the original issue and the temporary fix that I am working on replacing.

Marc

Status
– Started work on converting Prototype to jQuery. Keeping track of my changes as I go along.

Roadblocks
– None so far.

Next week
– Keep working on the changes over to jQuery. Will post a blog post as well as documenting the changes I have done so far so other members on the MarkUs team can follow along.

Ian

Status
– Configured Markus to use a public key for the SSH calls.
– Discussed the current state of the test framework with Brian and Mina.
They referred Nick and I to a Google doc
(https://docs.google.com/document/d/1IFfGQ3VIYq8p2ZSy13GJPkOAVoaAw-zRcrQOj9LyKuQ/edit). Read the document to get an idea of what needs to get done.

Roadblocks
– Getting tests to mark based on the rubric is difficult and
needs some design work.

Next Week
– Try to fix functional tests for the test framework. There’s currently over 130 failing.

Daryn

Status
– Setup branch with Marc to begin collaborating on migration.
Roadblock
– Lost the weekend to the MHacks and a test today (tues.)
Next Week
– Continuation of jQuery migration.

Nick

Status
– This week was a bit busy for me so I didn’t get to do as much as I
wanted. I mostly just did reading and planning (not much coding).

– Received and read an extremely helpful documentation about the MarkUs Automated Test Engine from last term’s developers:

https://docs.google.com/document/d/1IFfGQ3VIYq8p2ZSy13GJPkOAVoaAw-zRcrQOj9LyKuQ/edit#heading=h.b00omv8gqmm1

Thanks a lot guys!

– Removed a stray comma in this one file, and made a pull request. Avery minor typo, but with the comma there the Config UI won’t load at all.

– Decided that I’d work on UI.

Roadblock
– The document might be a bit out of date. There are several “UPDATE”
remarks saying that certain features should be implemented. But those
features are already implemented (which isn’t a bad thing).

My concern is that there are several screenshots in the document, and
I was planning to just follow them when working on the UI. But now I’m
worried those screenshots may be from old design plans.

Next week
– Start working on the Config UI. The bullet points on page 13 look good
to me and I’d like to start implementing some of them.

Mike Stewart

Status
– Going through the Graders tab under assignments to see how much of it can be used for the grade entry form.
Roadblocks
– Need to connect with Oussama to create a plan so that we don’t duplicate work.
Next week
– Start setting up graders for the grade entry form.

Oussama

Status
– working on giving TA’s the ability to enter grades
– reviewing relevant existing code under the grader’s tab

Roadblocks
– Had to submit multiple assignments and write a term test this past week

Next
– Attempt to recycle parts of the code as the functionality already exists for a different user role
– Plan possible changes to database schema

Daniel

Status
– Continued work on issue 19, though now that I’ve realized notifications are occasionally displayed at different levels in the page, I may have to re-think the solution.
– Read up on RESTful API design
– Tested and reviewed the code of the current Markus API
– Started a discussion and analysis of the current API via issue 1012

Roadblocks
– I could not get /api/submission_downloads working yet – I keep getting a 422 response despite having what would seem to be a valid request.

Next
– Update the documentation for the existing API
– Discuss with Karen what other new features, aside from gradebook entry, she’s like to see in the API

 

Written by Marc Bodmer

February 6th, 2013 at 9:26 am