#5709 - Unable to load variant sugar from HELM - system throws an err…

…or: Convert error! {} (#5716) - fixed sugar/phosphate/chem variant monomers render - fixed tests
epam · Oct 11, 2024 · 318564f · 318564f
1 parent 80f084a
commit 318564f
Show file tree

Hide file tree

Showing 6 changed files with 38 additions and 5 deletions.
diff --git a/ketcher-autotests/tests/Macromolecule-editor/Import-Saving-Files/import-saving-helm.spec.ts b/ketcher-autotests/tests/Macromolecule-editor/Import-Saving-Files/import-saving-helm.spec.ts
@@ -689,10 +689,11 @@ const incorrectHELMStrings: IHELMString[] = [
     helmDescription: '43. Negative probability (CHEM)',
     HELMString: 'CHEM1{([A6OH]:-10,[Az]:20)}$$$$V2.0',
   },
-  {
-    helmDescription: '44. Probability is greater than 100 (CHEM)',
-    HELMString: 'CHEM1{([A6OH]:10,[Az]:1000)}$$$$V2.0',
-  },
+  // {
+  //   helmDescription: '44. Probability is greater than 100 (CHEM)',
+  //   HELMString: 'CHEM1{([A6OH]:10,[Az]:1000)}$$$$V2.0',
+  // },
+  // It is not a bug because of Aleksandr Savelev suggestion
   {
     helmDescription: '45. Missing probability token (RNA)',
     HELMString: 'RNA1{R(A:,C:90)P}$$$$V2.0',

diff --git a/...rect-HELM-sequence-44-Probability-is-greater-than-100-CHEM-1-chromium-linux.png b/...rect-HELM-sequence-44-Probability-is-greater-than-100-CHEM-1-chromium-linux.png
diff --git a/packages/ketcher-core/src/application/render/renderers/AmbiguousMonomerRenderer.ts b/packages/ketcher-core/src/application/render/renderers/AmbiguousMonomerRenderer.ts
@@ -119,7 +119,9 @@ export class AmbiguousMonomerRenderer extends BaseMonomerRenderer {
   public show(theme) {
     super.show(theme);
     this.appendNumberOfMonomers();
-    this.appendEnumeration();
+    if (this.enumerationElementPosition) {
+      this.appendEnumeration();
+    }
     if (this.CHAIN_BEGINNING) {
       this.appendChainBeginning();
     }

diff --git a/packages/ketcher-core/src/application/render/renderers/constants.ts b/packages/ketcher-core/src/application/render/renderers/constants.ts
@@ -15,6 +15,7 @@ export const MONOMER_SYMBOLS_IDS = {
     hover: '#sugar-selection',
     selected: '#sugar-selection',
     body: '#sugar',
+    variant: '#sugar-variant',
   },
   [KetMonomerClass.Base]: {
     hover: '#rna-base-selection',
@@ -26,6 +27,7 @@ export const MONOMER_SYMBOLS_IDS = {
     hover: '#phosphate-selection',
     selected: '#phosphate-selection',
     body: '#phosphate',
+    variant: '#phosphate-variant',
   },
   [KetMonomerClass.RNA]: {
     hover: '#nucleotide-hover',

diff --git a/packages/ketcher-macromolecules/src/components/shared/monomerOnCanvas/Phosphate.tsx b/packages/ketcher-macromolecules/src/components/shared/monomerOnCanvas/Phosphate.tsx
@@ -41,5 +41,19 @@ export const PhosphateAvatar = () => (
         strokeWidth="1.5"
       />
     </symbol>
+    <symbol id="phosphate-variant" viewBox="0 0 70 70" width="70" height="70">
+      <rect
+        className="monomer-body"
+        width="27"
+        height="27"
+        data-actual-width="27"
+        data-actual-height="27"
+        stroke="#585858"
+        strokeWidth="0.5"
+        x="0.5"
+        y="0.5"
+        rx="15"
+      />
+    </symbol>
   </>
 );
diff --git a/packages/ketcher-macromolecules/src/components/shared/monomerOnCanvas/Sugar.tsx b/packages/ketcher-macromolecules/src/components/shared/monomerOnCanvas/Sugar.tsx
@@ -36,5 +36,19 @@ export const SugarAvatar = () => (
         strokeWidth="2.5"
       />
     </symbol>
+    <symbol id="sugar-variant" viewBox="0 0 72 72" width="70" height="70">
+      <rect
+        className="monomer-body"
+        width="28.5"
+        height="28.5"
+        data-actual-width="28.5"
+        data-actual-height="28.5"
+        x="0.5"
+        y="0.5"
+        rx="5"
+        stroke="#585858"
+        strokeWidth="0.5"
+      />
+    </symbol>
   </>
 );