Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
Mestizia opened this issue Jul 10, 2024 · 13 comments · Fixed by #464
Closed

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

Mestizia opened this issue Jul 10, 2024 · 13 comments · Fixed by #464
Assignees
Labels
Type: Bug Something is not working, and it is confirmed by maintainers to be a bug.

Comments

@Mestizia
Copy link

Mestizia commented Jul 10, 2024

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

@github-actions github-actions bot added the Status: Needs Triage This item is up for investigation. label Jul 10, 2024
@Freymaurer
Copy link
Collaborator

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

@Mestizia
Copy link
Author

Here you go
isa.study.xlsx

@Mestizia Mestizia closed this as not planned Won't fix, can't repro, duplicate, stale Jul 10, 2024
@Mestizia Mestizia reopened this Jul 10, 2024
@Freymaurer
Copy link
Collaborator

Freymaurer commented Jul 11, 2024

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
Copy link
Collaborator

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
Copy link
Collaborator

Will close for now, feel free to reopen

@arendd
Copy link

arendd commented Sep 24, 2024

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 Freymaurer reopened this Sep 24, 2024
@github-actions github-actions bot added the Status: Needs Triage This item is up for investigation. label Sep 24, 2024
@Freymaurer
Copy link
Collaborator

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

@arendd
Copy link

arendd commented Sep 24, 2024

Sure, here it is: isa.study.xlsx

@Freymaurer Freymaurer self-assigned this Sep 24, 2024
@Freymaurer Freymaurer removed the Status: Needs Triage This item is up for investigation. label Sep 24, 2024
@Freymaurer
Copy link
Collaborator

Thanks! Will look into this as soon as possible!

@arendd
Copy link

arendd commented Sep 26, 2024

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
Copy link
Collaborator

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
Copy link
Collaborator

@HLWeil bump

@Freymaurer Freymaurer transferred this issue from nfdi4plants/Swate Oct 23, 2024
@github-actions github-actions bot added the Status: Needs Triage This item is up for investigation. label Oct 23, 2024
@HLWeil
Copy link
Member

HLWeil commented Oct 23, 2024

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

@github-project-automation github-project-automation bot moved this from Ready to Done in ARCStack Oct 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Something is not working, and it is confirmed by maintainers to be a bug.
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants