tpmkms 9.8.0-beta.1 → 9.8.0-beta.10

package/common/actions.instance.json

@@ -166,7 +166,7 @@
       },
       "semantics": [
         {
-          "match": "({context, isA, stack, namespaced}) => !context.pullFromContext && !context.evaluate && (isA(context.marker, 'action') || isA(context.marker, 'doAction')) && !namespaced.get('actions', context, 'logged')",
+          "match": "({context, isA, contextHierarchy, stack, namespaced}) => {\n        return !context.pullFromContext && !contextHierarchy.under('call') && (isA(context.marker, 'action') || isA(context.marker, 'doAction')) && !namespaced.get('actions', context, 'logged')\n      }",
           "apply": "async ({context, _continue, testLog, g, remember}) => {\n        remember(context)\n        await testLog(() => g(context))\n        _continue()\n      }"
         }
       ]

package/common/actions.js

@@ -147,7 +147,9 @@ const config = {
     {
       where: where(),
       priority: -1,
-      match: ({context, isA, stack, namespaced}) => !context.pullFromContext && !context.evaluate && (isA(context.marker, 'action') || isA(context.marker, 'doAction')) && !namespaced.get('actions', context, 'logged'),
+      match: ({context, isA, contextHierarchy, stack, namespaced}) => {
+        return !context.pullFromContext && !contextHierarchy.under('call') && (isA(context.marker, 'action') || isA(context.marker, 'doAction')) && !namespaced.get('actions', context, 'logged')
+      },
       apply: async ({context, _continue, testLog, g, remember}) => {
         remember(context)
         await testLog(() => g(context))

package/common/askfor.instance.json

@@ -30,6 +30,7 @@
         },
         {
           "id": "known",
+          "where": "/home/dev/code/theprogrammablemind/kms/common/askfor.js:77",
           "isA": [
             "queryable"
           ],
@@ -37,6 +38,7 @@
         },
         {
           "id": "information",
+          "where": "/home/dev/code/theprogrammablemind/kms/common/askfor.js:86",
           "isA": [
             "noun"
           ],
@@ -57,6 +59,7 @@
         },
         {
           "id": "askfor_askfor",
+          "where": "/home/dev/code/theprogrammablemind/kms/common/askfor.js:112",
           "isA": [
             "verb"
           ],
@@ -64,7 +67,7 @@
             "ask"
           ],
           "bridge": "{\n            ...next(operator),\n            properties: after[0],\n            interpolate: [{ self: true }, { property: 'properties' }]\n          }",
-          "semantic": "async ({e, s, gp, objects, flatten, context, ask, fragments, toEValue}) => {\n            const argument = context.properties.argument\n            const getValue = (property) => async (args) => {\n              const value = toEValue(await e(property))\n              if (value.marker == 'answerNotKnown') {\n                return\n              }\n              return value\n            }\n\n            const setValue = (property) => async ({ context, namespaced, tag }) => {\n              const is = { marker: 'is', one: property, two: context }\n              namespaced.set('dialogs', is, 'allowHierarchy', false)\n              await s(is)\n            }\n\n            const properties = flattenInPlace(argument)\n            counter = 0\n            for (const property of properties.reverse()) {\n              counter += 1\n              const query = memoizeAsync(async () => await(gp(await fragments(\"what is the concept?\", { concept: property }))))\n              const compatible_types = property.compatible_types || [property.marker]\n              const matchr = ({context, isA}) => {\n                return !context.same && !context.evaluate && isA(context, compatible_types)\n              }\n              objects.askFor.push(property);\n              askForProperty({\n                ask,\n                tag: `ask#${counter}`,\n                getValue: getValue(property),\n                setValue: setValue(property),\n                query,\n                matchr,\n              })\n            }\n          }"
+          "semantic": "async ({e, s, gp, objects, flatten, context, ask, fragments, toEValue}) => {\n            const argument = context.properties.argument\n            const getValue = (property) => async (args) => {\n              const value = toEValue(await e(property))\n              if (value.marker === 'undefined') {\n                return\n              }\n              return value\n            }\n\n            const setValue = (property) => async ({ context, namespaced, tag }) => {\n              const is = { marker: 'is', one: property, two: context }\n              namespaced.set('dialogs', is, 'allowHierarchy', false)\n              await s(is)\n            }\n\n            const properties = flattenInPlace(argument)\n            counter = 0\n            for (const property of properties.reverse()) {\n              counter += 1\n              const query = memoizeAsync(async () => await(gp(await fragments(\"what is the concept?\", { concept: property }))))\n              const compatible_types = property.compatible_types || [property.marker]\n              const matchr = ({context, isA}) => {\n                return !context.same && isA(context, compatible_types)\n              }\n              objects.askFor.push(property);\n              askForProperty({\n                ask,\n                tag: `ask#${counter}`,\n                getValue: getValue(property),\n                setValue: setValue(property),\n                query,\n                matchr,\n              })\n            }\n          }"
         }
       ]
     },
@@ -893,12 +896,14 @@
         },
         {
           "id": "known",
+          "where": "/home/dev/code/theprogrammablemind/kms/common/askfor.js:77",
           "isA": [
             "queryable"
           ]
         },
         {
           "id": "information",
+          "where": "/home/dev/code/theprogrammablemind/kms/common/askfor.js:86",
           "isA": [
             "noun"
           ]
@@ -918,6 +923,7 @@
         },
         {
           "id": "askfor_askfor",
+          "where": "/home/dev/code/theprogrammablemind/kms/common/askfor.js:112",
           "isA": [
             "verb"
           ],

package/common/askfor.js

@@ -74,6 +74,7 @@ const template = {
         },
         {
           id: 'known',
+          where: where(),
           isA: ['queryable'],
           evaluator: async ({kms, context, e, callId, toList, flatten, toEValue, resolveEvaluate, objects}) => {
             const known = await kms.properties.api.getProperty(context.subjects[0], 'property')
@@ -82,6 +83,7 @@ const template = {
         },
         {
           id: 'information',
+          where: where(),
           isA: ['noun'],
           evaluator: async ({context, e, callId, toList, flatten, toEValue, resolveEvaluate, objects}) => {
             const properties = objects.askFor
@@ -107,6 +109,7 @@ const template = {
         },
         {
           id: 'askfor_askfor',
+          where: where(),
           isA: ['verb'],
           words: ['ask'],
           bridge: `{
@@ -118,7 +121,7 @@ const template = {
             const argument = context.properties.argument
             const getValue = (property) => async (args) => {
               const value = toEValue(await e(property))
-              if (value.marker == 'answerNotKnown') {
+              if (value.marker === 'undefined') {
                 return
               }
               return value
@@ -137,7 +140,7 @@ const template = {
               const query = memoizeAsync(async () => await(gp(await fragments("what is the concept?", { concept: property }))))
               const compatible_types = property.compatible_types || [property.marker]
               const matchr = ({context, isA}) => {
-                return !context.same && !context.evaluate && isA(context, compatible_types)
+                return !context.same && isA(context, compatible_types)
               }
               objects.askFor.push(property);
               askForProperty({

package/common/asking.js

@@ -26,6 +26,7 @@ const config = {
   bridges: [
     {
       id: "nevermind",
+      where: where(),
       bridge: "{ ...next(operator), complete: true }",
       semantic: (args) => {
         const {config, context} = args
@@ -51,6 +52,7 @@ const config = {
 
     {
       id: "whichOnesTestSetup",
+      where: where(),
       scope: "testing",
       generatorp: async ({context, gs}) => `${context.marker} ${await gs(context.choices)}`,
       bridge: "{ ...next(operator), choices: after[0] }",
@@ -91,6 +93,7 @@ const config = {
 
     {
       id: "nevermindTestSetup",
+      where: where(),
       scope: "testing",
       bridge: "{ ...next(operator), type: after[0], postModifiers: ['type'] }",
       semantic: ({ask, context}) => {
@@ -156,7 +159,9 @@ function getAsk(config) {
             oneShot: ask.oneShot ?? true,
             where: semantic.where || ask.where || where(2),
             source: 'response',
-            match: (args) => !args.context.isResponse && !args.context.evaluate && !args.context.isControl && semantic.match(args),
+            match: async (args) => await args.callOnce(args, 'asking.1', async (args) => {
+              return !args.context.isResponse &&                           args.context.topLevel && !args.context.isControl && await semantic.match(args)
+            }),
             apply: async (args) => {
               setWasApplied(true)
               await semantic.apply({ ...args, tag: ask.tag })
@@ -187,13 +192,13 @@ function getAsk(config) {
               matchq = () => !wasAsked,
               applyq = (args) => {
                 wasAsked = true
-                return ask.applyq({ ...args, tag: ask.tag })
+                return args.stack.wrap(() => ask.applyq({ ...args, tag: ask.tag }), where())
               }
             }
-            if (await matchq(args)) {
+            if (await args.stack.wrap(() => matchq(args), where())) {
               setWasApplied(false)
               // args.context.motivationKeep = true
-              const query = await applyq({ ...args, wasAsked: getWasAsked() })
+              const query = await args.stack.wrap(() => applyq({ ...args, wasAsked: getWasAsked() }), where())
               if (typeof query != 'string') {
                 throw new Error(`ask expects the value return from applyq to be a string. ${ask.where}. Instead got ${JSON.stringify(query, null, 2)}.`)
               }

package/common/avatar.js

@@ -27,7 +27,7 @@ const config = {
     {
        where: where(),
        notes: "unknown answer default response for avatar",
-       match: ({context}) => context.marker == 'answerNotKnown',
+       match: ({context}) => context.marker === 'undefined',
        apply: ({context}) => `I don't know`,
     },
     {

package/common/concept.instance.json

@@ -11,6 +11,7 @@
         {
           "id": "modifies",
           "enhanced_associations": false,
+          "where": "/home/dev/code/theprogrammablemind/kms/common/concept.js:28",
           "isA": [
             "verb"
           ],
@@ -100,6 +101,7 @@
       "bridges": [
         {
           "id": "isOpenCompound",
+          "where": "/home/dev/code/theprogrammablemind/kms/common/concept.js:154",
           "isA": [
             "verb"
           ],
@@ -142,6 +144,7 @@
         {
           "id": "modifies",
           "enhanced_associations": false,
+          "where": "/home/dev/code/theprogrammablemind/kms/common/concept.js:28",
           "isA": [
             "verb"
           ],
@@ -160,7 +163,7 @@
           "bridge": "{ ...next(operator), conceptModifiers: before[0], concept: after[0] }",
           "semantic": {
             "notes": "define a modifier",
-            "where": "/home/dev/code/theprogrammablemind/kms/common/concept.js:34"
+            "where": "/home/dev/code/theprogrammablemind/kms/common/concept.js:35"
           },
           "check": [
             "marker",
@@ -217,10 +220,10 @@
       "generators": [
         {
           "notes": "\"fire type, water type and earth type\" to \"fire water and earth type\"",
-          "where": "/home/dev/code/theprogrammablemind/kms/common/concept.js:84"
+          "where": "/home/dev/code/theprogrammablemind/kms/common/concept.js:85"
         },
         {
-          "where": "/home/dev/code/theprogrammablemind/kms/common/concept.js:123"
+          "where": "/home/dev/code/theprogrammablemind/kms/common/concept.js:124"
         }
       ]
     },
@@ -682,6 +685,7 @@
       "bridges": [
         {
           "id": "isOpenCompound",
+          "where": "/home/dev/code/theprogrammablemind/kms/common/concept.js:154",
           "isA": [
             "verb"
           ],
@@ -690,7 +694,7 @@
             "is"
           ],
           "semantic": {
-            "where": "/home/dev/code/theprogrammablemind/kms/common/concept.js:163"
+            "where": "/home/dev/code/theprogrammablemind/kms/common/concept.js:165"
           },
           "enhanced_associations": true,
           "check": [

package/common/concept.js

@@ -25,6 +25,7 @@ const config = {
     {
       id: "modifies",
       enhanced_associations: false,
+      where: where(),
       isA: ['verb'],
       words: [{ word: 'modifies', number: 'one', flatten: false }, { word: 'modify', number: 'many', flatten: true }],
       // bridge: "{ ...next(operator), modifiers: before, concept: after[0], flatten: true }"
@@ -150,6 +151,7 @@ const template = {
       bridges: [
         {
           id: 'isOpenCompound',
+          where: where(),
           isA: ['verb'],
           bridge: `{
             ...operator,

package/common/conjunction.js

@@ -39,10 +39,12 @@ const config = {
           passthrough: true
      }, 
       bridge: "{ ...operator, value: append(before, operator.value) }",
+      where: where(),
       semantic: {
         match: ({context, namespaced}) => !namespaced.get('conjunction', context, 'atomic'),
         apply: async ({context, flatten, s}) => {
           for (const value of flatten(context)) {
+            value.topLevel = context.topLevel;
             await s(value)
           }
         }

package/common/control.js

@@ -3,8 +3,8 @@ const tests = require('./control.test.json')
 
 function initializer({config}) {
   config.addArgs((args) => ({
-    callOnce: (args, tag, condition) => {
-      if (condition(args)) {
+    callOnce: async (args, tag, condition) => {
+      if (await condition(args)) {
         const { context } = args
 
         if (!context.control) {

package/common/countable.js

@@ -98,6 +98,7 @@ const config = {
     },
     { 
       id: "all", 
+      where: where(),
       generatorp: ({context}) => 'all',
       bridge: "{ ...next(operator), number: 'many', quantity: 'all' }" 
     },

package/common/dateTimeSelectors.instance.json

@@ -67,15 +67,15 @@
       ],
       "semantics": [
         {
-          "match": "({context, isA}) => context.evaluate && onOrIs('dayAfterDate', context)",
+          "match": "({context, isA}) => onOrIs('dayAfterDate', context)",
           "apply": "async ({context, isProcess, isTest, kms, isA, e}) => {\n            try {\n              const now = kms.time.api.now()\n              const date = afterOrIs('dayAfterDate', context)\n              const afterISO = (await e(date.after)).evalue\n              const day_ordinal = date.day.day_ordinal\n              const ordinal = date.day.ordinal.value\n              context.evalue = dateTimeSelectors_helpers.getNthWeekdayAfterDate(afterISO, day_ordinal, ordinal)\n            } catch ( e ) {\n              context.evalue = `Implement instatiate for this type of date, ${context.marker}. See the dateTimeSelectors KM ${where()}. ${e}`\n            }\n          }"
         },
         {
-          "match": "({context, isA}) => context.evaluate && onOrIs('dayOfMonth', context)",
+          "match": "({context, isA}) => onOrIs('dayOfMonth', context)",
           "apply": "({context, isProcess, isTest, kms, isA}) => {\n            try {\n              const now = kms.time.api.now()\n              const date = onOrIs('dayOfMonth', context)\n              context.evalue = dateTimeSelectors_helpers.getNthDayOfMonth(removeDatesSuffix(date.day.value), date.day.ordinal.value || 1, removeDatesSuffix(date.month.value), now)\n            } catch ( e ) {\n              context.evalue = `Implement instatiate for this type of date, ${context.marker}. See the dateTimeSelectors KM ${where()}. ${e}`\n            }\n          }"
         },
         {
-          "match": "({context, isA}) => {\n            if (!context.evaluate) {\n              return false\n            }\n            if (isA(context.marker, 'onDateValue_dates')) {\n              return true\n            }\n            if (isA(context.marker, 'dateTimeSelector')) {\n              return true\n            }\n          }",
+          "match": "({context, isA}) => {\n            if (isA(context.marker, 'onDateValue_dates')) {\n              return true\n            }\n            if (isA(context.marker, 'dateTimeSelector')) {\n              return true\n            }\n          }",
           "apply": "({context, isProcess, isTest, kms, isA}) => {\n            context.evalue = instantiate(kms, isA, isProcess || isTest || context.isTest, context)\n          }"
         }
       ],

package/common/dateTimeSelectors.js

@@ -94,7 +94,7 @@ const template = {
       ],
       semantics: [
         {
-          match: ({context, isA}) => context.evaluate && onOrIs('dayAfterDate', context),
+          match: ({context, isA}) => onOrIs('dayAfterDate', context),
           apply: async ({context, isProcess, isTest, kms, isA, e}) => {
             try {
               const now = kms.time.api.now()
@@ -109,7 +109,7 @@ const template = {
           },
         },
         {
-          match: ({context, isA}) => context.evaluate && onOrIs('dayOfMonth', context),
+          match: ({context, isA}) => onOrIs('dayOfMonth', context),
           apply: ({context, isProcess, isTest, kms, isA}) => {
             try {
               const now = kms.time.api.now()
@@ -122,9 +122,6 @@ const template = {
         },
         {
           match: ({context, isA}) => {
-            if (!context.evaluate) {
-              return false
-            }
             if (isA(context.marker, 'onDateValue_dates')) {
               return true
             }
@@ -156,7 +153,7 @@ knowledgeModule( {
     checks: {
       context: [
         // defaultContextCheck({ extra: ['date', 'time', 'response', 'after', 'day', 'month', 'year', 'evalue'] }),
-        defaultContextCheck({ extra: ['date', 'time', 'response', 'after', 'evalue'] }),
+        defaultContextCheck({ extra: ['date', 'time', 'after', 'evalue'] }),
       ],
     }
   },

package/common/dates.instance.json

@@ -184,6 +184,7 @@
         },
         {
           "id": "dateEra_dates",
+          "where": "/home/dev/code/theprogrammablemind/kms/common/dates.js:129",
           "isA": [
             "date_dates"
           ],
@@ -215,6 +216,7 @@
         },
         {
           "id": "monthDayYearWithSlashes_dates",
+          "where": "/home/dev/code/theprogrammablemind/kms/common/dates.js:143",
           "isA": [
             "date_dates"
           ],
@@ -266,6 +268,7 @@
         },
         {
           "id": "monthDay_dates",
+          "where": "/home/dev/code/theprogrammablemind/kms/common/dates.js:170",
           "convolution": true,
           "localHierarchy": [
             [
@@ -300,6 +303,7 @@
         },
         {
           "id": "monthYear_dates",
+          "where": "/home/dev/code/theprogrammablemind/kms/common/dates.js:181",
           "convolution": true,
           "before": [
             "preposition"
@@ -328,6 +332,7 @@
         },
         {
           "id": "monthDayYear_dates",
+          "where": "/home/dev/code/theprogrammablemind/kms/common/dates.js:191",
           "convolution": true,
           "before": [
             "preposition",
@@ -15147,6 +15152,7 @@
         },
         {
           "id": "dateEra_dates",
+          "where": "/home/dev/code/theprogrammablemind/kms/common/dates.js:129",
           "isA": [
             "date_dates"
           ],
@@ -15177,6 +15183,7 @@
         },
         {
           "id": "monthDayYearWithSlashes_dates",
+          "where": "/home/dev/code/theprogrammablemind/kms/common/dates.js:143",
           "isA": [
             "date_dates"
           ],
@@ -15227,6 +15234,7 @@
         },
         {
           "id": "monthDay_dates",
+          "where": "/home/dev/code/theprogrammablemind/kms/common/dates.js:170",
           "convolution": true,
           "localHierarchy": [
             [
@@ -15260,6 +15268,7 @@
         },
         {
           "id": "monthYear_dates",
+          "where": "/home/dev/code/theprogrammablemind/kms/common/dates.js:181",
           "convolution": true,
           "before": [
             "preposition"
@@ -15287,6 +15296,7 @@
         },
         {
           "id": "monthDayYear_dates",
+          "where": "/home/dev/code/theprogrammablemind/kms/common/dates.js:191",
           "convolution": true,
           "before": [
             "preposition",

package/common/dates.js

@@ -126,6 +126,7 @@ const template = {
         },
         { 
           id: 'dateEra_dates', 
+          where: where(),
           isA: ['date_dates'],
           convolution: true,
           bridge: "{ ...next(after[0]), era: after[1], interpolate: concat(after[0].interpolate, ' ${era}') }",
@@ -139,6 +140,7 @@ const template = {
         },
         { 
           id: 'monthDayYearWithSlashes_dates', 
+          where: where(),
           isA: ['date_dates'],
           before: ['preposition'],
           separators: "||||",
@@ -165,6 +167,7 @@ const template = {
         },
         { 
           id: 'monthDay_dates', 
+          where: where(),
           convolution: true,
           localHierarchy: [['ordinal', 'dayNumber_dates']],
           before: ['preposition'],
@@ -175,6 +178,7 @@ const template = {
         },
         { 
           id: 'monthYear_dates', 
+          where: where(),
           convolution: true,
           before: ['preposition'],
           isA: ['date_dates'],
@@ -184,6 +188,7 @@ const template = {
         },
         { 
           id: 'monthDayYear_dates', 
+          where: where(),
           convolution: true,
           before: ['preposition', 'monthDay_dates'],
           isA: ['date_dates'],

package/common/debug.instance.json

@@ -8,11 +8,13 @@
       "bridges": [
         {
           "id": "setdebugbreak",
+          "where": "/home/dev/code/theprogrammablemind/kms/common/debug.js:16",
           "bridge": "{\n            ...next(operator),\n            tag: after[0]\n          }",
           "semantic": "({context, debug}) => {\n            debug.hit(context.tag.word)\n          }"
         },
         {
           "id": "debugger",
+          "where": "/home/dev/code/theprogrammablemind/kms/common/debug.js:27",
           "semantic": "(args) => {\n            debugger // eslint-disable-line no-debugger\n            debugger // eslint-disable-line no-debugger\n          }"
         }
       ]
@@ -28,10 +30,12 @@
       "bridges": [
         {
           "id": "setdebugbreak",
+          "where": "/home/dev/code/theprogrammablemind/kms/common/debug.js:16",
           "bridge": "{\n            ...next(operator),\n            tag: after[0]\n          }"
         },
         {
-          "id": "debugger"
+          "id": "debugger",
+          "where": "/home/dev/code/theprogrammablemind/kms/common/debug.js:27"
         }
       ]
     }

package/common/debug.js

@@ -13,6 +13,7 @@ const template = {
       bridges: [
         { 
           id: 'setdebugbreak',
+          where: where(),
           bridge: `{
             ...next(operator),
             tag: after[0]
@@ -23,6 +24,7 @@ const template = {
         },
         {
           id: 'debugger',
+          where: where(),
           semantic: (args) => {
             debugger // eslint-disable-line no-debugger
             debugger // eslint-disable-line no-debugger

package/common/dialogues.js

@@ -110,6 +110,7 @@ const config = {
     },
     {
       id: 'makeObject',
+      where: where(),
       bridge: "{ ...next(operator), object: after[0] }",
       enhanced_associations: false,
       generatorp: async ({context, gp}) => `${context.word} ${await gp(context.object)}`,
@@ -119,6 +120,7 @@ const config = {
     },
     {
       id: 'setIdSuffix',
+      where: where(),
       enhanced_associations: false,
       bridge: "{ ...next(operator), suffix: after[0] }",
       generatorp: async ({context, gp}) => `${context.word} ${await gp(context.suffix)}`,
@@ -128,6 +130,7 @@ const config = {
     },
     {
       id: 'resetIdSuffix',
+      where: where(),
       semantic: ({context, api}) => {
         api.setIdSuffix('')
       }
@@ -301,13 +304,13 @@ const config = {
     {
       where: where(),
       notes: "unknown answer default response",
-      match: ({context, namespaced}) => context.marker == 'answerNotKnown' && context.brief,
+      match: ({context, namespaced}) => context.marker === 'undefined' && context.brief,
       apply: ({context}) => `not known`,
     },
     {
       where: where(),
       notes: "unknown answer default response",
-      match: ({context}) => context.marker == 'answerNotKnown',
+      match: ({context}) => context.marker === 'undefined',
       apply: ({context}) => `that is not known`,
     },
     {
@@ -446,7 +449,7 @@ const config = {
       notes: 'x is y',
       match: ({context, hierarchy}) => hierarchy.isA(context.marker, 'is') && !context.evalue,
       apply: async ({isA, context, g, gp, debug, gr, callId}) => {
-        if ((context.two.evalue || {}).marker == 'answerNotKnown') {
+        if ((context.two.evalue || {}).marker === 'undefined') {
           return await g(context.two.evalue)
         }
 
@@ -454,8 +457,8 @@ const config = {
           return `${await gp(context.one)} ${isMany(context.one) || isMany(context.two) || isMany(context) ? "are" : "is"} ${await g(context.two)}`
         }
 
-        if (context.response) {
-          context = context.response
+        if (context.evalue) {
+          context = context.evalue
         }
 
         function hasFocus(property) {
@@ -575,6 +578,7 @@ const config = {
       apply: async ({context, s, log, km, objects, e, toEValue}) => {
         const one = context.one;
         const two = context.two;
+
         let query, value;
         if (one.query) {
           query = one;
@@ -591,7 +595,7 @@ const config = {
           return
         }
 
-        if (toEValue(instance)?.marker == 'answerNotKnown') {
+        if (toEValue(instance)?.marker === 'undefined') {
           context.evalue = instance
           context.isResponse = true
           return