From 9ca99377c80329ac371a40b9f292380cb1d1279c Mon Sep 17 00:00:00 2001
From: Al-Jami Ismail <aljami1379@gmail.com>
Date: Mon, 17 Jul 2023 17:25:12 -0400
Subject: [PATCH 1/9] feat: make model parameters editable

---
 .../src/components/models/tera-model.vue      | 167 +++++++++++++++---
 .../petrinet/petrinet-service.ts              |  32 ++++
 2 files changed, 172 insertions(+), 27 deletions(-)

diff --git a/packages/client/hmi-client/src/components/models/tera-model.vue b/packages/client/hmi-client/src/components/models/tera-model.vue
index 69a9f7b9fc..722acbc5e7 100644
--- a/packages/client/hmi-client/src/components/models/tera-model.vue
+++ b/packages/client/hmi-client/src/components/models/tera-model.vue
@@ -89,7 +89,7 @@
 				:project="project"
 				:dialog-flavour="'model'"
 			/>
-			<Accordion multiple :active-index="[0, 1, 2, 3, 4, 5, 6]" @click="editSection">
+			<Accordion multiple :active-index="[0, 1, 2, 3, 4, 5, 6]">
 				<!-- Description -->
 				<AccordionTab>
 					<template #header>Description</template>
@@ -105,7 +105,7 @@
 					<template #header>
 						Parameters<span class="artifact-amount">({{ parameters?.length }})</span>
 					</template>
-					<main v-if="parameters.length > 0" class="datatable" style="--columns: 4">
+					<main v-if="parameters.length > 0" class="datatable" style="--columns: 5">
 						<header>
 							<div>ID</div>
 							<div>Value</div>
@@ -113,21 +113,35 @@
 							<div>Extractions</div>
 						</header>
 						<section
-							v-for="parameter in parameters"
+							v-for="(parameter, i) in parameters"
 							:key="parameter.id"
 							:class="[
 								{ active: isSectionEditable === `parameter-${parameter.id}` },
 								`parameter-${parameter.id}`
 							]"
 						>
-							<template v-if="isSectionEditable === `parameter-${parameter.id}`">
-								<div><input type="text" :value="parameter?.name ?? '--'" /></div>
-								<div><input type="text" :value="parameter?.value ?? '--'" /></div>
+							<template v-if="isRowEditable === `parameter-${parameter.id}`">
+								<div>
+									<input
+										type="text"
+										:value="parameter?.id ?? '--'"
+										@input="updateTable('parameters', i, 'id', $event.target?.value)"
+									/>
+								</div>
+								<div>
+									<input
+										type="text"
+										:value="parameter?.value ?? '--'"
+										@input="updateTable('parameters', i, 'id', $event.target?.value)"
+									/>
+								</div>
 								<div>--</div>
 								<div>
-									<!-- TODO: needs to make those button active -->
-									<Button icon="pi pi-check" text rounded aria-label="Save" />
-									<Button icon="pi pi-times" text rounded aria-label="Discard" />
+									<template v-if="parameter?.distribution?.parameters">
+										[{{ round(parameter?.distribution?.parameters.minimum, 4) }},
+										{{ round(parameter?.distribution?.parameters.maximum, 4) }}]
+									</template>
+									<template v-else>--</template>
 								</div>
 								<div v-if="extractions?.[parameter?.id]" style="grid-column: 1 / span 4">
 									<tera-model-extraction :extractions="extractions[parameter.id]" />
@@ -150,6 +164,13 @@
 									<template v-else>--</template>
 								</div>
 							</template>
+							<div v-if="!isRowEditable">
+								<Button icon="pi pi-file-edit" text @click="editRow" />
+							</div>
+							<div v-else-if="isRowEditable === `parameter-${parameter.id}`">
+								<Button icon="pi pi-check" text rounded aria-label="Save" @click="confirmEdit" />
+								<Button icon="pi pi-times" text rounded aria-label="Discard" @click="cancelEdit" />
+							</div>
 						</section>
 					</main>
 				</AccordionTab>
@@ -159,7 +180,7 @@
 					<template #header>
 						State variables<span class="artifact-amount">({{ states.length }})</span>
 					</template>
-					<main v-if="states.length > 0" class="datatable" style="--columns: 5">
+					<main v-if="states.length > 0" class="datatable" style="--columns: 6">
 						<header>
 							<div>Id</div>
 							<div>Name</div>
@@ -168,19 +189,32 @@
 							<div>Extractions</div>
 						</header>
 						<section
-							v-for="state in states"
+							v-for="(state, i) in states"
 							:key="state.id"
-							:class="[{ active: isSectionEditable === `state-${state.id}` }, `state-${state.id}`]"
+							:class="[{ active: isRowEditable === `state-${state.id}` }, `state-${state.id}`]"
 						>
-							<template v-if="isSectionEditable === `state-${state.id}`">
-								<div><input type="text" :value="state.id ?? '--'" /></div>
-								<div><input type="text" :value="state?.name ?? '--'" /></div>
+							<template v-if="isRowEditable === `state-${state.id}`">
+								<div>
+									<input
+										type="text"
+										:value="state?.id ?? '--'"
+										@input="updateTable('states', i, 'id', $event.target?.value)"
+									/>
+								</div>
+								<div>
+									<input
+										type="text"
+										:value="state?.name ?? '--'"
+										@input="updateTable('states', i, 'name', $event.target?.value)"
+									/>
+								</div>
 								<div><input type="text" :value="state?.units?.expression ?? '--'" /></div>
 								<div>Identifiers</div>
 								<div>
-									<!-- TODO: needs to make those button active -->
-									<Button icon="pi pi-check" text rounded aria-label="Save" />
-									<Button icon="pi pi-times" text rounded aria-label="Discard" />
+									<template v-if="extractions?.[state?.id]">
+										<Tag :value="extractions?.[state?.id].length" />
+									</template>
+									<template v-else>--</template>
 								</div>
 								<div v-if="extractions?.[state?.id]" style="grid-column: 1 / span 4">
 									<tera-model-extraction :extractions="extractions[state.id]" />
@@ -217,6 +251,13 @@
 									<template v-else>--</template>
 								</div>
 							</template>
+							<div v-if="!isRowEditable">
+								<Button icon="pi pi-file-edit" text @click="editRow" />
+							</div>
+							<div v-else-if="isRowEditable === `state-${state.id}`">
+								<Button icon="pi pi-check" text rounded aria-label="Save" @click="confirmEdit" />
+								<Button icon="pi pi-times" text rounded aria-label="Discard" @click="cancelEdit" />
+							</div>
 						</section>
 					</main>
 				</AccordionTab>
@@ -525,7 +566,10 @@ import Textarea from 'primevue/textarea';
 import TeraAsset from '@/components/asset/tera-asset.vue';
 import TeraRelatedPublications from '@/components/widgets/tera-related-publications.vue';
 import TeraModal from '@/components/widgets/tera-modal.vue';
-import { convertToAMRModel } from '@/model-representation/petrinet/petrinet-service';
+import {
+	convertToAMRModel,
+	replaceValuesInMathML
+} from '@/model-representation/petrinet/petrinet-service';
 import { RouteName } from '@/router/routes';
 import { getCuriesEntities } from '@/services/concept';
 import { createModel, addModelToProject, getModel, updateModel } from '@/services/model';
@@ -702,6 +746,7 @@ const time = computed(() =>
 	model.value?.semantics?.ode?.time ? [model.value?.semantics.ode.time] : []
 );
 const states = computed(() => model.value?.model?.states ?? []);
+const transientTableValue = ref();
 
 // Model Transitions
 const transitions = computed(() => {
@@ -743,6 +788,7 @@ const otherConcepts = computed(() => {
 	return [];
 });
 const isSectionEditable = ref<string | null>();
+const isRowEditable = ref<string | null>();
 
 const relatedTerariumModels = computed(
 	() => relatedTerariumArtifacts.value.filter((d) => isModel(d)) as Model[]
@@ -866,20 +912,87 @@ async function updateModelName() {
 	if (model.value) {
 		const modelClone = cloneDeep(model.value);
 		modelClone.name = newModelName.value;
-		updateModel(modelClone);
-		isRenamingModel.value = false;
+		await updateModel(modelClone);
 		model.value = await getModel(props.assetId);
 		// FIXME: Names aren't updated in sidebar
 	}
+	isRenamingModel.value = false;
 }
 
-// Toggle rows to become editable, display extractions to choose from.
-function editSection(event: Event) {
+// Toggle rows to become editable
+function editRow(event: Event) {
 	if (!event?.target) return;
-	const section = (event.target as HTMLElement).closest('.datatable section');
-	if (!section) return;
-	isSectionEditable.value =
-		isSectionEditable.value === section.className ? null : section.className;
+	const row = (event.target as HTMLElement).closest('.datatable section');
+	if (!row) return;
+	isRowEditable.value = isRowEditable.value === row.className ? null : row.className;
+}
+
+async function confirmEdit() {
+	if (model.value && transientTableValue.value) {
+		const { tableType, idx, updateProperty } = transientTableValue.value;
+		const modelClone = cloneDeep(model.value);
+
+		switch (tableType) {
+			case 'parameters':
+				if (modelClone.semantics?.ode.parameters) {
+					Object.entries(updateProperty).forEach(([key, value]) => {
+						modelClone.semantics!.ode.parameters![idx][key] = value;
+
+						if (key === 'id') {
+							// if we update parameter id we also need to update rate expression and expression_mathml
+							const updatedRates = model.value!.semantics!.ode.rates.map((rate) => ({
+								...rate,
+								expression: rate.expression.replaceAll(
+									model.value!.semantics!.ode.parameters![idx][key],
+									value as string
+								),
+								expression_mathml: rate.expression_mathml
+									? replaceValuesInMathML(
+											rate.expression_mathml,
+											model.value!.semantics!.ode.parameters![idx][key],
+											value as string
+									  )
+									: undefined
+							}));
+							modelClone.semantics!.ode.rates = updatedRates;
+						}
+					});
+				}
+				break;
+			case 'states':
+				Object.entries(updateProperty).forEach(([key, value]) => {
+					if (key !== 'unit') {
+						// TODO: remove this condition when we proper editing of unit
+						modelClone.model.states[idx][key] = value;
+					}
+					// TODO: update all of the properties affected by state id
+				});
+				break;
+			default:
+				logger.info(`${tableType} not recognized`);
+		}
+		await updateModel(modelClone);
+		model.value = await getModel(props.assetId);
+	}
+	isRowEditable.value = null;
+	transientTableValue.value = {};
+}
+
+function cancelEdit() {
+	isRowEditable.value = null;
+	transientTableValue.value = {};
+}
+
+function updateTable(tableType, idx, key, value) {
+	transientTableValue.value = {
+		...transientTableValue.value,
+		tableType: tableType,
+		idx: idx,
+		updateProperty: {
+			...transientTableValue.value?.updateProperty,
+			[key]: value
+		}
+	};
 }
 </script>
 
diff --git a/packages/client/hmi-client/src/model-representation/petrinet/petrinet-service.ts b/packages/client/hmi-client/src/model-representation/petrinet/petrinet-service.ts
index 7bc31db043..f6547025c3 100644
--- a/packages/client/hmi-client/src/model-representation/petrinet/petrinet-service.ts
+++ b/packages/client/hmi-client/src/model-representation/petrinet/petrinet-service.ts
@@ -408,6 +408,38 @@ export const updateTransition = (
 	});
 };
 
+// function to replace the content inside the tags of a mathml expression
+export const replaceValuesInMathML = (
+	mathmlExpression: string,
+	wordToReplace: string,
+	replaceWord: string
+): string => {
+
+	let expressionBuilder = '';
+	let isTag = false;
+	let content = '';
+
+	[...mathmlExpression].forEach((c) => {
+		if (!isTag && c === '<') {
+			isTag = true;
+		}
+		if (isTag && c === '>') {
+			isTag = false;
+		}
+
+		if (isTag) {
+			expressionBuilder += content.replaceAll(wordToReplace, replaceWord);
+			content = '';
+			expressionBuilder += c;
+		}
+		if (!isTag) {
+			content += c;
+		}
+	});
+
+	return expressionBuilder;
+};
+
 // Replace typing semantics
 export const addTyping = (amr: Model, typing: TypingSemantics) => {
 	if (amr.semantics) {

From 1907fbd419cf4bff41fec37b712e6cac4cc60f77 Mon Sep 17 00:00:00 2001
From: Al-Jami Ismail <aljami1379@gmail.com>
Date: Tue, 18 Jul 2023 08:22:24 -0400
Subject: [PATCH 2/9] dumb lint

---
 .../client/hmi-client/src/components/models/tera-model.vue    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/packages/client/hmi-client/src/components/models/tera-model.vue b/packages/client/hmi-client/src/components/models/tera-model.vue
index 722acbc5e7..882b074e33 100644
--- a/packages/client/hmi-client/src/components/models/tera-model.vue
+++ b/packages/client/hmi-client/src/components/models/tera-model.vue
@@ -986,8 +986,8 @@ function cancelEdit() {
 function updateTable(tableType, idx, key, value) {
 	transientTableValue.value = {
 		...transientTableValue.value,
-		tableType: tableType,
-		idx: idx,
+		tableType,
+		idx,
 		updateProperty: {
 			...transientTableValue.value?.updateProperty,
 			[key]: value

From 3cca4d620b5f7f4142d0a1d4d4793f7f157ed07e Mon Sep 17 00:00:00 2001
From: Al-Jami Ismail <aljami1379@gmail.com>
Date: Tue, 18 Jul 2023 08:25:51 -0400
Subject: [PATCH 3/9] typo

---
 packages/client/hmi-client/src/components/models/tera-model.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/client/hmi-client/src/components/models/tera-model.vue b/packages/client/hmi-client/src/components/models/tera-model.vue
index 882b074e33..793068fc24 100644
--- a/packages/client/hmi-client/src/components/models/tera-model.vue
+++ b/packages/client/hmi-client/src/components/models/tera-model.vue
@@ -132,7 +132,7 @@
 									<input
 										type="text"
 										:value="parameter?.value ?? '--'"
-										@input="updateTable('parameters', i, 'id', $event.target?.value)"
+										@input="updateTable('parameters', i, 'value', $event.target?.value)"
 									/>
 								</div>
 								<div>--</div>

From bd9d19a3755f27d266a204528cc235d0e13936a5 Mon Sep 17 00:00:00 2001
From: Al-Jami Ismail <aljami1379@gmail.com>
Date: Tue, 18 Jul 2023 09:49:58 -0400
Subject: [PATCH 4/9] prettier

---
 .../src/model-representation/petrinet/petrinet-service.ts        | 1 -
 1 file changed, 1 deletion(-)

diff --git a/packages/client/hmi-client/src/model-representation/petrinet/petrinet-service.ts b/packages/client/hmi-client/src/model-representation/petrinet/petrinet-service.ts
index a3000d6900..720e3da700 100644
--- a/packages/client/hmi-client/src/model-representation/petrinet/petrinet-service.ts
+++ b/packages/client/hmi-client/src/model-representation/petrinet/petrinet-service.ts
@@ -421,7 +421,6 @@ export const replaceValuesInMathML = (
 	wordToReplace: string,
 	replaceWord: string
 ): string => {
-
 	let expressionBuilder = '';
 	let isTag = false;
 	let content = '';

From 991bc8c4f502948f1231fb7f67b45e95580f5219 Mon Sep 17 00:00:00 2001
From: Al-Jami Ismail <aljami1379@gmail.com>
Date: Tue, 18 Jul 2023 10:09:29 -0400
Subject: [PATCH 5/9] typecheck

---
 .../hmi-client/src/components/models/tera-model.vue       | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/packages/client/hmi-client/src/components/models/tera-model.vue b/packages/client/hmi-client/src/components/models/tera-model.vue
index 793068fc24..55252e738d 100644
--- a/packages/client/hmi-client/src/components/models/tera-model.vue
+++ b/packages/client/hmi-client/src/components/models/tera-model.vue
@@ -125,14 +125,14 @@
 									<input
 										type="text"
 										:value="parameter?.id ?? '--'"
-										@input="updateTable('parameters', i, 'id', $event.target?.value)"
+										@input="updateTable('parameters', i, 'id', $event.target?.['value'])"
 									/>
 								</div>
 								<div>
 									<input
 										type="text"
 										:value="parameter?.value ?? '--'"
-										@input="updateTable('parameters', i, 'value', $event.target?.value)"
+										@input="updateTable('parameters', i, 'value', $event.target?.['value'])"
 									/>
 								</div>
 								<div>--</div>
@@ -198,14 +198,14 @@
 									<input
 										type="text"
 										:value="state?.id ?? '--'"
-										@input="updateTable('states', i, 'id', $event.target?.value)"
+										@input="updateTable('states', i, 'id', $event.target?.['value'])"
 									/>
 								</div>
 								<div>
 									<input
 										type="text"
 										:value="state?.name ?? '--'"
-										@input="updateTable('states', i, 'name', $event.target?.value)"
+										@input="updateTable('states', i, 'name', $event.target?.['value'])"
 									/>
 								</div>
 								<div><input type="text" :value="state?.units?.expression ?? '--'" /></div>

From cdfecb71a96129ba0feba0fc7059629a3564d26c Mon Sep 17 00:00:00 2001
From: Al-Jami Ismail <aljami1379@gmail.com>
Date: Tue, 18 Jul 2023 11:38:32 -0400
Subject: [PATCH 6/9] more robust string replacement

---
 .../src/components/models/tera-model.vue      |  4 +-
 .../petrinet/petrinet-service.ts              | 40 ++++++++++++++++++-
 2 files changed, 42 insertions(+), 2 deletions(-)

diff --git a/packages/client/hmi-client/src/components/models/tera-model.vue b/packages/client/hmi-client/src/components/models/tera-model.vue
index 55252e738d..586f1561fb 100644
--- a/packages/client/hmi-client/src/components/models/tera-model.vue
+++ b/packages/client/hmi-client/src/components/models/tera-model.vue
@@ -568,6 +568,7 @@ import TeraRelatedPublications from '@/components/widgets/tera-related-publicati
 import TeraModal from '@/components/widgets/tera-modal.vue';
 import {
 	convertToAMRModel,
+	replaceValuesInExpression,
 	replaceValuesInMathML
 } from '@/model-representation/petrinet/petrinet-service';
 import { RouteName } from '@/router/routes';
@@ -942,7 +943,8 @@ async function confirmEdit() {
 							// if we update parameter id we also need to update rate expression and expression_mathml
 							const updatedRates = model.value!.semantics!.ode.rates.map((rate) => ({
 								...rate,
-								expression: rate.expression.replaceAll(
+								expression: replaceValuesInExpression(
+									rate.expression,
 									model.value!.semantics!.ode.parameters![idx][key],
 									value as string
 								),
diff --git a/packages/client/hmi-client/src/model-representation/petrinet/petrinet-service.ts b/packages/client/hmi-client/src/model-representation/petrinet/petrinet-service.ts
index 720e3da700..c86e04fda6 100644
--- a/packages/client/hmi-client/src/model-representation/petrinet/petrinet-service.ts
+++ b/packages/client/hmi-client/src/model-representation/petrinet/petrinet-service.ts
@@ -415,6 +415,44 @@ export const updateTransition = (
 	});
 };
 
+const replaceExactString = (str: string, wordToReplace: string, replacementWord: string): string =>
+	str.trim() === wordToReplace.trim() ? str.replace(wordToReplace, replacementWord) : str;
+
+export const replaceValuesInExpression = (
+	expression: string,
+	wordToReplace: string,
+	replaceWord: string
+): string => {
+	let expressionBuilder = '';
+	let isOperator = false;
+	let content = '';
+
+	[...expression].forEach((c) => {
+		// not sure if this is an exhaustive list of operators or if it includes any operators it shouldn't
+		if ([',', '(', ')', '+', '-', '*', '/', '^'].includes(c)) {
+			isOperator = true;
+		} else {
+			isOperator = false;
+		}
+
+		if (isOperator) {
+			expressionBuilder += replaceExactString(content, wordToReplace, replaceWord);
+			content = '';
+			expressionBuilder += c;
+		}
+		if (!isOperator) {
+			content += c;
+		}
+	});
+
+	// if we reach the end of an expression and it doesn't end with an operator, we need to add the updated content
+	if (!isOperator) {
+		expressionBuilder += replaceExactString(content, wordToReplace, replaceWord);
+	}
+
+	return expressionBuilder;
+};
+
 // function to replace the content inside the tags of a mathml expression
 export const replaceValuesInMathML = (
 	mathmlExpression: string,
@@ -434,7 +472,7 @@ export const replaceValuesInMathML = (
 		}
 
 		if (isTag) {
-			expressionBuilder += content.replaceAll(wordToReplace, replaceWord);
+			expressionBuilder += replaceExactString(content, wordToReplace, replaceWord);
 			content = '';
 			expressionBuilder += c;
 		}

From e2ae354afeebada5d7be9888408cd3f4d2360731 Mon Sep 17 00:00:00 2001
From: Al-Jami Ismail <aljami1379@gmail.com>
Date: Tue, 18 Jul 2023 13:21:18 -0400
Subject: [PATCH 7/9] minor fix to mathml parser

---
 .../hmi-client/src/components/models/tera-model.vue      | 9 +++++----
 .../model-representation/petrinet/petrinet-service.ts    | 7 ++++++-
 2 files changed, 11 insertions(+), 5 deletions(-)

diff --git a/packages/client/hmi-client/src/components/models/tera-model.vue b/packages/client/hmi-client/src/components/models/tera-model.vue
index 586f1561fb..4e8095da9c 100644
--- a/packages/client/hmi-client/src/components/models/tera-model.vue
+++ b/packages/client/hmi-client/src/components/models/tera-model.vue
@@ -935,23 +935,24 @@ async function confirmEdit() {
 
 		switch (tableType) {
 			case 'parameters':
-				if (modelClone.semantics?.ode.parameters) {
+				if (model.value.semantics?.ode.parameters) {
 					Object.entries(updateProperty).forEach(([key, value]) => {
 						modelClone.semantics!.ode.parameters![idx][key] = value;
 
 						if (key === 'id') {
+							const ode = model.value!.semantics!.ode;
 							// if we update parameter id we also need to update rate expression and expression_mathml
-							const updatedRates = model.value!.semantics!.ode.rates.map((rate) => ({
+							const updatedRates = ode.rates.map((rate) => ({
 								...rate,
 								expression: replaceValuesInExpression(
 									rate.expression,
-									model.value!.semantics!.ode.parameters![idx][key],
+									ode.parameters![idx][key],
 									value as string
 								),
 								expression_mathml: rate.expression_mathml
 									? replaceValuesInMathML(
 											rate.expression_mathml,
-											model.value!.semantics!.ode.parameters![idx][key],
+											ode.parameters![idx][key],
 											value as string
 									  )
 									: undefined
diff --git a/packages/client/hmi-client/src/model-representation/petrinet/petrinet-service.ts b/packages/client/hmi-client/src/model-representation/petrinet/petrinet-service.ts
index c86e04fda6..c5cd49563a 100644
--- a/packages/client/hmi-client/src/model-representation/petrinet/petrinet-service.ts
+++ b/packages/client/hmi-client/src/model-representation/petrinet/petrinet-service.ts
@@ -477,7 +477,12 @@ export const replaceValuesInMathML = (
 			expressionBuilder += c;
 		}
 		if (!isTag) {
-			content += c;
+			// this only works if there is no '>' literal in the non-tag content
+			if (c !== '>') {
+				content += c;
+			} else {
+				expressionBuilder += c;
+			}
 		}
 	});
 

From 22a6f00579b75961c3a75049bd31c5a70c06aac6 Mon Sep 17 00:00:00 2001
From: Al-Jami Ismail <aljami1379@gmail.com>
Date: Wed, 19 Jul 2023 10:55:28 -0400
Subject: [PATCH 8/9] update model configurations if updating param value of og
 model

---
 .../src/components/models/tera-model.vue      | 46 +++++++++----------
 .../petrinet/petrinet-service.ts              | 40 +++++++++++++++-
 packages/client/hmi-client/src/types/Types.ts |  1 +
 .../dataservice/modelparts/ModelMetadata.java |  5 ++
 4 files changed, 66 insertions(+), 26 deletions(-)

diff --git a/packages/client/hmi-client/src/components/models/tera-model.vue b/packages/client/hmi-client/src/components/models/tera-model.vue
index 4e8095da9c..5fbadbb989 100644
--- a/packages/client/hmi-client/src/components/models/tera-model.vue
+++ b/packages/client/hmi-client/src/components/models/tera-model.vue
@@ -568,8 +568,8 @@ import TeraRelatedPublications from '@/components/widgets/tera-related-publicati
 import TeraModal from '@/components/widgets/tera-modal.vue';
 import {
 	convertToAMRModel,
-	replaceValuesInExpression,
-	replaceValuesInMathML
+	updateConfigFields,
+	updateParameterId
 } from '@/model-representation/petrinet/petrinet-service';
 import { RouteName } from '@/router/routes';
 import { getCuriesEntities } from '@/services/concept';
@@ -747,7 +747,14 @@ const time = computed(() =>
 	model.value?.semantics?.ode?.time ? [model.value?.semantics.ode.time] : []
 );
 const states = computed(() => model.value?.model?.states ?? []);
-const transientTableValue = ref();
+
+// Used to keep track of the values of the current row being edited
+interface ModelTableTypes {
+	tableType: string;
+	idx: number;
+	updateProperty: { [key: string]: string };
+}
+const transientTableValue = ref<ModelTableTypes | null>(null);
 
 // Model Transitions
 const transitions = computed(() => {
@@ -941,23 +948,12 @@ async function confirmEdit() {
 
 						if (key === 'id') {
 							const ode = model.value!.semantics!.ode;
-							// if we update parameter id we also need to update rate expression and expression_mathml
-							const updatedRates = ode.rates.map((rate) => ({
-								...rate,
-								expression: replaceValuesInExpression(
-									rate.expression,
-									ode.parameters![idx][key],
-									value as string
-								),
-								expression_mathml: rate.expression_mathml
-									? replaceValuesInMathML(
-											rate.expression_mathml,
-											ode.parameters![idx][key],
-											value as string
-									  )
-									: undefined
-							}));
-							modelClone.semantics!.ode.rates = updatedRates;
+							// update the parameter id in the model (as well as rate expression and expression_mathml)
+							updateParameterId(modelClone, ode.parameters![idx][key], value as string);
+
+							// note that this is making a call to an async function to update the different model configs
+							// but we don't need to wait for it to finish because we don't need immediate access to the model configs
+							updateConfigFields(model.value!.id, ode.parameters![idx][key], value as string);
 						}
 					});
 				}
@@ -965,7 +961,7 @@ async function confirmEdit() {
 			case 'states':
 				Object.entries(updateProperty).forEach(([key, value]) => {
 					if (key !== 'unit') {
-						// TODO: remove this condition when we proper editing of unit
+						// TODO: remove this condition when we have proper editing of unit
 						modelClone.model.states[idx][key] = value;
 					}
 					// TODO: update all of the properties affected by state id
@@ -974,19 +970,21 @@ async function confirmEdit() {
 			default:
 				logger.info(`${tableType} not recognized`);
 		}
+
 		await updateModel(modelClone);
 		model.value = await getModel(props.assetId);
 	}
+
 	isRowEditable.value = null;
-	transientTableValue.value = {};
+	transientTableValue.value = null;
 }
 
 function cancelEdit() {
 	isRowEditable.value = null;
-	transientTableValue.value = {};
+	transientTableValue.value = null;
 }
 
-function updateTable(tableType, idx, key, value) {
+function updateTable(tableType: string, idx: number, key: string, value: string) {
 	transientTableValue.value = {
 		...transientTableValue.value,
 		tableType,
diff --git a/packages/client/hmi-client/src/model-representation/petrinet/petrinet-service.ts b/packages/client/hmi-client/src/model-representation/petrinet/petrinet-service.ts
index ac17c962a0..4f7e0415b5 100644
--- a/packages/client/hmi-client/src/model-representation/petrinet/petrinet-service.ts
+++ b/packages/client/hmi-client/src/model-representation/petrinet/petrinet-service.ts
@@ -3,6 +3,8 @@ import API from '@/api/api';
 import { IGraph } from '@graph-scaffolder/types';
 import { PetriNetModel, Model, PetriNetTransition, TypingSemantics } from '@/types/Types';
 import { PetriNet } from '@/petrinet/petrinet-service';
+import { getModelConfigurations } from '@/services/model';
+import { updateModelConfiguration } from '@/services/model-configurations';
 
 export interface NodeData {
 	type: string;
@@ -419,7 +421,7 @@ export const updateTransition = (
 const replaceExactString = (str: string, wordToReplace: string, replacementWord: string): string =>
 	str.trim() === wordToReplace.trim() ? str.replace(wordToReplace, replacementWord) : str;
 
-export const replaceValuesInExpression = (
+const replaceValuesInExpression = (
 	expression: string,
 	wordToReplace: string,
 	replaceWord: string
@@ -455,7 +457,7 @@ export const replaceValuesInExpression = (
 };
 
 // function to replace the content inside the tags of a mathml expression
-export const replaceValuesInMathML = (
+const replaceValuesInMathML = (
 	mathmlExpression: string,
 	wordToReplace: string,
 	replaceWord: string
@@ -490,6 +492,40 @@ export const replaceValuesInMathML = (
 	return expressionBuilder;
 };
 
+export const updateParameterId = (amr: Model, id: string, newId: string) => {
+	if (amr.semantics?.ode.parameters) {
+		amr.semantics.ode.parameters.forEach((param) => {
+			if (param.id === id) {
+				param.id = newId;
+			}
+		});
+
+		// update the expression and expression_mathml fields
+		amr.semantics.ode.rates.forEach((rate) => {
+			rate.expression = replaceValuesInExpression(rate.expression, id, newId);
+			if (rate.expression_mathml) {
+				rate.expression_mathml = replaceValuesInMathML(rate.expression_mathml, id, newId);
+			}
+		});
+
+		// if there's a timeseries field with the old parameter id then update it to the new id
+		if (amr.metadata?.timeseries) {
+			delete Object.assign(amr.metadata.timeseries, { [newId]: amr.metadata.timeseries[id] })[id];
+		}
+	}
+};
+
+export const updateConfigFields = async (modelId: string, id: string, newId: string) => {
+	const modelConfigs = await getModelConfigurations(modelId);
+
+	modelConfigs.forEach((config) => {
+		updateParameterId(config.configuration, id, newId);
+		// note that this is making an async call but we don't need to wait for it to finish
+		// since we don't immediately need the updated configs
+		updateModelConfiguration(config);
+	});
+};
+
 // Replace typing semantics
 export const addTyping = (amr: Model, typing: TypingSemantics) => {
 	if (amr.semantics) {
diff --git a/packages/client/hmi-client/src/types/Types.ts b/packages/client/hmi-client/src/types/Types.ts
index 27c89f2ce6..e1d678310b 100644
--- a/packages/client/hmi-client/src/types/Types.ts
+++ b/packages/client/hmi-client/src/types/Types.ts
@@ -269,6 +269,7 @@ export interface ModelMetadata {
     variable_statements?: VariableStatement[];
     annotations?: Annotations;
     attributes: any[];
+    timeseries?: { [index: string]: any };
 }
 
 export interface ModelGrounding {
diff --git a/packages/services/hmi-server/src/main/java/software/uncharted/terarium/hmiserver/models/dataservice/modelparts/ModelMetadata.java b/packages/services/hmi-server/src/main/java/software/uncharted/terarium/hmiserver/models/dataservice/modelparts/ModelMetadata.java
index 0ae3a96b5e..ef305845d7 100644
--- a/packages/services/hmi-server/src/main/java/software/uncharted/terarium/hmiserver/models/dataservice/modelparts/ModelMetadata.java
+++ b/packages/services/hmi-server/src/main/java/software/uncharted/terarium/hmiserver/models/dataservice/modelparts/ModelMetadata.java
@@ -7,6 +7,7 @@
 import com.fasterxml.jackson.annotation.JsonProperty;
 
 import java.util.List;
+import java.util.Map;
 
 import software.uncharted.terarium.hmiserver.annotations.TSOptional;
 import software.uncharted.terarium.hmiserver.models.dataservice.modelparts.metadata.VariableStatement;
@@ -33,4 +34,8 @@ public class ModelMetadata {
 
 	@JsonProperty("attributes")
 	private List<JsonNode> attributes;
+
+	@JsonProperty("timeseries")
+	@TSOptional
+	private Map<String, Object> timeseries;
 }

From 399714ff614f2175fa47ab5e246cca767e3c7ad1 Mon Sep 17 00:00:00 2001
From: Al-Jami Ismail <aljami1379@gmail.com>
Date: Wed, 19 Jul 2023 12:31:03 -0400
Subject: [PATCH 9/9] update how timeseries filed is updated

---
 .../src/model-representation/petrinet/petrinet-service.ts    | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/packages/client/hmi-client/src/model-representation/petrinet/petrinet-service.ts b/packages/client/hmi-client/src/model-representation/petrinet/petrinet-service.ts
index 4f7e0415b5..4b909f8163 100644
--- a/packages/client/hmi-client/src/model-representation/petrinet/petrinet-service.ts
+++ b/packages/client/hmi-client/src/model-representation/petrinet/petrinet-service.ts
@@ -509,8 +509,9 @@ export const updateParameterId = (amr: Model, id: string, newId: string) => {
 		});
 
 		// if there's a timeseries field with the old parameter id then update it to the new id
-		if (amr.metadata?.timeseries) {
-			delete Object.assign(amr.metadata.timeseries, { [newId]: amr.metadata.timeseries[id] })[id];
+		if (amr.metadata?.timeseries && amr.metadata.timeseries[id]) {
+			amr.metadata.timeseries[newId] = amr.metadata.timeseries[id];
+			delete amr.metadata.timeseries[id];
 		}
 	}
 };