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

Add a simpler example #3

Open
choldgraf opened this issue Apr 24, 2019 · 7 comments
Open

Add a simpler example #3

choldgraf opened this issue Apr 24, 2019 · 7 comments

Comments

@choldgraf
Copy link
Contributor

The demo for this project is really cool - though it is a bit complex. I stepped through the GUI, but was still a bit unclear how I could integrate the script into my own workflow. For example, the "usage" section says that you don't technically need Jupyter to run this package, and uses the command line in its sample code, whereas the example utilizes a pre-rendered GUI. I think there should be a minimal example showing the XML structure that this supports, the command to process it, and the kind of output it generates.

@choldgraf
Copy link
Contributor Author

Another thing I noticed is that the structure of the python configuration file isn't super clearly explained (I also don't see a CLI API documentation in the README anywhere)

@rheiland
Copy link
Owner

rheiland commented May 6, 2019

Integrating into one's own workflow currently assumes that workflow involves a custom PhysiCell simulation. I've tried to emphasize this in the README (as well as the paper). I did provide a minimal example showing the supported XML structure in the README. For my part, I think I've been a bit confused about what needs to be said in a JOSS paper vs. a JOSS repo. Obviously, it doesn't hurt to duplicate, but it does seem a bit redundant.

Not sure I understand the comment about a CLI API doc. Don't think there's enough there for an API, but maybe I'm misunderstanding.

@choldgraf
Copy link
Contributor Author

re: the CLI stuff, I think I had mis-interpreted the functionality of the package, so feel free to dismiss that :-)

re: what should go into the JOSS paper vs. the repository, I think the repository will have much more potential viewership than the paper, so any "useful" information should be in the repo documentation IMO

@choldgraf
Copy link
Contributor Author

Looking through this more, I think the documentation would benefit from some basic information about the inputs and outputs. E.g. many Python modules have a section like

Inputs
------
input_parameter : type
  description

Outputs
-------
output_one : type
  description

and most CLI modules have help associated with them (e.g. if you used argparse to parse inputs, you could provide help for each input).

Could you add in the README a section that explains what each input to the script is, an example of how it's structured, etc, as well as the script will output? E.g., it seems like it outputs a full python module that's meant to be imported (but can only be done from within the same folder since it won't be in the PATH)

@choldgraf
Copy link
Contributor Author

Another question: I've gotten this to correctly output a widget that shows the configuration values. But I haven't figured out how to make the plot with visualizations etc. The documentation says there are "static files" you can add somewhere, but there's no example or instructions for how to do so. Can you add this into the README?

@rheiland
Copy link
Owner

Added documentation header to xml2jupyter.py. Still not using argparse, but familiar with it and may reconsider in the future. Also added brief descriptions of modules in README.

Added section in README describing how one runs the full GUI (with the sample data visualized).

@choldgraf
Copy link
Contributor Author

Nice - that definitely helps. I added that content to the README in a pull request here: #6 I think that would make it more discoverable

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

No branches or pull requests

2 participants