nfdi4plants / ARCitect

Cross-platform app to create and manage ARCs.
9 stars 6 forks source link

[BUG] Blank content in assay sheets (for MS data) #213

Closed garibayh closed 4 months ago

garibayh commented 4 months ago

OS and framework information (please complete the following information):

Describe the bug

  1. Open Sheet from Assay, that works Ok :) 20240709_Assay-CorrectSheet

  2. Open MS-related sheet that looks empty (but is not empty!) :( 20240709_Assay-MSSheet

The sheets are completely available in the downloaded isa file: 20240709_Assay-MSSheet-IsaFile

Freymaurer commented 4 months ago

Hello! Is this ARC publicly available? Can you open the developer console (Ctrl + Shift + i) and keep it open while trying to open the "MS-related sheet" and sent us a screen shot + text of possible logs?

garibayh commented 4 months ago

Hello! An error appears in the console when I try to open the blank sheets: 20240709_Assay-MSSheet_ConsoleError

Received Assay ValidTiming_BuP02_MS from ARCitect! 14index-DJejvIHO.js:108 Error: Match failure: ARCtrl.CompositeCell at ore (index-DJejvIHO.js:108:209127) at index-DJejvIHO.js:108:216344 at bF.f (index-DJejvIHO.js:4:15988) at bF.GetEnumerator (index-DJejvIHO.js:4:13971) at bt (index-DJejvIHO.js:1:1799) at So (index-DJejvIHO.js:4:9685) at Ae (index-DJejvIHO.js:4:16255) at index-DJejvIHO.js:108:215884 at bF.f (index-DJejvIHO.js:4:15988) at bF.GetEnumerator (index-DJejvIHO.js:4:13971)

index-DJejvIHO.js:108 Uncaught Error: Match failure: ARCtrl.CompositeCell at ore (index-DJejvIHO.js:108:209127) at index-DJejvIHO.js:108:216344 at bF.f (index-DJejvIHO.js:4:15988) at bF.GetEnumerator (index-DJejvIHO.js:4:13971) at bt (index-DJejvIHO.js:1:1799) at So (index-DJejvIHO.js:4:9685) at Ae (index-DJejvIHO.js:4:16255) at index-DJejvIHO.js:108:215884 at bF.f (index-DJejvIHO.js:4:15988) at bF.GetEnumerator (index-DJejvIHO.js:4:13971) ore @ index-DJejvIHO.js:108 (anonymous) @ index-DJejvIHO.js:108 (anonymous) @ index-DJejvIHO.js:4 GetEnumerator @ index-DJejvIHO.js:4 bt @ index-DJejvIHO.js:1 So @ index-DJejvIHO.js:4 Ae @ index-DJejvIHO.js:4 (anonymous) @ index-DJejvIHO.js:108 (anonymous) @ index-DJejvIHO.js:4 GetEnumerator @ index-DJejvIHO.js:4 bt @ index-DJejvIHO.js:1 (anonymous) @ index-DJejvIHO.js:4 System.Collections.IEnumerator.MoveNext @ index-DJejvIHO.js:4 (anonymous) @ index-DJejvIHO.js:4 System.Collections.IEnumerator.MoveNext @ index-DJejvIHO.js:4 (anonymous) @ index-DJejvIHO.js:4 System.Collections.IEnumerator.MoveNext @ index-DJejvIHO.js:4 (anonymous) @ index-DJejvIHO.js:4 System.Collections.IEnumerator.MoveNext @ index-DJejvIHO.js:4 So @ index-DJejvIHO.js:4 Ae @ index-DJejvIHO.js:4 rw @ index-DJejvIHO.js:108 px @ index-DJejvIHO.js:106 l$ @ index-DJejvIHO.js:108 o$ @ index-DJejvIHO.js:108 tJ @ index-DJejvIHO.js:108 S1 @ index-DJejvIHO.js:108 $S @ index-DJejvIHO.js:108 BT @ index-DJejvIHO.js:108 mf @ index-DJejvIHO.js:106 (anonymous) @ index-DJejvIHO.js:108

Freymaurer commented 4 months ago

Could you send us the relevant file? I will investigate this issue!

garibayh commented 4 months ago

It happens with several files... I placed a second one that is not MS related isa.assay.xlsx isa.assay.xlsx

Freymaurer commented 4 months ago

Thank you for sharing the files, i tested the files with the current development version of Swate and they are running. I suspect the error to be related to our implementation work towards the datamap specification.

https://github.com/nfdi4plants/ARC-specification/pull/104

It seems like ARCitect, Swate, ARCtrl are out of sync at the moment for some cases. This will hopefully be fixed soon. Expect the next Swate release tomorrow night. This should resolve your issue! I will close this for now, should it persist over the next few days or you have more questions feel free to reopen 🙂