search

nfdi4plants / ARCCommander

Tool to manage your ARCs
MIT License
11 stars 9 forks source link

[Feature Request] STUDY PUBLICATIONS section in isa.study.xlsx #197

Closed Brilator closed 10 months ago

Brilator commented 1 year ago

Is your feature request related to a problem? Please describe.

The default "study" sheet was reduced somewhere between arc v0.4 and v0.5.

Unfortunately, one (IMO important) section was lost during this update: STUDY PUBLICATIONS

STUDY PUBLICATIONS  
Study Publication PubMed ID  
Study Publication DOI  
Study Publication Author List  
Study Publication Title  
Study Publication Status  
Study Publication Status Term Accession Number  

Study Publication Status Term Source REF

Describe the solution you'd like This section is very useful to reference external data and I'd suggest to add it back.

HLWeil commented 1 year ago

The section still exists in the isa.investigation.xlsx. IMO, keeping both sections synced causes too much overhead and too many sources for error compared to the added benefits.

Brilator commented 1 year ago

Hm, but then why keep the Study sheet anyways? Just for Study Identifier and Study File Name

HLWeil commented 12 months ago

https://github.com/nfdi4plants/ARCtrl/issues/167 We are considering moving the study section to the study file altogether. This is not how it's like in ISA-Tab, but it could be a possibility in ISA-XLSX, as we do have sheets. This could be more intuitive for the user to have all information about a given object (e.g. study) in a single file.

What do you think, @Brilator?

HLWeil commented 12 months ago

*moving includes removing from investigation file

Brilator commented 11 months ago

Makes sense in terms of everything as one package, e.g. if I want to share only one study or assay (folder) rather than the whole ARC. Might however come with a bit of (teaching/explaining) overhead, if we deviate from original ISA too much.

HLWeil commented 11 months ago

Also at the cost of straying further from standard ISA...