-
Notifications
You must be signed in to change notification settings - Fork 6
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] Changes to tables made outside of ARCitect not properly loaded #241
Comments
Can you share the example file? Then we will look into it 🙂 |
@Freymaurer sure, here is the file on OneDrive: https://1drv.ms/x/c/f72c4b5fae1f0d2e/ESJ36nx9u2dOo2AA__TxBo0BT7JlC3vz9mHYsovQU-0InQ?e=WN4D1C some more details: I started with 180 lines, one per sample. I have paired-end data so this means 360 files, so I wanted a line per file, so (outside of ARCitect) I duplicated the 180 lines, added filenames, then sorted on first column so each pair of lines from the same sample is shown together. ARCitect now shows me only the first 180 lines of the new sorted file. See both the "SRA - submissions" table as the "RNA-Seq" table |
I did some further testing as I felt it was related to nfdi4plants/ARCtrl#462 That issue refers to macos and windows, the issue in ubuntu is probably connected. Here is a mock table I created in arcitect (ubuntu): Opening it in open office gives no error, I added some data in open office as you can see: After saving and closing office, I open again arcitect and open the table. However, the values in rows that were not created in arcitect still do not show up (6). Additionally, columns where there was no data filled in before are also missing the values (5). Notably, cells for which there were already values in both row and column in arcitect do work (4). Further testing shows that I can successfully edit cells that were already filled in arcitect with open office and the changes will show up in arcitect. |
I do not have permission to access the isa.assay file 😅 |
oops, updated permissions, better now? https://1drv.ms/x/c/f72c4b5fae1f0d2e/ESJ36nx9u2dOo2AA__TxBo0BT7JlC3vz9mHYsovQU-0InQ?e=WN4D1C |
Yes! And i already found the issue! Swate (and underlying ARCtrl) work based on xlsx tables. Tip The content of the annotation table MUST be placed in an xlsx table whose name starts with annotationTable. Each sheet MUST contain at most one such annotation table. Only cells inside this table are considered as part of the formatted metadata. https://github.com/nfdi4plants/ARC-specification/blob/main/ISA-XLSX.md#annotation-table-sheets In your example file the table only spans to row 181. Therefore, ARCtrl ignores everything else.
Can you verifiy if this is the same issue? With the table not spanning row 3?
This is the "Term Source REF" column, which is hidden by default in Swate. Can you expand the composite column by clicking on this icon? |
@Freymaurer thanks for troubleshooting! ok... I learned something new about Excel files today ;P ..and more importantly I can continue with my ARC again, so thanks a bunch for the quick response! |
Yes it is there, sorry for the delay! |
Awesome! I'll close this issue for now, feel free to reopen! |
OS and framework information (please complete the following information):
Describe the bug
I sometimes need to edit my tables outside of ARCitect (e.g. for batch operations on large numbers of rows, this is not really convenient/doable yet in ARCitect). However, the changes I make outside of ARCitect are only partially detected, more specifically, any added rows are not loaded in ARCitect.
Steps to reproduce
The text was updated successfully, but these errors were encountered: