1 new changeset in galaxy-central:
http://bitbucket.org/galaxy/galaxy-central/changeset/6dc381fe8af9/
changeset: r5490:6dc381fe8af9
user: fubar
date: 2011-04-29 23:32:33
summary: second test for picard asmetrics fails on buildbot. This fix works on my own testbed - but then, it worked before too...
affected #: 1 file (42 bytes)
--- a/tools/picard/rgPicardASMetrics.xml Fri Apr 29 17:23:06 2011 -0400
+++ b/tools/picard/rgPicardASMetrics.xml Fri Apr 29 17:32:33 2011 -0400
@@ -62,6 +62,7 @@
<param name="sorted" value="true" /><param name="adaptors" value="" /><param name="maxinsert" value="100000" />
+ <param name="index" value="hg18" /><param name="refGenomeSource" value="indexed" /><param name="index" value="hg18" /><param name="input_file" value="picard_input_sorted_pair.sam" dbkey="hg18" />
Repository URL: https://bitbucket.org/galaxy/galaxy-central/
--
This is a commit notification from bitbucket.org. You are receiving
this because you have the service enabled, addressing the recipient of
this email.
1 new changeset in galaxy-central:
http://bitbucket.org/galaxy/galaxy-central/changeset/7ae63fde7f78/
changeset: r5488:7ae63fde7f78
user: dan
date: 2011-04-29 22:51:51
summary: Only allow a user to rerun if they have access permissions on the dataset.
affected #: 1 file (246 bytes)
--- a/lib/galaxy/web/controllers/tool_runner.py Fri Apr 29 15:32:04 2011 -0400
+++ b/lib/galaxy/web/controllers/tool_runner.py Fri Apr 29 16:51:51 2011 -0400
@@ -90,6 +90,9 @@
error( "Invalid value for 'id' parameter" )
# Get the dataset object
data = trans.sa_session.query( trans.app.model.HistoryDatasetAssociation ).get( id )
+ #only allow rerunning if user is allowed access to the dataset.
+ if not trans.app.security_agent.can_access_dataset( trans.get_current_user_roles(), data.dataset ):
+ error( "You are not allowed to access this dataset" )
# Get the associated job, if any. If this hda was copied from another,
# we need to find the job that created the origial hda
job_hda = data
Repository URL: https://bitbucket.org/galaxy/galaxy-central/
--
This is a commit notification from bitbucket.org. You are receiving
this because you have the service enabled, addressing the recipient of
this email.
1 new changeset in galaxy-central:
http://bitbucket.org/galaxy/galaxy-central/changeset/3496a98f9cbe/
changeset: r5487:3496a98f9cbe
user: greg
date: 2011-04-29 21:32:04
summary: Apply patch from taoliu - ticket #529 - Missing controller when creating user with require_login & allow_user_creatio is on.
affected #: 1 file (17 bytes)
--- a/lib/galaxy/web/controllers/user.py Fri Apr 29 13:59:11 2011 -0400
+++ b/lib/galaxy/web/controllers/user.py Fri Apr 29 15:32:04 2011 -0400
@@ -371,7 +371,7 @@
redirect_url = url_for( '/' )
if not user and trans.app.config.require_login:
if trans.app.config.allow_user_creation:
- header = require_login_creation_template % web.url_for( action='create' )
+ header = require_login_creation_template % web.url_for( action='create', cntrller='user' )
else:
header = require_login_nocreation_template
return trans.fill_template( '/user/login.mako',
Repository URL: https://bitbucket.org/galaxy/galaxy-central/
--
This is a commit notification from bitbucket.org. You are receiving
this because you have the service enabled, addressing the recipient of
this email.
1 new changeset in galaxy-central:
http://bitbucket.org/galaxy/galaxy-central/changeset/62a5d6d82151/
changeset: r5485:62a5d6d82151
user: fubar
date: 2011-04-29 18:29:13
summary: gcbias still failing on test - works fine here using same data - clean up of fasta munging
affected #: 2 files (166 bytes)
--- a/tools/picard/picard_wrapper.py Fri Apr 29 10:37:59 2011 -0400
+++ b/tools/picard/picard_wrapper.py Fri Apr 29 12:29:13 2011 -0400
@@ -361,19 +361,19 @@
tmp_dir = opts.outdir
# set ref and dict files to use (create if necessary)
- if opts.ref_file:
+ ref_file_name = opts.ref
+ if opts.ref_file <> None:
csd = 'CreateSequenceDictionary'
realjarpath = os.path.split(opts.jar)[0]
jarpath = os.path.join(realjarpath,'%s.jar' % csd) # for refseq
tmp_ref_fd, tmp_ref_name = tempfile.mkstemp( dir=opts.tmpdir , prefix = pic.picname)
- ref_file_name = '%s.fa' % tmp_ref_name
+ ref_file_name = '%s.fasta' % tmp_ref_name
# build dict
- ## need to change name of fasta to have fasta ext
- dict_file_name = ref_file_name.replace( '.fa', '.dict' )
+ dict_file_name = '%s.dict' % tmp_ref_name
os.symlink( opts.ref_file, ref_file_name )
cl = ['REFERENCE=%s' % ref_file_name]
cl.append('OUTPUT=%s' % dict_file_name)
- cl.append('URI=%s' % os.path.basename( ref_file_name ))
+ cl.append('URI=%s' % os.path.basename( ref_file ))
cl.append('TRUNCATE_NAMES_AT_WHITESPACE=%s' % opts.trunc_names)
if opts.species_name:
cl.append('SPECIES=%s' % opts.species_name)
@@ -381,11 +381,9 @@
cl.append('GENOME_ASSEMBLY=%s' % opts.build_name)
pic.delme.append(dict_file_name)
pic.delme.append(ref_file_name)
+ pic.delme.append(tmp_ref_name)
s = pic.runPic(jarpath, cl)
- elif opts.ref:
- ref_file_name = opts.ref
-
- # run relevant command(s)
+ # run relevant command(s)
cl = ['VALIDATION_STRINGENCY=LENIENT',]
@@ -446,13 +444,15 @@
elif pic.picname == 'CollectAlignmentSummaryMetrics':
+ # Why do we do this fakefasta thing? Because we need NO fai to be available or picard barfs unless it has the same length as the input data.
+ # why? Dunno
fakefasta = os.path.join(opts.outdir,'%s_fake.fasta' % os.path.basename(ref_file_name))
try:
os.symlink(ref_file_name,fakefasta)
except:
s = '## unable to symlink %s to %s - different devices? May need to replace with shutil.copy'
info = s
- shutil.copy(ref_file_name,fakefasta)
+ shutil.copy(ref_file_name,fakefasta)
pic.delme.append(fakefasta)
cl.append('ASSUME_SORTED=%s' % opts.assumesorted)
adaptorseqs = ''.join([' ADAPTER_SEQUENCE=%s' % x for x in opts.adaptors])
@@ -478,13 +478,13 @@
# sigh. Why do we do this fakefasta thing? Because we need NO fai to be available or picard barfs unless it has the same length as the input data.
# why? Dunno
fakefasta = os.path.join(opts.outdir,'%s_fake.fasta' % os.path.basename(ref_file_name))
- pic.delme.append(fakefasta)
try:
os.symlink(ref_file_name,fakefasta)
except:
s = '## unable to symlink %s to %s - different devices? May need to replace with shutil.copy'
info = s
- shutil.copy(ref_file_name,fakefasta)
+ shutil.copy(ref_file_name,fakefasta)
+ pic.delme.append(fakefasta)
x = 'rgPicardGCBiasMetrics'
pdfname = '%s.pdf' % x
jpgname = '%s.jpg' % x
--- a/tools/picard/rgPicardGCBiasMetrics.xml Fri Apr 29 10:37:59 2011 -0400
+++ b/tools/picard/rgPicardGCBiasMetrics.xml Fri Apr 29 12:29:13 2011 -0400
@@ -40,7 +40,7 @@
<param name="mingenomefrac" value="0.00001" type="float" label="Minimum Genome Fraction"
help="For summary metrics, exclude GC windows that include less than this fraction of the genome. Default value: 1.0E-5." /><param name="maxheap" type="select" help="If in doubt, choose 8G and read Picard documentation please"
- label="Java heap size. Java is a pig, the art is long but memory is short">
+ label="Java heap size"><option value="1G">1GB: very small data</option><option value="2G" selected="true">2GB</option><option value="4G">4GB for larger datasets </option>
Repository URL: https://bitbucket.org/galaxy/galaxy-central/
--
This is a commit notification from bitbucket.org. You are receiving
this because you have the service enabled, addressing the recipient of
this email.
1 new changeset in galaxy-central:
http://bitbucket.org/galaxy/galaxy-central/changeset/337605f6b630/
changeset: r5484:337605f6b630
user: natefoo
date: 2011-04-29 16:37:59
summary: When using a logout href, load that link at _top, not in the galaxy_main panel.
affected #: 1 file (7 bytes)
--- a/templates/webapps/galaxy/base_panels.mako Fri Apr 29 10:35:42 2011 -0400
+++ b/templates/webapps/galaxy/base_panels.mako Fri Apr 29 10:37:59 2011 -0400
@@ -132,7 +132,7 @@
menu_options = [ [ '<li>Logged in as <span id="user-email">%s</span></li>' % email ] ]
if app.config.use_remote_user:
if app.config.remote_user_logout_href:
- menu_options.append( [ 'Logout', app.config.remote_user_logout_href, "galaxy_main" ] )
+ menu_options.append( [ 'Logout', app.config.remote_user_logout_href, "_top" ] )
else:
menu_options.append( [ 'Preferences', h.url_for( controller='/user', action='index', cntrller='user' ), "galaxy_main" ] )
if app.config.get_bool( 'enable_tracks', False ):
Repository URL: https://bitbucket.org/galaxy/galaxy-central/
--
This is a commit notification from bitbucket.org. You are receiving
this because you have the service enabled, addressing the recipient of
this email.
1 new changeset in galaxy-central:
http://bitbucket.org/galaxy/galaxy-central/changeset/4933de88cdb0/
changeset: r5483:4933de88cdb0
user: natefoo
date: 2011-04-29 16:35:42
summary: Minor fix for setting the username during user creation under use_remote_user = True. Converts the user portion of the user's email to lowercase and replaces invalid characters with '-' as per the public name format requirements.
affected #: 1 file (231 bytes)
--- a/lib/galaxy/web/framework/__init__.py Thu Apr 28 22:07:58 2011 -0400
+++ b/lib/galaxy/web/framework/__init__.py Fri Apr 29 10:35:42 2011 -0400
@@ -437,14 +437,17 @@
if not user.default_permissions:
self.app.security_agent.user_set_default_permissions( user, history=True, dataset=True )
elif user is None:
- username = remote_user_email.split( '@', 1 )[0]
+ username = remote_user_email.split( '@', 1 )[0].lower()
random.seed()
user = self.app.model.User( email=remote_user_email )
user.set_password_cleartext( ''.join( random.sample( string.letters + string.digits, 12 ) ) )
user.external = True
+ # Replace invalid characters in the username
+ for char in filter( lambda x: x not in string.ascii_lowercase + string.digits + '-', username ):
+ username = username.replace( char, '-' )
# Find a unique username - user can change it later
if ( self.sa_session.query( self.app.model.User ).filter_by( username=username ).first() ):
- i = 0
+ i = 1
while ( self.sa_session.query( self.app.model.User ).filter_by( username=(username + '-' + str(i) ) ).first() ):
i += 1
username += '-' + str(i)
Repository URL: https://bitbucket.org/galaxy/galaxy-central/
--
This is a commit notification from bitbucket.org. You are receiving
this because you have the service enabled, addressing the recipient of
this email.