Camelia, the Perl 6 bug

IRC log for #bioperl, 2013-02-20

| Channels | #bioperl index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
00:04 deafferret here's where my demo would help. your %args and %args2 is messy. It'd be obvious if you had a single query_param_append()
00:04 deafferret https://github.com/jhannah/san​dbox/blob/master/census/uri.pl
00:09 census_ thank you for this again!  what exactly is line 9 about?  or more so, i should say the 17 and 12?
00:10 deafferret did you run the program?
00:10 deafferret they're just examples
00:11 deafferret the point is you can have ANY arbitrary list there, and they're added to the $uri object
00:12 census_ no because when you said copy and paste ugliness, i was not sure which ugliness?  obviously the lines before i even input any data just to get into the site  . . .i'm sorry if that's a really bad q
00:13 deafferret it's ugly, but not ugly enough that I would bother changing it
00:17 census_ my mouse is acting screwy...i'm going to reboot.
00:17 census_ i'll be right back
00:17 census_ i was confused by ugliness i think
00:17 census_ i think i thought you meant ugliness = lines 1-70 of my code
00:17 census_ but i think you meant you merely copy+pasted the 1 ugly part of the url that i had hard-coded
00:17 census_ both of which are ugly
00:18 census_ i'm goign to reboot, hopefully that fixes the mouse problem, and then i'lll run teh code.
00:18 census_ i do think i'll need to incorporate the other ugly, lines 1-70 . . .(or more like 1-50 or so .. .the part before i input any data . . .)
00:19 deafferret 'sse.dll?db=1940usfedcen&rank=1&...' in source code is ugly. that's all I meant.
00:19 deafferret Not your fault, that's their URL.
00:19 deafferret pretty source code looks like   (db => '1940usfedcen', rank => 1, ...)
00:20 deafferret source code, ideally, doesn't have long squished strings of goop in it
00:20 deafferret separating logic from formats is always good.
00:21 deafferret but, in this case, not worth bothering with, in my opinion
00:21 deafferret for the initial $uri. But it IS worth bothering with for all the arguments that YOU'RE ADDING with your logic
00:21 deafferret which, currently, is hard to keep track of. in my opinion.
00:27 census joined #bioperl
00:27 census sorry about that.  mouse seems to be working much better, at least :)
00:29 census sorry, i cannot find the backlog.  do you mind showing me that script again?  i lost it with my bad mouse . . .
00:30 deafferret https://github.com/jhannah/san​dbox/blob/master/census/uri.pl
00:31 deafferret census: reload, I added my comments to the bottom
00:42 census mouse is acting up again.
00:42 census but i ran the code and i think i see what is goign on
00:42 census it takes the keys and sticks them to the end (the order in which they go in the url does nto matter), and for the moment, you've assigned them values of 12 and 17, respectively
00:43 deafferret yup. the point being that you turn add a perl hash to the URI object without doing your own (probably buggy) logic to escape and do so
00:45 census_ joined #bioperl
00:45 census_ ok,i'm on a different computer for the moment (which has keyboard issues)
00:46 census_ was i correct?
00:46 deafferret yup
00:47 census_ i'm not fully sure though which lines from my original code would carry over and which those would replace
00:47 census_ i'm not sure how i'd read in th $l data, for example, if not with args . . .
00:53 carandraug joined #bioperl
00:53 carandraug joined #bioperl
02:00 carandraug joined #bioperl
03:49 carandraug joined #bioperl
12:46 ObseLeTe joined #bioperl
13:46 ObseLeTe joined #bioperl
13:57 census joined #bioperl
16:22 ObseLeTe joined #bioperl
17:14 looper joined #bioperl
17:16 census deafferrret:  Thank you so much for the help yesterday!! I figured that whole part of the code out ;)
17:41 ObseLeTe joined #bioperl
17:42 carandraug joined #bioperl
18:03 drin_m joined #bioperl
19:04 census_ joined #bioperl
19:04 census_ deafferret: are you around?
20:35 census joined #bioperl
20:47 carandraug joined #bioperl
20:58 census deafferret: Hey!
21:09 deafferret census: no, running off to an appt. might be back on late tonight
21:09 census ok.  sounds good.  thank you for all of the help last night.  good luck with your appointment!
21:10 deafferret thanks  :)  &
21:24 sl33v3_ joined #bioperl
22:33 github [bioperl-live] cjfields pushed 3 new commits to v1: https://github.com/bioperl/bioperl-liv​e/compare/37eb121b1dbf...b7d042d6393a
22:33 github bioperl-live/v1 3762441 Chris Fields: squash another warning from Graph module oddities
22:33 github bioperl-live/v1 631792d Chris Fields: accession no longer valid, using updated one
22:33 github bioperl-live/v1 b7d042d Chris Fields: add test and TODO for Issue #3416
22:36 github [bioperl-live] cjfields pushed 3 new commits to master: https://github.com/bioperl/bioperl-liv​e/compare/573b3f6b57ef...261c6accd926
22:36 github bioperl-live/master f56ef89 Paul Cantalupo: removing searchio-blastxml files
22:36 github bioperl-live/master 95d072f Paul Cantalupo: removing searchio-writer-bsmlresultwriter files
22:36 github bioperl-live/master 261c6ac Chris Fields: Merge branch 'master' of github.com:bioperl/bioperl-live

| Channels | #bioperl index | Today | | Search | Google Search | Plain-Text | summary