✋ I am reviewing this. @alk224 thanks for your submission!

from joss-reviews.

from joss-reviews.

@arfon great, I'll get into it

from joss-reviews.

Hi @arfon. So sorry about the silence. I'm slowly reaching my goals. Problems with the installer we're identified last week, but until it is fully updated, I still can't address the reviewer comments with respect to silent modifications of .bashrc file. I'm still interested and will advise as soon as this is ready to move back toward review. Thanks for all your patience.

from joss-reviews.

Hi @alk224 - thanks for submitting this. Before we can proceed with this submission please take a look at the author guidelines (http://joss.theoj.org/about#author_guidelines) on how to format the paper.md file. Specifically, the metadata needs to be formatted exactly as the example shows below. In addition, the references need to be put in a separate bibtex file named paper.bib and citations to these entries made using this formatting.

---
title: 'Fidgit: An ungodly union of GitHub and figshare'
tags:
  - example
  - tags
  - for the paper
authors:
 - name: Arfon M Smith
   orcid: 0000-0002-3957-2474
   affiliation: GitHub Inc.
date: 14 February 2016
bibliography: paper.bib
---

# Summary

Your summary here....

# References

from joss-reviews.

Thanks Arfon,

Will try to make the necessary changes this weekend. I appreciate the help.

Andrew Krohn
PhD Candidate, NAU Biological Sciences
Research Technician, EnGGen

On Mon, Sep 5, 2016 at 9:50 PM, Arfon Smith [email protected]
wrote:

Hi @alk224 https://github.com/alk224 - thanks for submitting this.
Before we can proceed with this submission please take a look at the author
guidelines (http://joss.theoj.org/about#author_guidelines) on how to
format the paper.md file. Specifically, the metadata needs to be
formatted exactly as the example shows below. In addition, the
references need to be put in a separate bibtex file named paper.bib and
citations to these entries made using this formatting
http://rmarkdown.rstudio.com/authoring_bibliographies_and_citations.html#citations
.

---

title: 'Fidgit: An ungodly union of GitHub and figshare'
tags:

• example
• tags
• for the paper
  authors:

  • name: Arfon M Smith
    orcid: 0000-0002-3957-2474
    affiliation: GitHub Inc.
    date: 14 February 2016
    bibliography: paper.bib

Summary

Your summary here....

References

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#57 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEDNZpP-vJy5BZv0xdOfKyw55sXRSMQ8ks5qnPEIgaJpZM4J1c6R
.

from joss-reviews.

I believe I have fixed the issues raised here. The paper.md should be properly formatted and there is a bibtex file containing the references. I used the tool here to convert to bibtex and cleaned up the output manually. I also used this tool from the JOSS page to produce a metadata file (codemeta.json). Do you require that I also issue a new release that wraps all of these new files into it?

from joss-reviews.

Do you require that I also issue a new release that wraps all of these new files into it?

Nope, we don't need those. Thanks for asking.

from joss-reviews.

/ cc @openjournals/joss-reviewers - would anyone be willing to review this submission?

If you would like to review this submission then please comment on this thread so that others know you're doing a review (so as not to duplicate effort). Something as simple as :hand: I am reviewing this will suffice.

Reviewer instructions

• Please work through the checklist at the start of this issue.
• If you need any further guidance/clarification take a look at the reviewer guidelines here http://joss.theoj.org/about#reviewer_guidelines
• Please make a publication recommendation at the end of your review

Any questions, please ask for help by commenting on this issue! 🚀

from joss-reviews.

@whedon commands

from joss-reviews.

✋ I am reviewing this. Thanks for your submission @alk224

from joss-reviews.

Here are some things you can ask me to do:

# List all of Whedon's capabilities
@whedon commands

# Assign a GitHub user as the reviewer of this submission
@whedon assign @username as reviewer

# List the GitHub usernames of the JOSS editors
@whedon list editors

# List of JOSS reviewers together with programming language preferences and domain expertise
@whedon list reviewers

# Change editorial assignment
@whedon assign @username as editor

# Open the review issue
@whedon start review

🚧 Important 🚧

This is all quite new. Please make sure you check the top of the issue after running a @whedon command (you might also need to refresh the page to see the issue update).

from joss-reviews.

@whedon editors

from joss-reviews.

@whedon assign @arfon as editor

from joss-reviews.

OK, the editor is @arfon

from joss-reviews.

@whedon assign @tracykteal as reviewer

from joss-reviews.

OK, the reviewer is @tracykteal

from joss-reviews.

Friendly bump on this @tracykteal 😁

from joss-reviews.

Someone just showed me a bug in the akutils test command. This happened as a result of a recent rewrite of the strip_primers command. The bug is fixed and the test command should function properly now.

from joss-reviews.

Friendly bump on this @tracykteal 😁

👋 @tracykteal - are you still able to complete this review for JOSS?

from joss-reviews.

@alk224 - looks like @tracykteal isn't going to be able to do this review for us. Could you suggest any people from your research field who could potentially help with a review here?

from joss-reviews.

from joss-reviews.

@arfon @genomematt thanks for your help. I asked some colleagues about anyone they might suggest as well.

from joss-reviews.

I haven't gotten any responses from willing individuals. To be clear for whomever does review this, you simply need a clean Ubuntu 14.04 OS installed natively or within a VM to run everything. Anyone new willing to take on this review? Please?

from joss-reviews.

@serine, you willing to review this?

from joss-reviews.

Thanks @drpowell, @alk224 yeh I'll be keen to do it. I've done a bit of work on 16S like profiling (viruses instead of bacteria) and used usearch and its related tools, but I also used vsearch fairly extensively. My qiime knowledge are more theoretical than practical, never the less I've used most of the tools that qiime wraps around and will be happy to write it up.

p.s @alk224 easy and clean install with miniconda, pip install also works fine, but user might need to install some tools manually.

p.s.s qiime2 is coming sometime soonish.. don't want to wait for it to come out? I think its mainly migration to python3

from joss-reviews.

@whedon assign @serine as reviewer

from joss-reviews.

OK, the reviewer is @serine

from joss-reviews.

Thanks @drpowell, @alk224 yeh I'll be keen to do it.

Wonderful, thanks @serine. Please take a look at the reviewer guidelines (http://joss.theoj.org/about#reviewer_guidelines) to get started. Basically, you just need to work through the check list above as you proceed. Feel free to ask questions here of the submitting authors.

from joss-reviews.

@serine Thanks so much!

miniconda install came after I started working on this. The installer here: https://github.com/alk224/akutils_ubuntu_installer as per the instructions here: http://alk224.github.io/akutils-v1.2/ will do all of the work for you.

QIIME2 will be released pretty soon, yes, but this is all based on 1.9/1.9.1 so if it doesn't get out soon it will be obsolete. The QIIME team have reached out about setting up an akutils module for QIIME2, but I just don't have the time right now. I have several data sets that have been processed with this pipeline now so it would be nice to have something to reference directly.

from joss-reviews.

@alk224 Hey sorry took a bit of time to get to this and I also mis-understood what I was reviewing.. I though I was reviewing qiime not akutils - I got it now :).

Your license is missing year and holder

Copyright (c) <year> <copyright holders> bottom of the license file

from joss-reviews.

@alk224 with your git repo. It's a little confusing (to me at least). Usually you'd have a single repo e.g akutils and git clone http://blah/akults.git and then you have branches and tags (releases) with different versions. I can see that you have at least one other akutils repo - akutils-v1.1.1. I'm sure you have a reason for that, but little confusing. I guess v1.2 is the next iteraction of v1.1 ?

from joss-reviews.

Am I missing something.. but can't get it going...

[bioinformatics-platform2]~/downloads/akutils-v1.2 ((v1.2.3))$ bash install 

Command autocompletion already configured for akutils.
	
Command PATH already configured for akutils.
	
akutils is properly configured. Restart your terminal or run:
source ~/.bashrc
	
[bioinformatics-platform2]~/downloads/akutils-v1.2 ((v1.2.3))$ akutils test
akutils: command not found
[bioinformatics-platform2]~/downloads/akutils-v1.2 ((v1.2.3))$ source ~/.bashrc
[bioinformatics-platform2]~/downloads/akutils-v1.2 ((v1.2.3))$ akutils test
akutils: command not found

I can get it going this way, but then I get stuck at the config step and it just hangs there

[bioinformatics-platform2]~/downloads/akutils-v1.2 ((v1.2.3))$ ./scripts/akutils test

No config file is present.  Running config utility to establish configurable
options.
	

This will help you configure your akutils config file for running akutils workflows.

First, would you like to configure your global settings or make a local config file
to override your global settings?  A local config file will reside within your
current directory.

Or else, you can choose rebuild if you want to generate a fresh global config file.
This is useful if you recently updated akutils and need to integrate newly available
configuration options.

Enter "global," "local," or "rebuild."

local

OK. Checking for existing config file in current directory.
(/home/kirill/downloads/akutils-v1.2/)
	

No config file detected in local directory. Creating new local config file.

from joss-reviews.

@arfon didn't mean to close this issue... my mistake, sorry

from joss-reviews.

@serine Sorry about the confusion and the .bashrc. I can add in the license information.

Sometimes the source .bashrc doesn't seem to work and you just have to do a reboot. Also, I've had a few of the sources from the install script miss something this time or that and then just rerun the installer and it usually fills everything in. The .bashrc changes are meant to make it easy to use the akutils commands in a cluster environment where you can call all of the necessary modules rather than having the power to install stuff yourself. I also really enjoy autocomplete (referenced via .bashrc). v1.2 is much better than v1.1 which was my first major coding experience. It is rather clumsily written and lacks command integration and such. I transferred all of the files from v1.1 to 1.2 and rewrote most everything. I'm not good enough at git to really make use of branches, but I should have several releases of v1.2. It would make sense to run
akutils update
after a reboot to ensure everything is working right. I just had to fix a bug due to a change in the biom command syntax which was breaking a few other commands.

from joss-reviews.

@serine meant to use a code block there. Do:

akutils update  

from joss-reviews.

@alk224 sorry but I can't get it working. I took a quick pick at your install script and I couldn't figure out what you are doing with the PATH variable.. if I simply export PATH=/path/to/akutils-v1.2/scripts:$PATH I can get akutils on cmd, but still the same problem - it finds no existing config file, then gives me options to select from to presumably make new config - I pick local.. and nothing happens. I think this is actually a bug. I took a pick at your scripts/configure_slave.sh line 120 has an if block with only echo nothing else happens in this block

from joss-reviews.

@arfon Is this intentional to have this question twice?

 A statement of need: Do the authors clearly state what problems the software is designed to solve and who the target audience is?

Its under Documentations and under Software paper

from joss-reviews.

@serine Sorry about that. It is entirely likely there is a bug as all the sources seem to shift freely and without announcing that they will do so. I'll run a clean install on a VM in the morning and hopefully get this all working again.

from joss-reviews.

I think I missed something..Can akutils work without VM? or can it only work in a particular environmnet? Provided I have all of the tools installed it should work right?

from joss-reviews.

Its under Documentations and under Software paper

Yes, this is intentional. The point being that there should be a statement of need (presumably the same one?) in both the documentation and the paper.

from joss-reviews.

right, cool

from joss-reviews.

@serine You want a clean Ubuntu 14.04 environment. See http://alk224.github.io/akutils-v1.2/

from joss-reviews.

What does that mean? Like you want no varibales set in your current shell? Can I just get a new shell and unset everything, something like this env -i env ?

from joss-reviews.

@serine Generally just a fresh install using the 14.04.1 .iso. You could use a system that has other things installed, but given all the nuance of a QIIME install, you may find some things present or absent or incorrect versions of dependencies here and there. I have tried to enforce compatible versions during the install which could further break existing tools that you may like and want. I've never run env -i env so I'm not sure what that result might be. I know the installer will not work on Ubuntu 12.04 or 16.04 due to differences in package names in the respective repositories and probably similarly with the intermediate versions as well.

from joss-reviews.

To me first point is confusing

1. Set up your system appropriately. I have done this on a large workstation as well as within a VirtualBox on my netbook. Install Ubuntu 14.04LTS. I have had trouble with 14.04.3 base installs so prefer 14.04.1 install and then update:
   http://old-releases.ubuntu.com/releases/14.04.2/ubuntu-14.04.1-desktop-amd64.iso

I think you should either remove it or simplify to something like requires ubuntu 14.04.1 distro. It is still very restrictive. I would have thought that at least within ubuntu everything should be fairly similar. Not sure if you want to go this far, but docker or other containers might be great - easy install solution.

For now lets assume my env has been set appropriately and assume I have all dependencies installed

I've done 3 and 4. Now upto running akutils step.. and I'm having issues with configuration step..

would you be able to have a look in you scripts/configure_slave.sh line ~ 120 (I think in github its 123). There is an if block there with one statement echo blah and nothing else. I fall into that if block leading to akutils hanging until I ctrl^C ...

from joss-reviews.

@serine I could specify 14.04.1, but that wouldn't strictly be true. My workstation at work is now on 14.04.5 because I continue to run all security updates. Ubuntu repos change with every release, and for some reason as they pack more bits into subsequent iterations of the same release (.1, .2 etc), the system behavior can change here and there. Annoying yes. I do not think I have the time to learn of something like docker given the complexity of the existing installer which I use to restore my workstations (I manage several, though I am a biologist by training) when someone messes them up or the system drive fails.

Curious on the config script. I checked that line of code and there is an open if statement, but it closes later. Look closely and you will see a series of nested conditionals which all close at lines 163, 169 and 170.

Perhaps there is an issue with the config script in that release (or did you clone the repo? That's what I would do). If you downloaded the release instead, try running a git pull from within the akutils directory and then try the akutils command with no arguments.

from joss-reviews.

My bad, yes if statement has bunch of other stuff in it, It was hard to read, cause of differnet indentations style.

I followed installtion guide. I git clone http:/blah/auktils-v1.2 I tested that, didn't work for me. I then git checkout v1.2.3 which is your current release right? but still can't get pass configuration step

Any ideas what I am doing wrong?

from joss-reviews.

That was one of my first exercises with conditionals. Some of my inexperience will still be glaring in terms of indents and such.

Can you delete your cloned akutils repo rm -r ~/akutils-v1.2

Then try a fresh clone of the repo in your $HOME directory followed by the bash install command from within the cloned directory?

from joss-reviews.

[bioinformatics-platform2]~$ pwd
/home/kirill
[bioinformatics-platform2]~$ echo $HOME
/home/kirill
[bioinformatics-platform2]~$ git clone https://github.com/alk224/akutils-v1.2.git
Cloning into 'akutils-v1.2'...
remote: Counting objects: 2870, done.
remote: Compressing objects: 100% (44/44), done.
remote: Total 2870 (delta 30), reused 0 (delta 0), pack-reused 2826
Receiving objects: 100% (2870/2870), 7.33 MiB | 2.79 MiB/s, done.
Resolving deltas: 100% (1805/1805), done.
Checking connectivity... done.
Checking out files: 100% (342/342), done.
[bioinformatics-platform2]~$ cd akutils-v1.2/
[bioinformatics-platform2]~/akutils-v1.2 (master)$ bash install 

Command autocompletion successfully configured for akutils.
		
Command PATH successfully configured for akutils.
		
akutils is properly configured. Restart your terminal or run:
source ~/.bashrc
	
[bioinformatics-platform2]~/akutils-v1.2 (master)$ source ~/.bashrc
[bioinformatics-platform2]~/akutils-v1.2 (master)$ akutils 

No config file is present.  Running config utility to establish configurable
options.
	

This will help you configure your akutils config file for running akutils workflows.

First, would you like to configure your global settings or make a local config file
to override your global settings?  A local config file will reside within your
current directory.

Or else, you can choose rebuild if you want to generate a fresh global config file.
This is useful if you recently updated akutils and need to integrate newly available
configuration options.

Enter "global," "local," or "rebuild."

local

OK. Checking for existing config file in current directory.
(/home/kirill/akutils-v1.2/)
	

No config file detected in local directory. Creating new local config file.

EDITED I didn't have to export both autocomplete and export PATH were added into ~/.bashrc fine this time. But still get stuck at the configuration step

from joss-reviews.

Does it hang right there?

from joss-reviews.

And did you restart your terminal? I see you entered your own $PATH command.

from joss-reviews.

yep, and yep i source ~/.bashrc

from joss-reviews.

OK, thanks for persisting. I will check it out in the morning at work on a clean VM.

from joss-reviews.

can you refresh this github page. I removed - edited my previous comment. Yours worked fine after source ~/.bashrc

from joss-reviews.

Wait it is working now?

from joss-reviews.

I can run akutils but it hangs at the configuration step :)

[bioinformatics-platform2]~/akutils-v1.2 (master)$ pwd
/home/kirill/akutils-v1.2
[bioinformatics-platform2]~/akutils-v1.2 (master)$ which akutils 
/home/kirill/akutils-v1.2/scripts/akutils
[bioinformatics-platform2]~/akutils-v1.2 (master)$ akutils 

No config file is present.  Running config utility to establish configurable
options.
	

This will help you configure your akutils config file for running akutils workflows.

First, would you like to configure your global settings or make a local config file
to override your global settings?  A local config file will reside within your
current directory.

Or else, you can choose rebuild if you want to generate a fresh global config file.
This is useful if you recently updated akutils and need to integrate newly available
configuration options.

Enter "global," "local," or "rebuild."

local

OK. Checking for existing config file in current directory.
(/home/kirill/akutils-v1.2/)
	

No config file detected in local directory. Creating new local config file.

from joss-reviews.

I see. That is less catastrophic than I was thinking. I'll fix this in the morning. I appreciate you bringing it to my attention.

from joss-reviews.

no probs, thanks

from joss-reviews.

Starting on this now.

from joss-reviews.

Looks like there are issues with pip and python. No surprise there in my experience as they change available module names pretty continuously. I know this all worked when I submitted akutils for review back in October. I will get it working again and then ping you on this thread to get the review going again. I don't think I will be able to completely solve these issues today or even this weekend. I really appreciate your willingness to review, and your patience with these issues. If you have used QIIME in the past, you might be accustomed to the rather arduous installs which is what my installer script is supposed to circumvent.

from joss-reviews.

Yeh no problem when you ready.

Qiime has miniconda install now. I don't know how long they had it for, probably fairly recent, but it installs everything with ease. Might be worth while considering if you are having issues with pip install. From what I understand conda (miniconda) wraps everything into one "packages" such that there shouldn't be any issues with dependencies.

In general it would be great if you could do few more stdout/stderr messages to keep user upto date with the progress. Like with that config option - I couldn't be sure if akutils was doing something or not. It might have been downloading and installing all the pip packages, but as far as I was conserned it was just handing there. Don't worry about this for now though.

p.s I'll be on annual next week, so might not hear from me that week

from joss-reviews.

How are we getting on here @alk224?

from joss-reviews.

Friendly reminder on this @alk224

from joss-reviews.

👋 @alk224, how are you getting on?

from joss-reviews.

from joss-reviews.

Hi Arfon, sorry for the delay. Working on some manuscripts right now so I can graduate and plan to attend to akutils later this fall.

OK no problem. I'll remind you again sometime in early October if that works?

from joss-reviews.

👋 @alk224 - are you ready to pick this up again?

from joss-reviews.

If @alk224 doesn't re-appear in the next week I'll assume she's no longer interested in pursing this publication.

from joss-reviews.

Thanks for the update @alk224. Just wanted to make sure you were still interested in pursuing this.

from joss-reviews.

@arfon thank you for hitting pause on this. Goals are being achieved (grad school completed, manuscripts prepared), but time remains short. Am hoping for time to really update this by December since by now Ubuntu 14.04 will see end of support in less than a year, and QIIME2 is now available with the wonderful provenance feature, but still lacks many options for performing a comprehensive ITS region analysis. As such, this should probably remain paused for now, and I will get back to this as soon as time allows, and simultaneously address reviewer concerns.

from joss-reviews.

I've been in touch with @alk224 over email and she's agreed that at this point in time, it's best if she withdraws this submission.

@serine - many thanks for all of your help with this review, I'm sorry this paper didn't make it through to publication.

from joss-reviews.

all good, thanks to everyone and good luck

from joss-reviews.