nfdi4plants / ARCtrl

Library for management of Annotated Research Contexts (ARCs) using an in-memory representation and runtime-agnostic contract systems.
MIT License
16 stars 8 forks source link

[Bug]: Cannot open isa.study.xlsx file in Excel (macos and windows) #462

Closed Mestizia closed 2 weeks ago

Mestizia commented 4 months ago

Contact Details

No response

What happened?

When opening isa.study.xlsx file in Excel after adding a study sheet and populating it with headers (both macos and windows) it prompts an alert message (see screenshots). To open the file, it has to "repair it". If you then save the changes, the sheet will not appear in arcitect any longer. image image

The changes automatically made are the following (macos): image

What did you expect?

No response

Host

ARCitect

Host Details

Excel version 16.78 (macos)

Swate version

Swate Release Version v1.0.0-beta.04

Freymaurer commented 4 months ago

Hello! Could you share an example file with us? I will look into this!

Mestizia commented 4 months ago

Here you go isa.study.xlsx

Freymaurer commented 3 months ago

Ah okay, i see the issue the table from worksheet NewTable0 is removed, after reopening. This makes it not readable for Swate/ARCitect. Sadly this seems to be a deeper nested issue, which might take some time to investigate.

image

Freymaurer commented 3 months ago

Hey again, i tried to investigate this issue but i am unable to reproduce it with the columns you used. I am on windows 11. You said you had the same issues on windows?

Freymaurer commented 3 months ago

Will close for now, feel free to reopen

arendd commented 1 month ago

I'm facing the same issue. I can easily reproduce it. I just created a new table and added a "Characteristics" column. When I then want to open the xlsx with Excel then i receive the mentioned warning and the file is broken afterwards.

Freymaurer commented 1 month ago

Hey @arendd can you upload an file? I want to checkout the exact file 🙂

arendd commented 1 month ago

Sure, here it is: isa.study.xlsx

Freymaurer commented 1 month ago

Thanks! Will look into this as soon as possible!

arendd commented 1 month ago

Hey Kevin, I think i have an idea, what is the reason. This error comes when creating a new empty table with just some headers, but without rows. As long as you do not add any row via "+" button you can save the table within the ARCitect, but you can not open the according xlsx with Excel. After adding at least one line, Excel no longer throws this warning. Can you reproduce this?

Freymaurer commented 1 month ago

Hey Kevin, I think i have an idea, what is the reason. This error comes when creating a new empty table with just some headers, but without rows. As long as you do not add any row via "+" button you can save the table within the ARCitect, but you can not open the according xlsx with Excel. After adding at least one line, Excel no longer throws this warning. Can you reproduce this?

Yes can replicate this!

@HLWeil i thought we resolved this issue somewhen in ARCtrl? I am running version [2.0.0-beta.8.swate].

Freymaurer commented 2 weeks ago

@HLWeil bump

HLWeil commented 2 weeks ago

Ah sry, overlooked this. I agree that this probably is caused by the missing body of the table.