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] Adding assay performer fails and argument -s is irritating #214

Closed
andreaschrader opened this issue Aug 30, 2023 · 1 comment · Fixed by #219
Closed

[BUG] Adding assay performer fails and argument -s is irritating #214

andreaschrader opened this issue Aug 30, 2023 · 1 comment · Fixed by #219

Comments

@andreaschrader
Copy link
Member

Describe the bug

When registering an assay performer, the argument -a is not recognized.

arc a person register -s MY_STUDY -a MY_ASSAY -l LASTNAME -f FIRSTNAME

and

arc a person register-a MY_ASSAY -l LASTNAME -f FIRSTNAME

do not work:

ERROR: ERROR: unrecognized argument: '-a'.

When reading the help, it says --assayidentifier or -s. This is irritating. However I executed this as well.

arc a person register -s MY_ASSAY -l LASTNAME -f FIRSTNAME

Start processing command with the arguments:
Address:Field ""
Affiliation:Field ""
AssayIdentifier:Field "MY_ASSAY"
Email:Field ""
Fax:Field ""
FirstName:Field "FIRSTNAME"
LastName:Field "LASTNAME"
MidInitials:Field ""
ORCID:Field ""
Phone:Field ""
Roles:Field ""
RolesTermAccessionNumber:Field ""
RolesTermSourceREF:Field ""

Start Person Register
Done processing command.
ERROR: System.Exception: Metadata sheetname Investigation could not be found System.Exception: Metadata sheetname Investigation could not be found
at Microsoft.FSharp.Core.PrintfModule.PrintFormatToStringThenFail@1448.Invoke(String message)
at ArcCommander.APIs.AssayAPI.Contacts.register(ArcConfiguration arcConfiguration, FSharpMap2 personArgs) at ArcCommander.Program.handleAssayContactsSubCommands@128-2.Invoke(ArcConfiguration arcConfiguration, FSharpMap2 personArgs)
at ArcCommander.Program.processCommand[a,T](ArcConfiguration arcConfiguration, FSharpFunc2 commandF, ParseResults1 r)
at ArcCommander.Program.handleAssayContactsSubCommands(ArcConfiguration arcConfiguration, AssayPersonCommand contactsVerb)
at ArcCommander.Program.handleAssaySubCommands(ArcConfiguration arcConfiguration, AssayCommand assayVerb)
at ArcCommander.Program.handleCommand(ArcConfiguration arcConfiguration, ArcCommand command)
at ArcCommander.Program.main(String[] argv)

Moreover, the assay performer fields in the isa.assay.xlsx file remain empty and nothing is added to the isa.investigation.xlsx file.

Expected behavior
arc a person register -a MY_ASSAY -l LASTNAME -f FIRSTNAME

with
--assayidentifier or -a to pass the assay identifier

and with the information being added to the isa.assay.xlsx file as well as to the isa.investigation.xlsx file.

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

  • OS: macOS
  • OS Version: Ventura 13.5.1
  • ARC Commander Version: v0.5.0
@HLWeil
Copy link
Member

HLWeil commented Sep 4, 2023

Ah okay I see, the short-hand flag is of course flat out wrong and needs to be updated.

We're currently working on integrating the new back-end https://github.com/nfdi4plants/ARCtrl into the ArcCommander (It is also the back-end for the ARCitect). This should also fix many of the issues the ArcCommander has, as the ARCtrl is already well tested.
Just as a heads-up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants