dragoon-lab / laitsv3 Goto Github PK

Open bookworm problem (CPI-2014-ps1-01, section lms-CPI_fall_2017) in student mode using login.html.

Bug 1:

1. Create the accumulator, entering books and books * 2 to get the eqn wrong twice.
2. Create the parameter node.
3. Delete the parameter node.
4. Try creating the parameter node again. The description is disabled.

Bug 2 (start over using new user or restart problem):

1. Click create node, select the parameter description (weekly books bought)
2. Delete the parameter node.
3. Create the accumulator, entering books and books * 2 to get the eqn wrong twice.
4. Try creating the parameter node again. The description is disabled.

@jonwetzel commented on Wed Aug 09 2017

As you can see in this image, the tabs in the graph window have no styling--they are just text (i.e. "Graph" "Table"):

At one point in Dragoon's history, we had styled tabs but they were broken for a long time before that. I believe this might have been the commit that first fixed the tabs:
890aad1

Line 6 of index.html adds this line <link rel="stylesheet" href="./dijit/themes/claro/claro.css">

But somehow this commit got taken out of the history of index.html...maybe a bad merge or someone rebased by mistake at some point. The tab styling has been broken ever since. How about we try adding this line back and see if it works?

@jonwetzel commented on Thu Sep 07 2017

Reassigning to @dharanisakary

Steps:

1. Open rabbits in student mode
2. Create the accumulator ("population") and get the equation wrong twice. (E.g. "death rate" -> "- death rate") You should get a yellow net growth as expected.
3. Click done, then delete all nodes except the accumulator. When you delete "Net Growth", check the console--4 or 5 Assertion Failed errors appear related to that node id. (maybe this causes the bug?)
4. Now delete the accumulator.
5. Create the accumulator for "population" again. Enter "net growth" and click Check expression.
   The equation gets filled in with "net growth" and turns yellow, but the net growth node is not created. (The effect of the BUG)
6. Re-open the accumulator to see the blank equation in yellow state. The user is now stuck.

This is broken since we created secure URL. Create a new hidden form which will pass the necessary parameters and open a history problem.

Make sure user can always close error popups.

Seems inconsistent.

Currenlty, we have % actions correct; this would be % nodes correct. This allows a rough grading of models .

The following changes from Benjamin Nye should be done on dragoon.asu.edu/PAL3.

API Update for Working With External Active Resources (e.g., Dragoon): Based on some new needs for PAL3, we had to do some small changes to the API. The most substantial change was that we now have multiple backend servers for different purposes (e.g., a main one, a dev, one for just small studies/prototying). Currently, Dragoon doesn't know anything about that and just sends its scores to the main PAL3 server, regardless of which one invokes Dragoon.

We have changed the messaging slightly to pass this information so that Dragoon could respond back properly. Do you think that you would be able to do the minor changes that would be needed to have this message properly? We still occasionally demo Dragoon in PAL3 to Navy folks and others and highlight your system and what it does, but it gets tricky because we can only do it on PAL3 builds that point to a certain server.

I'm pasting the notes about the new API. You've already got the basic messaging in place, so this would just be some small edits to the existing REST call (nothing huge, and Adam can support on this):

For URL activities, we send data via URL parameters. For the data to PAL3, we receive it by a REST endpoint. Right now we are still relying on a lightweight version that is easier to deal with, though we're likely to replace this with official xAPI stuff down the road.

SENT VIA URL PARAMS: When a resource is started from pal, it is sent the following data:

• player_id
• player_name
• resource_guid
• resource_session_id (the specific instance of this resource for this player)*
• database_url (where the data will be stored)*

RECEIVED VIA REST: Pal Receives, When a resource is completed, pal receives the following data

• score
• player_id
• resource_guid
• resource_session_id*
• duration (how long it took to complete)*
  Optional:
• Pair<knowledge component id, score> (these scores override the previously listed score)*
  (you can ignore the KC: score optional one, unless you are giving different scores for different KC's)

So in short, we send everything at we need back, except for the duration and the score. You can send us the specific KC's that you're using (though we'd need to make sure you know our ID's for them), but the easiest approach right now is that we register the resources with our ID's.

Dharani: document what steps happened leading up to the bug on this card.
Sachin: Add the steps you found last year as well.

If we can figure out how to reproduce it we'll fix it.

There might be work to do on the LMS side too. Let me know if that's the case.

Use branch ET-xapi

• Disable the code for ETConnector.js. We'll delete it in a future task.
• Add the two .js files https://github.com/andrewtackett/ET_Recommender/tree/master/hostedResources/lib to our code base. These are the libraries we need to communicate with the LRS.
• Create a config file to somehow pass in the authentication information. List it on .gitignore and never upload it to github.
• Write a function that takes in the problem name and gives back the Learning_Resource_Name from the big spreadsheet.
• Add code to call sendKCScore when the user clicks "done". Do this ONLY if the server path equals "ET". (See the pal3 code which uses tincan.js for example of how to do this.)
  • First argument should be "DRAGOON",
  • The second should be the LearningResourceName from the spreadsheet (need new code to generate this string)
  • The third will be the dictionary we used to send in tincan.js or ETConnector--communicate with Andrew to finalize this.
  • More details on these arguments can be found in the email thread.

• Make it a dojo module for use in history-widget
• Make sure it works with index.html
• Make usre it works with authordemo.html

Next to each slider, add an "Update node" button which sets the value of that node to the selected value, then re-renders the graph and resets the slider.

Points at the maximum value of the graph range do not render. This will make it easier to see a line which runs along the top of the graph.

It also seems that a graph consisting of a straight line does not render. (Make an accumulator with initial value of 1 and add 0 to it every cycle.) Maybe this is related?

1. Open moose population 1 problem in student mode.
2. Fill in the parameter node with the correct answer (2)

The value field should turn green, but it turns grey.

Click "Help" then "Math Functions" and allow popup to see the list.

These should either be grouped logically by family or just sorted alphabetically.

1. Make an accumulator node, foo, with initial value 0
2. Change its function to sin(bar)
3. Make bar some parameter.
4. Change the equation of the accumulator to bar and click check expression. BUG: no plus sign appears in the accumulator
5. Change the equation to -bar, click Done. Now the plus sign appears and a - icon appears on the arrow from bar to foo (correct).

Various editing causes the icons to be missing when they should be there.

In SoS 326. Jon will discuss with Kurt at backlog update.

We got a json file of PAL3 KCs, some of which are not in our problems. However, we don't know if they're expecting them. We'll keep this issue open till they complain or tell us it's working fine. If we need to make changes, we'll add them to a check list here:

Below is a "diff" @dharanisakary created so we know which schemas are missing relative to each list.

These are the KCs which PAL3 expects, but are not a part of our schemas -
Power Supply Function,
Invalid KC Entry,
Basic Capacitor Filter Behavior,
AutoTutor Interactions,
CE Transistor Physics,
CE Transistor Function,
CE Transistor Fixed Bias Behavior,
Zener Diode Structure,
Navy General Information,
UAV Navigation,
FC A-School Integration,
FC E-3 Rank Achieved,
Electro Optical Sensor System Function,
Generic.

In these KCs, Invalid KC Entry is a placeholder for unknown KC values, Generic is used When a resource isn't associated with any knowledge component. Got to know these form descriptions given in PAL3's json.

Following are the KCs present in schema class PAL3 in our schemas, but not used in PAL3

Clamper Behavior,
Limitor Behavior,
Bridge Rectifier with Zener Behavior,
Resistor Series Behavior,
Resistor Parallel Behavior,
CE Transistor AC Behavior,
CE Transistor DC Behavior,
CE Transistor Fixed Bias DC Behavior,
CE Transistor Fixed Bias AC Behavior,
CE Transistor Voltage Divider Bias DC Behavior,
CE Transistor Voltage Divider Bias AC Behavior,
CB Transistor Amplifier DC Behavior,
CB Transistor Amplifier AC Behavior,
CC Transistor Amplifier DC Behavior,
CC Transistor Amplifier AC Behavior,
CE Transistor Amplifier DC Behavior,
CE Transistor Amplifier AC Behavior,
CE Push Pull Amplifier DC Behavior,
CE Push Pull Amplifier AC Behavior,
CC Push Pull Amplifier DC Behavior,
CC Push Pull Amplifier AC Behavior,
RC Coupled Multistage Amplifier DC Behavior,
RC Coupled Multistage Amplifier AC Behavior,

1. Set accumulator to moose births - moose deaths.
2. click check expression
3. click done

The new node is created but the links are not. It is

User using chrome for mac version 68.0.3440.106

• Display the node's name in that color.
• In the graph window, show that color in the variable name

In the Graph vs Parameter tab of the graph window, the user can type in the dropdown box used to select the x-axis variable. Change it from a combo box to a plain dropdown so the user can't type in it anymore.

• 404 on "Introduction Webpage" -- remove this option
• Rename "Introductory Video" to "Electronics Tutorial Video"

Ignacio writes:

As part of the class activities I asked the students to briefly comment on their experience with Dragoon and to suggest some ideas for improvement. In general I have to say that students are very happy with the software, and most of the have had little problems in using it.

Attachment: Dragoon Improvement Suggestions from Students.docx

Discovered by Dharani. Example which breaks current system:

Two functions:
a = 15
b = a + 10

• Static tab should just show a message "There are no parameters to graph the unknowns against."
• Rename tabs "Graph vs Time" and "Graph vs Parameter"

Remove the whole sentence which Ignacio underlined in his email.

The teacher who first creates the class is the owner.

The class owner can add other teachers to the class as teachers by entering their email address or username.

Once a teacher is added to a class, they can access the class page from their classes page, as if the class was theirs. They can do everything except invite other teachers (only the owner can do that).

Prettify spreads out the nodes pretty far, making the model take a lot of screen space when there's a lot of nodes.

Kurt says this could make a nice undergraduate/volunteer project.

I think this message should always show regardless of the number of times the user has seen it, but a CPI 360 student said it didn't appear when the model was done.

• Figure out why the model doesn't load
• Try to fix it, or create a working copy of the students work so Kurt can grade it.

See email for info.

In cement, the authors models is

When the student gets the sum slot wrong, OMRaaT marks one of the arguments wrong instead

To see the list of warnings, run the build (go to /www and run "gulp build"), then open the report (found in /release/www/build-report.txt) and search for ": WARNING" to find them easily.

After conferring with Kurt he has confirmed: all the controls should be editable on given nodes.

Edit: For example of this problem: Open the assignments in my first comment below from CPI Fall 2017 class (see assignment links; kvl account).

Make sure that no matter how many times someone edits an equation and hits check expression, the auto-created nodes can be edited (type box unlocked, no console errors).

Also update db user/passwords for:

• /home/laits/stable
• /home/laits/devel

Details:

user: Alma Camdzic
password: acamdzic

On clicking graph/table button, only an empty dialog pops up. No errors produced on console. Graph and table works fine on other problems.

@jonwetzel commented on Fri Aug 18 2017

From Kurt:

I just created in kvl on DRAGOON a new problem in the folder CPI fall 2017 called retirement 1. When I click on the Graph or Table button, nothing happens.