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

tool development slides improvements #184

Closed
2 of 3 tasks
bgruening opened this issue Jan 16, 2017 · 5 comments
Closed
2 of 3 tasks

tool development slides improvements #184

bgruening opened this issue Jan 16, 2017 · 5 comments
Assignees
Labels

Comments

@bgruening
Copy link
Member

bgruening commented Jan 16, 2017

  • repeat slides -> enumerate is not used
  • tests slide needs to use sim_size for pngs
  • hands-on steps for planemo test, include planemo lint
@lecorguille lecorguille self-assigned this Jan 18, 2017
@lecorguille
Copy link
Member

  • debug a wrapper with the good conda environment (without have to relaunch x time planemo) using
planemo --no_cleanup
cd /tmp/tmpQ2UkMa/job_working_directory/000/9/working
../tool_script.sh

@nsoranzo nsoranzo mentioned this issue Jun 29, 2017
6 tasks
@nsoranzo
Copy link
Member

nsoranzo commented Jun 29, 2017

tests slide needs to use sim_size for pngs

For the test copy-pasted from graphlan wrapper:

<output name="png_output_image" file="png_image.png" />

actually works fine.

@bgruening
Copy link
Member Author

What do you think about dropping the explanation of the parameters from the slides. It takes a long time and parts of it is explained in the planemo docs and are also easy to grasp. It took a long time during the training imho.

@nsoranzo
Copy link
Member

We can shorten it, but I'd keep some.

@lecorguille
Copy link
Member

  • hands-on steps for planemo test, include planemo lint
    Done within the Planemo tutorial

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

No branches or pull requests

3 participants