openworm / openworm/c302

List and dictionary command line arguments from README not executable in some shells

Open
#36 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Python
Stars
156
Forks
37
PR merge metrics
No merged PRs in 30d

Description

This command in the README:
c302 MyNetwork parameters_C -cells ["AVBR","VD3"] -cellstostimulate ["AVBR"] -paramoverride {"unphysiological_offset_current":"2.9pA"} -duration 300

Does not execute as-is in some shells, like ZSH, which attempt to parse ["AVBR","VD3"] as a shell glob and fail. (For ZSH, the failure produces a message, zsh: no matches found: [AVBR,VD3], rather than removing quote marks and passing [AVBR,VD3] to c302 as Bash apparently does.) ["AVBR"] is handled similarly. On the other hand, {"unphysiological_offset_current":"2.9pA"} apparently isn't interpreted as a glob and is handled as intended.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start in the README at the c302 command using -cells, -cellstostimulate, and -paramoverride. Run the example in ZSH and Bash to compare argument handling, then update the README example so the list and dictionary arguments execute as intended in the affected shells.

Written by the indexing model from the issue text.

Assessment

Tech stack
bash, python, zsh
Domain
cli, documentation
Issue type
Documentation
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.