implemented temporary fix for displaying metadata, need to conform to…

… v8 standards, ready to begin writing method to write metadata
vzhang03 · Jun 7, 2024 · 1fb9919 · 1fb9919
1 parent 9697a97
commit 1fb9919
Show file tree

Hide file tree

Showing 3 changed files with 30 additions and 8 deletions.
diff --git a/examples/timeline-variables.html b/examples/timeline-variables.html
@@ -5,18 +5,26 @@
     <script src="../packages/plugin-html-keyboard-response/dist/index.browser.js"></script>
     <script src="../packages/plugin-image-keyboard-response/dist/index.browser.js"></script>
     <script src="../packages/plugin-preload/dist/index.browser.js"></script>
+    <script src="../packages/metadata/dist/index.browser.js"></script>  
     <link rel="stylesheet" href="../packages/jspsych/css/jspsych.css">
   </head>
   <body></body>
   <script>
 
     var jsPsych = initJsPsych({
       on_finish: function() {
-        jsPsych.data.displayData();
+        metadata.generate(jsPsych.data.get().json());
+
+        // metadata.saveAsJsonFile();
+        jsPsych.data.displayData();  
+        metadata.displayMetadata(jsPsych.getDisplayElement());
       },
       default_iti: 250
     });
 
+    var metadata = new jsPsychMetadata(); // could declare with jsPsych to show the after
+
+
     // manually preload the image files, since they are passed to the image-keyboard-response stimulus parameter
     // through timeline variables, and therefore cannot be automatically preloaded
     var images = ['img/happy_face_1.jpg', 'img/happy_face_2.jpg', 'img/happy_face_3.jpg'];

diff --git a/packages/metadata/rollup.config.mjs b/packages/metadata/rollup.config.mjs
@@ -1,3 +1,3 @@
-import { makeNodeRollupConfig } from "@jspsych/config/rollup";
+import { makeRollupConfig } from "@jspsych/config/rollup";
 
-export default makeNodeRollupConfig();
+export default makeRollupConfig("jsPsychMetadata");
diff --git a/packages/metadata/src/index.ts b/packages/metadata/src/index.ts
@@ -10,7 +10,7 @@ import { VariablesMap } from "./VariablesMap";
  * @class JsPsychMetadata
  * @typedef {JsPsychMetadata}
  */
-export class JsPsychMetadata {
+export default class JsPsychMetadata {
   /**
    * Field that contains all metadata fields that aren't represented as a list.
    *
@@ -213,15 +213,16 @@ export class JsPsychMetadata {
    *
    * @param {string} [elementId="jspsych-metadata-display"] - Id for how to style the metadata. Defaults to default styling.
    */
-  displayMetadata(elementId = "jspsych-metadata-display") {
+  displayMetadata(display_element) {
+    const elementId = "jspsych-metadata-display";
     const metadata_string = JSON.stringify(this.getMetadata(), null, 2);
-    const display_element = this.JsPsych.getDisplayElement();
+    // const display_element = this.JsPsych.getDisplayElement();
     display_element.innerHTML += `<p id="jspsych-metadata-header">Metadata</p><pre id="${elementId}" class="jspsych-preformat"></pre>`;
-    document.getElementById(elementId).textContent = metadata_string;
+    document.getElementById(elementId).textContent += metadata_string;
   }
 
   /**
-   * Method that begins a download for the dataset_dea.cription.json at the end of experiment.
+   * Method that begins a download for the dataset_description.json at the end of experiment.
    * Allows you to download the metadat.
    */
   saveAsJsonFile(): void {
@@ -238,4 +239,17 @@ export class JsPsychMetadata {
 
     URL.revokeObjectURL(url);
   }
+
+  generate(data) {
+    if (typeof data === "string") {
+      data = JSON.parse(data);
+    }
+
+    for (const data_point of data) {
+      // console.log("NEW ELEMENT \n")
+      console.log(data_point);
+    }
+
+    return this.getMetadata();
+  }
 }