Error when importing Swate Templates

nfdi4plants / Swate

Excel Add-In for annotation of experimental data and computational workflows.

https://swate-alpha.nfdi4plants.org

MIT License

31 stars 6 forks source link

Error when importing Swate Templates #439

Closed frsommer closed 3 months ago

frsommer commented 3 months ago

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

OS Windows 10

Describe the bug

[When Trying to open a template for sampel description (community or curated template) error occurs ] A clear and concise description of what the bug is
[add assay, add newTable, pick template , add template - Error: "Specified index is out of table range! Table contains only 0 columns." Occurs with template icon on top and with template picker on right hand Swate menue
But sometimes not reproducible... ] Steps to reproduce
[ ] Screenshots
[add template table ] Expected behavior
[none ] Ideas for solution

Freymaurer commented 3 months ago

Can you give me the exact template you picked for this?

frsommer commented 3 months ago

no specific template, i tried several templates (e.g. RPTU Massspec (comunity) or one of the curated (whatever that means) forgot which ones i tried ), but error is sometimes consistent, sometimes not reproducible. sorry i know you cannot work with non reproducible errors, it just occured since app. 1 or 2 weeks, never saw that before. but versions did not change as far as i can see.

will add when i can find a systematic in the error occurance, or can you see which process generates the error?

Freymaurer commented 3 months ago

Will have to look into it 👍 We are currently working on a larger update on the core logic. That might just resolve the error.

SabrinaZander commented 3 months ago

I've noticed this error from time to time, too. But I can't reproduce it. I have the feeling that the sheet isn't being recognized correctly. If you click on the sheet again in the tab below and then add the template, the error goes away.

Freymaurer commented 3 months ago

Sadly i cannot reproduce the error. If i should ever encounter it i will look deeper into it, for now i close the issue. If it happens again after the next release, feel free to reopen!

frsommer commented 3 months ago

THX

From: Kevin Frey @.> Sent: Wednesday, June 19, 2024 2:10 PM To: nfdi4plants/ARCitect @.> Cc: Frederik Sommer @.>; Author @.> Subject: Re: [nfdi4plants/ARCitect] Error when importing Swate Templates (Issue nfdi4plants/Swate#439)

Sadly i cannot reproduce the error. If i should ever encounter it i will look deeper into it, for now i close the issue. If it happens again after the next release, feel free to reopen!

— Reply to this email directly, view it on GitHubhttps://github.com/nfdi4plants/Swate/issues/439, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AXEZTEU2ZNF7LMOFVLRXRALZIFYLHAVCNFSM6AAAAABJJ336YSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZYGUZTSMBXGU. You are receiving this because you authored the thread.Message ID: @.**@.>>

Freymaurer commented 3 months ago

I was able to replicate the bug and found the reason! Will be fixed soon!

(For anyone interested, if you click in one table on a cell, therefore set it as active, then delete the table. Swate still remembers the index of the active cell, which was not removed). Then when trying to use the active cell it notices it does not exist! 😥😞

EDIT: The same for creating a new table)

Freymaurer commented 3 months ago

Done! This will be part of the next release!